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
mpv: build and install macOS App Bundle #43612
Conversation
mkdir -p $out/Applications | ||
cp -r build/mpv.app $out/Applications | ||
wrapProgram $out/Applications/mpv.app/Contents/MacOS/mpv \ | ||
--add-flags "--scripts=${concatStringsSep "," scripts}" \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to refactor this to avoid duplication. In these cases it may make sense to just do:
mkdir -p $out/Applications
cp -r build/mpv.app $out/Applications
rm $out/Applications/mpv.app/Contents/MacOS/mpv
ln -s $out/bin/mpv $out/Applications/mpv.app/Contents/MacOS/mpv
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested this and mpv.app just quits whenever I try to open it, I don't know too much about macOS so I didn't manage to find any error messages related to this. The only method I know of was checking Console.app, but that doesn't show anything related.
At the very least I could move all the wrapProgram flags into a variable to avoid duplication that way.
d0609b4
to
80316cd
Compare
@GrahamcOfBorg build mpv edit: lol |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: enzime Partial log (click to expand)
|
No attempt on x86_64-linux (full log) The following builds were skipped because they don't evaluate on x86_64-linux: enzime Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: enzime Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: mpv Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: mpv Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: mpv Partial log (click to expand)
|
80316cd
to
0351e9e
Compare
@GrahamcOfBorg build mpv |
Failure on x86_64-darwin (full log) Attempted: mpv Partial log (click to expand)
|
Hmm, this worked before. |
Success on x86_64-linux (full log) Attempted: mpv Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: mpv Partial log (click to expand)
|
I am able to build it locally. |
@matthewbauer are you building this with mpv 0.29.0? |
0351e9e
to
f8fb13e
Compare
@GrahamcOfBorg build mpv |
Success on x86_64-linux (full log) Attempted: mpv Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: mpv Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: mpv Partial log (click to expand)
|
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)