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
riot-desktop: use own yarn2nix #58597
Conversation
github ref: closes NixOS#58593
I don't think vendoring moretea/yarn2nix this is the right approach. Can't you just use https://github.com/Profpatsch/yarn2nix and the required dependencies in a nix expression? I guess @Profpatsch will gladly help in case you encounter issues with using that :-) |
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.
see comment above.
@flokli wrote:
Wouldn't it be the same? From https://github.com/Profpatsch/yarn2nix#using-the-generated-nix-files-to-build-a-project:
|
Thanks @pacien for following up so quickly.
Yeah that sounds identical to what's done here. Though I'm not familiar with these tools, so my opinion won't be influential. |
Yeah, it’s essentially the same. If yarn2nix already works for you, vendoring might be the best solution for now. I’m not exactly sure why nomeata’s version was pulled from nixpkgs, but I could think about upstreaming my library to nixpkgs & make it beta. Next problem: I use |
@Profpatsch wrote:
If that's not possible, I think it would be very nice to have at least a common |
At least for my implementation, the library is bound to the version of the tool (because changes normally propagate though to the library). Bear in mind that |
So I'm not seeing anything raised that @pacien can make an action on, so I'm partial to this being merged. If I don't get a counterargument in a couple of days I will merge this. |
Motivation for this change
Use our own copy of
yarn2nix
as that package is being removed (#58424).github ref: closes #58593
CC: @worldofpeace
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)