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
yakyak init at 1.5.4-beta-rolling and (patched) electron-packager init at 14.0.6 #69393
Conversation
Hi, What's the next step for this PR please? Cheers |
For an easy merge of this, I'd suggest not using I know this is Nixpkgs' approach is inconsistent but it was mentioned before that this approach is better and as you can see, it's easier to maintain.. |
I would suggest splitting this PR into one pull request per package. |
Thank you for your contributions. This has been automatically marked as stale because it has had no activity for 180 days. If this is still important to you, we ask that you leave a comment below. Your comment can be as simple as "still important to me". This lets people see that at least one person still cares about this. Someone will have to do this at most twice a year if there is no other activity. Here are suggestions that might help resolve this more quickly:
|
still important to me |
Unsurprisingly, there are merge conflicts. You may find https://discourse.nixos.org/t/call-for-nodepackages-maintainers/9176/ relevant for this. |
I marked this as stale due to inactivity. → More info |
Still important for me. |
If this PR is still important to anyone, could this person feel responsible for applying reviewer's suggestions and resolving conflicts? |
it seems yakyak is mostly abandoned, I don't have interest in moving this forward, whoever's interested and wants to use the work I did thus far, feel free to open a new pr |
Motivation for this change
new package from #55289
Things done
the wildcards in this PR are two things, first is this patched wrapper around electron-packager, I'm sure I can reuse it for other electron packages in the future. It just prevents electron-packager from downloading the binaries and tells it to use the ones already provided from nixpkgs (so overriding the electron input would be the way to choose a different electron version).
The second wildcard is the hangupsjs, which had a terrible conflict in coffee-script. Using the version provided in the yakyak npm package causes compilation error, this syntax error is already a ticket here yakyak/hangupsjs#79 and when it's solved, I will bump and remove the hangusjs as a global nodePackage (and use it just as part of the yakyakBuldDeps).
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)