-
-
Notifications
You must be signed in to change notification settings - Fork 365
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
NixOS package missing version and git commit #304
Comments
@aaschmid re #301 (comment), are there any file permission restriction on the environment? That is among the likely causes. The test is basically testing a background daemon and creates a pid file in the process. I was trying to mimic an actual use as much as possible. Anyways, I am already looking at stubbing it out as it seems to be an issue here. |
I am not aware of any special permissions, though. In addition, the PR for v0.4.2 (see NixOS/nixpkgs#174013) does also have failed build, see e.g. https://github.com/NixOS/nixpkgs/pull/174013/checks?check_run_id=6545228494. |
As a workaround we could just disable the tests for now... (ugly but effective to get the new version available). |
That is fine by me. But how easy would it be to re-enable it? I would definitely be looking to get it fixed before the next release. |
I made a new change that should simply skip the test on failure. You can try the branch once again and see. Thanks. |
This issue seems to be about the upgrade to the newest version. The missing version is already fixed by NixOS/nixpkgs#173914. It's better to track these kinds of issues in Nixpkgs though. This isn't related to this app's code, just our packaging of it. |
I finally reproduced this error #301 (comment). It required me setting up Nix and running in a pure nix shell Not a big issue and tests can be skipped for v0.4.2. It will be properly addressed ahead of future releases. PS: current master should no longer fail the tests. |
You mean |
Oops yeah, I meant |
Test seems to be skipped but now on my machine the following fails:
Where is the directory try to be created? P.S.: the output for the skipped test
|
This has been fixed for 0.3.4. after upgrading to Nix 2.8.1.
@aaschmid should we close the issue and track the failing tests on nixpkgs repo? |
We can track the failing tests on nixpkgs repo also, yes. As you prefer, I am flexible ;-) |
@aaschmid as a side node, what resources do I use to keep up with Nix? I gave Nix a second try due to this issue (tried it before, gave up and reverted to Brew) and I think I finally got a hang of it. Only to check out flakes which led me to 2.0 commands, and now I am totally lost. My use case was generally covered with I keep seeing flakes everywhere and sounds like it is the direction Nix is heading, but there is not as much documentation compared to what I am used to with the tools I generally use. And much of docs assume you're already a Nix expert. Anyways, enough of the rant. Closing the issue. |
https://discourse.nixos.org/c/announcements/8 is a pretty good place. The new stuff with the new nix CLI, flakes etc. is still experimental and subject to change. If you use-cases are covered by nix-shell and hm, that's absolutely fine and you should keep using them. That's how my usage looks like too. Official and non-official docs are sparse but if you have any suggestions on the official ones, feel free to open issues on Nix and Nixpkgs. |
I totally agree with @Atemu.
Yes, I am quite familiar with the usage of nix, nix-darwin and home-manager but the learning curve was rather steep. For the creation of packages there is a bunch of additional things to learn which I try on demand - and I always learn new things ;-) |
Describe the Issue
Running
colima version
on nixpkg shows wrong version and git commit.Version
N/A
Colima Version:
What is the output of
colima version
Lima Version:
What is the output of
limactl --version
N/A
Qemu Version
What is the output of
qemu-img --version
N/A
Operating System
To Reproduce
Steps to reproduce the behavior:
colima version
Expected behavior
Colima shows version and revision
Additional context
cc @aaschmid @Atemu
The text was updated successfully, but these errors were encountered: