-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
vlang: Propagate build inputs, add $LDFLAGS #70466
Conversation
|
I wasn't aware there was another way to fix this. |
@Chiiruno Usually for open source software we apply patches and/or simple wrappers to make things work the "Nix way". |
@veprbl I did attempt to use a wrapper instead, unfortunately it simply would not work. |
@Chiiruno You need to document the issue better. What doesn't work? How does it not work?
Everything I've tried compiles for me as long as I use |
Because it doesn't work without that. |
To elaborate, |
Okay, it works with that command. |
In principle there is nothing wrong with the current situation. It makes a perfect sense that if you want to compile something that uses, for example, the glfw, then you need to include the If V only supports limited number of libraries, it should be possible to make them all work out of the box. Perhaps putting them to the |
Okay, I had a fundamental misunderstanding when testing apparently. |
@GrahamcOfBorg build vlang |
Motivation for this change
Some examples weren't working.
Also for
-compress
option.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)