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
vlc: add skin support #109235
vlc: add skin support #109235
Conversation
Also, please squash your commits. |
IDK why this is happening but when I force push my squashed commits it pushes the commit before it too. The hash changes because the commit before change too. Probably for the merge it will be necessary to cherry-pick the commit to master. |
if you're doing
|
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 2 packages marked as broken and skipped:
21 packages failed to build and are new build failures:
1 package built:
|
Please fix the libvlc build failure. |
Signed-off-by: GitHub <noreply@github.com>
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 2 packages marked as broken and skipped:
21 packages failed to build and are new build failures:
1 package built:
|
Your last commit does not fix libvlc which I expected because currently it does nothing at all. |
Signed-off-by: GitHub <noreply@github.com>
I think now libvlc is fine xD When you asked to fix I didn't understand how to fix the problem then, before sleep, I realized that the problem is skins2 enabled in libvlc, that doesn't make sense, then I set skins2 default parameter to false if libvlc is being built. This is the first PR that I sent that I was scared because I broke something and I didn't know how to fix it but this kind of problem usually has the solution right before sleep xD. It was an honor question, I must find this solution. This is one of the things that makes programming awesome. |
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
This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch) Result of 1 package built:
The following issues got detected with the above build packages. vlc: Please consider this feature to be alpha: A substituteInPlace with an unused --replace got detected:
Please check the offending substituteInPlace for typos or changes in source. |
Motivation for this change
Lack of support to skins in the NixOS VLC compilation
Things done
Add this support as an option in the package
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)