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
molotov: init at 4.2.2 #92884
molotov: init at 4.2.2 #92884
Conversation
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
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.
Are the desktop icons set up right with this way?
https://github.com/NixOS/nixpkgs/pull/95116/files#diff-d1a400d53589e8ce54a2e73b92151e33R17-R25
extraInstallCommands = ''
mv $out/bin/${name} $out/bin/${pname}
install -m 444 -D \
${appimageContents}/${pname}.desktop \
$out/share/applications/${pname}.desktop
substituteInPlace $out/share/applications/${pname}.desktop \
--replace 'Exec=AppRun' 'Exec=${pname}'
cp -r ${appimageContents}/usr/share/icons $out/share
'';
integrated your variations, last week end I tried to only extract the binary from the appimage, but was stuck with a SIGTRAP error (I think due to chrome-sandbox requiring suid, but I am not sure). I added me and @apeyroux as maintainer also if you are ok ? |
You can try with https://github.com/apeyroux/molotov.nix I didn't have any problem during the execution (authentication and watching TV)
Yes |
Well using wrapType2 it is also ok on my side, it doesn't work if I try to extract just the app from the appimage + buildFHSEnv machinery |
@ofborg build molotov |
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.
You should only do one commit. I don't think that history will pass.
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
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.
diff LGTM
gui launches fine
https://github.com/NixOS/nixpkgs/pull/92884
1 package built:
molotov
Motivation for this change
Package online tv application molotov which lets French users to access a range of TV channels
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)