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: init at 0.1.16 #63738
vlang: init at 0.1.16 #63738
Conversation
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.
I tried it and could successfully run the tetris example in a nix-shell with freetype curl glfw
. Because of vlang/v#542 I also had to copy the font to my working directory.
If it takes them a long time to figure out vlang/v#457, maybe we could solve it with a wrapper, at least for the standard library. |
This changed quick. |
Waiting until the releases page and v.c is updated, don't want to flip flop between revision and release revision if possible. |
A bit meta-question: isn't "v" a bit too short/generic identifier? In particular, it seems that a couple distros use it already for a different project: https://github.com/rupa/v |
Since V is a language, changing it to vlang should be fine IMO. |
👍 That choice is also supported by them using https://github.com/vlang. |
Get this problem with the bootstrapped V compiler, tested locally and it doesn't happen, will make issue both here and on the V repo, not sure which is at fault.
|
Not a nix issue, 0.1.4 should be the version that finally gets merged. |
Okay, things should be stable enough in this aspect for merge now. |
Oh whoops, forgot to change it to vlang instead of v, one sec. |
Double checked; Ready. |
Like this? |
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.
We usually indent our files with 2 spaces everywhere. This uses 3.
Not sure how my tabs got messed up, but it's fixed now. |
@srhb The "wait for upstream" tag is no longer needed, could you remove it please? |
Executable says 0.1.9 when executed, but the release is 0.1.10, maybe a release issue. |
@Chiiruno , Sounds like another wait-for-upstream issue then? |
Upstream is really fast and they're already at v0.1.12. |
Looks fine to me now. |
@GrahamcOfBorg eval |
@GrahamcOfBorg build vlang |
@Chiiruno
|
To fix that, either we would need to trick the V compilation process into thinking it was compiled at the nix store directory, or have a way to set the V repo directory to the nix store directory. |
This may be fixed on the next update, with vlib being moved into /usr/lib. |
Should be ready. |
@GrahamcOfBorg build vlang |
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.
Seems to work at least on darwin
Motivation for this change
V is an interesting new language, might as well have a package for it.
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)