Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/universal build of thirdparty binaries #282

Merged
merged 3 commits into from
Dec 8, 2024

Conversation

jmcintyre
Copy link
Collaborator

Update atomic parsley and mp4chaps to latest releases and use universal binary builds to that they work properly with both Intel and Apple Silicon Macs.
Relates to #278

@jmcintyre
Copy link
Collaborator Author

@griff This shouldn't be committed yet. I need to test the output of the build process to make sure signing works with the new builds of these binaries. Unfortunately, the only way I say to test the process was to put in a pull request.

@jmcintyre
Copy link
Collaborator Author

@griff That didn't quite work out as I'd hoped. I didn't realize the tests didn't generate a binary that was saved for testing. I'm not sure the how best to validate this.

@jmcintyre
Copy link
Collaborator Author

Updated with changes to atom management and support for newer command changes in the current AtomicParsley

@jmcintyre jmcintyre force-pushed the fix/universal-build-of-thirdparty-binaries branch 2 times, most recently from e77100e to b565707 Compare November 26, 2024 20:59
@griff
Copy link
Owner

griff commented Nov 26, 2024

We could change the build process to upload PR builds to the server where I used to host the website.

@griff
Copy link
Owner

griff commented Nov 26, 2024

I mean the server where I still host the website.

@griff
Copy link
Owner

griff commented Nov 26, 2024

When a PR is work in progress making it a draft might make sense. Don't know exactly how draft PR's work but we could test it.

@jmcintyre jmcintyre linked an issue Nov 27, 2024 that may be closed by this pull request
@jmcintyre jmcintyre marked this pull request as draft November 27, 2024 03:52
@jmcintyre
Copy link
Collaborator Author

jmcintyre commented Nov 27, 2024

Switched to draft. If there is a way to modify the build process so we can get a look at the build artifacts without generating another official pre-release, I think that would be useful.

@jmcintyre
Copy link
Collaborator Author

There is an issue with the mp4chaps trying to load a library from my build location on other computers. I'll need to fix that.

@jmcintyre jmcintyre force-pushed the fix/universal-build-of-thirdparty-binaries branch from b565707 to a5c6b06 Compare December 2, 2024 06:03
@jmcintyre
Copy link
Collaborator Author

I redid the mp4chaps binary so it can run standalone.

@jmcintyre
Copy link
Collaborator Author

@griff, I have pretty high confidence in the change itself now. I'm running my locally built binary on multiple computers, both Intel and Apple Silicon. In that sense, it is ready for review, but I don't understand the code signing process enough to know whether the version built on the server will resign the command line executables. Otherwise, they will just be signed from my development build.

@griff
Copy link
Owner

griff commented Dec 6, 2024

The only thing signed right now is the Sparkle update bundle and that is done with a local key I just generated years and years ago. I have never joined the Apple Developer Program and so have never set up code signing for MetaZ.

I looked over the changes and if you are good with them I am fine with merging this PR.

@jmcintyre
Copy link
Collaborator Author

Alright, I'll post it and we'll see what happens.

@jmcintyre jmcintyre marked this pull request as ready for review December 8, 2024 04:40
@griff griff merged commit 1571b27 into develop Dec 8, 2024
2 checks passed
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.

Not adding files to TV app
2 participants