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
sit: 0.3.0 -> 0.3.2 #40032
sit: 0.3.0 -> 0.3.2 #40032
Conversation
@GrahamcOfBorg build sit |
Success on x86_64-linux (full log) Attempted: sit Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: sit Partial log (click to expand)
|
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.
aarch64: see log, the same test fails for current version sit 0.3.0. Please try to fix it (or, if considered safe) disable the failing test case.
|
||
meta = with stdenv.lib; { | ||
description = "Serverless Information Tracker"; | ||
homepage = http://sit-it.org/; | ||
homepage = https://sit.sh/; | ||
license = with licenses; [ asl20 /* or */ mit ]; | ||
maintainers = with maintainers; [ dywedir ]; |
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.
Would you like to add yourself as a maintainer since you have been most active on this package recently?
I need to find an aarch64 host to fix it. As for maintainership, sure. I'll add myself. |
Thanks.
Short of fixing it, you could also selectively disable the failing test case for aarch64 to get it to build so interested people can give it a try... I'll merge the update anyway, it's no regression since the current version doesn't build on aarch64 either. |
So, I investigated this issue on an aarch64 host a bit. Here's what I found:
Bottom line: the "directory exists" error happens in release mode, but doesn't happen in debug. At least this is how it looks like to me. |
I logged this finding here rust-lang/rust#50516 |
Thanks for investigating. It would be great to have sit build on aarch64, but there's no rush. We can always add a patch in nixpkgs to disable the failing test if needed. |
I already have a patch that works around the problem, without eliminating the test. I will extract it and add to nixpkgs as well, with the intention to kill it upon next release, which will include the patch anyway. |
As we found out in NixOS#40032, sit tests won't pass on aarch64. The problem seems to be related to `create_dir` not returning an error if the directory already exists, happening specifically on aarch64+debug (not aarch64+release) This update injects a patch for tests that will also be included in subsequent versions of SIT.
Motivation for this change
New version released
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)