Skip to content

Combined dependabot and docklib PRs#30

Merged
erikng merged 17 commits intomainfrom
dependabot-and-docklib
Jul 8, 2021
Merged

Combined dependabot and docklib PRs#30
erikng merged 17 commits intomainfrom
dependabot-and-docklib

Conversation

@natewalck
Copy link
Copy Markdown
Member

This should be a bundle of all currently open PRs so it is easier to test the python compilation.

@natewalck natewalck requested a review from erikng July 8, 2021 14:25
@erikng
Copy link
Copy Markdown
Member

erikng commented Jul 8, 2021

Check is failing :/

@natewalck
Copy link
Copy Markdown
Member Author

This should make it more obvious which .so is not universal.

@natewalck
Copy link
Copy Markdown
Member Author

Woot:

Shared objects do not match, resulting in a non-universal Python framework.
Total shared objects found: 175
Universal shared objects found: 174
./recommended/payload//Library/ManagedFrameworks/Python/Python3.framework/Versions/Current/lib/python3.9/site-packages/yaml/_yaml.cpython-39-darwin.so
Error: Process completed with exit code 1.

@natewalck
Copy link
Copy Markdown
Member Author

I believe this is ready for review.

@natewalck natewalck requested a review from gregneagle July 8, 2021 20:19
@natewalck natewalck self-assigned this Jul 8, 2021
@erikng erikng mentioned this pull request Jul 8, 2021
@erikng
Copy link
Copy Markdown
Member

erikng commented Jul 8, 2021

The CI is failing, but after investigating it's because of the following:

if: github.ref == 'refs/heads/main'. The p12 certs are only created if on main, so the CI partially fails due to artifacts. We shouldn't be creating a release for PRs.

@erikng
Copy link
Copy Markdown
Member

erikng commented Jul 8, 2021

Check are passing! Merging.

@erikng erikng merged commit eb1558c into main Jul 8, 2021
@erikng erikng deleted the dependabot-and-docklib branch July 8, 2021 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants