Skip to content
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

xidlehook: init at 0.4.6 #39968

Merged
merged 1 commit into from May 7, 2018
Merged

xidlehook: init at 0.4.6 #39968

merged 1 commit into from May 7, 2018

Conversation

jD91mZM2
Copy link
Member

@jD91mZM2 jD91mZM2 commented May 4, 2018

No description provided.

repo = "xidlehook";
rev = version;

sha256 = "0mj02w959h0sr8b2r3l54dqn4j3y88xdr434y3jyfqd371kg0i39";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am getting another hash here:

unpacking source archive /build/0.4.5.tar.gz
fixed-output derivation produced path '/nix/store/f78zppjisrw0chk8dmx3vifqrwzkshql-source' with sha256 hash '1jdy85sh7w8drcahx33swqwnv2ik7rpxl0hc07waxw3iwwk63m66' instead of the expected hash '0mj02w959h0sr8b2r3l54dqn4j3y88xdr434y3jyfqd371kg0i39'

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops, I forgot to update the hash since 0.0.4. My bad!

sha256 = "0mj02w959h0sr8b2r3l54dqn4j3y88xdr434y3jyfqd371kg0i39";
};

cargoSha256 = "0grx7qcyah6xclkl944a4cg4fvmma7k6qnsakwzanwqcv8snkbzg";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also for this one:

directory = "/build/source/vendor"
fixed-output derivation produced path '/nix/store/nsn04g4mflzdjsddz9hc8578jfbspj9y-xidlehook-0.4.5-vendor' with sha256 hash '0a1bl6fnfw6xy71q3b5zij52p9skylj1ivqj8my44bfsid2qfn7d' instead of the expected hash '0grx7qcyah6xclkl944a4cg4fvmma7k6qnsakwzanwqcv8snkbzg'

@jD91mZM2
Copy link
Member Author

jD91mZM2 commented May 4, 2018

I'm sorry for the delay, I can't seem to find how to clear build cache.

@WilliButz
Copy link
Member

@jD91mZM2 I'm not sure, is the included .cargo folder needed?

@jD91mZM2
Copy link
Member Author

jD91mZM2 commented May 4, 2018

Probably not as I didn't put it there (generated, presumably)
EDIT: deleted

cargoSha256 = "0a1bl6fnfw6xy71q3b5zij52p9skylj1ivqj8my44bfsid2qfn7d";

buildInputs = [ x11 xorg.libXScrnSaver libpulseaudio ];
propagatedBuildInputs = [ pkgconfig makeWrapper ];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you move these to nativeBuildInputs? :)

Copy link
Member Author

@jD91mZM2 jD91mZM2 May 4, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure! What's the difference?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nativeBuildInputs are the ones used on the build host and thus have to be compiled for it's architecture. So when a x86 host cross-compiles something for aarch64, the build tools have to be built for x86 even though the target is aarch64.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here is some more information in the manual :)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks!

@WilliButz
Copy link
Member

@GrahamcOfBorg build xidlehook

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: xidlehook

Partial log (click to expand)

cp: -r not specified; omitting directory 'target/release/.fingerprint'
cp: -r not specified; omitting directory 'target/release/examples'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/wsjklrqkyn3qidcb9gbcvs9y8mnw9y70-xidlehook-0.4.5
shrinking /nix/store/wsjklrqkyn3qidcb9gbcvs9y8mnw9y70-xidlehook-0.4.5/bin/.xidlehook-wrapped
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/wsjklrqkyn3qidcb9gbcvs9y8mnw9y70-xidlehook-0.4.5/bin
patching script interpreter paths in /nix/store/wsjklrqkyn3qidcb9gbcvs9y8mnw9y70-xidlehook-0.4.5
checking for references to /build in /nix/store/wsjklrqkyn3qidcb9gbcvs9y8mnw9y70-xidlehook-0.4.5...
/nix/store/wsjklrqkyn3qidcb9gbcvs9y8mnw9y70-xidlehook-0.4.5

@GrahamcOfBorg
Copy link

Failure on aarch64-linux (full log)

Attempted: xidlehook

Partial log (click to expand)

    = note: expected type `*const u8`
               found type `*mut i8`

error: aborting due to 2 previous errors

error: Could not compile `xidlehook`.

To learn more, run the command again with --verbose.
builder for '/nix/store/dmcbnd409y0j0i4c9dh1dgl5qdr95ljd-xidlehook-0.4.5.drv' failed with exit code 101
�[31;1merror:�[0m build of '/nix/store/dmcbnd409y0j0i4c9dh1dgl5qdr95ljd-xidlehook-0.4.5.drv' failed

@GrahamcOfBorg
Copy link

Success on x86_64-darwin (full log)

Attempted: xidlehook

Partial log (click to expand)

cp: -r not specified; omitting directory 'target/release/build'
cp: -r not specified; omitting directory 'target/release/deps'
cp: -r not specified; omitting directory 'target/release/examples'
cp: -r not specified; omitting directory 'target/release/incremental'
cp: -r not specified; omitting directory 'target/release/native'
post-installation fixup
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/5354zlm82basnrfa2y0qa98zsr1fcp1q-xidlehook-0.4.5/bin
patching script interpreter paths in /nix/store/5354zlm82basnrfa2y0qa98zsr1fcp1q-xidlehook-0.4.5
/nix/store/5354zlm82basnrfa2y0qa98zsr1fcp1q-xidlehook-0.4.5

@jD91mZM2
Copy link
Member Author

jD91mZM2 commented May 4, 2018

That's odd. I think that is because I'm using u8 and not c_char. On it!

@jD91mZM2 jD91mZM2 force-pushed the xidlehook branch 2 times, most recently from ffd9fb1 to f37e2b0 Compare May 4, 2018 14:18
@jD91mZM2
Copy link
Member Author

jD91mZM2 commented May 4, 2018

@GrahamcOfBorg build xidlehook

@WilliButz
Copy link
Member

For that to work, you have to be in the list of trusted users :)
@GrahamcOfBorg build xidlehook

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: xidlehook

Partial log (click to expand)

cp: -r not specified; omitting directory 'target/release/native'
cp: -r not specified; omitting directory 'target/release/deps'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/cw3cgd88p6p0yy2ifxww20nf4m7azn8p-xidlehook-0.4.6
shrinking /nix/store/cw3cgd88p6p0yy2ifxww20nf4m7azn8p-xidlehook-0.4.6/bin/.xidlehook-wrapped
strip is /nix/store/gp7fylxwn18b7pl2c18ks89hsiaxyfvf-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/cw3cgd88p6p0yy2ifxww20nf4m7azn8p-xidlehook-0.4.6/bin
patching script interpreter paths in /nix/store/cw3cgd88p6p0yy2ifxww20nf4m7azn8p-xidlehook-0.4.6
checking for references to /build in /nix/store/cw3cgd88p6p0yy2ifxww20nf4m7azn8p-xidlehook-0.4.6...
/nix/store/cw3cgd88p6p0yy2ifxww20nf4m7azn8p-xidlehook-0.4.6

@GrahamcOfBorg
Copy link

Success on x86_64-darwin (full log)

Attempted: xidlehook

Partial log (click to expand)

cp: -r not specified; omitting directory 'target/release/build'
cp: -r not specified; omitting directory 'target/release/deps'
cp: -r not specified; omitting directory 'target/release/examples'
cp: -r not specified; omitting directory 'target/release/incremental'
cp: -r not specified; omitting directory 'target/release/native'
post-installation fixup
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/3ncdw11fijm30kwy8lqylj18bdw05q37-xidlehook-0.4.6/bin
patching script interpreter paths in /nix/store/3ncdw11fijm30kwy8lqylj18bdw05q37-xidlehook-0.4.6
/nix/store/3ncdw11fijm30kwy8lqylj18bdw05q37-xidlehook-0.4.6

@jD91mZM2
Copy link
Member Author

jD91mZM2 commented May 4, 2018

Aha 😛
It was worth a shot ¯\_(ツ)_/¯

@jD91mZM2 jD91mZM2 changed the title xidlehook: init at 0.4.5 xidlehook: init at 0.4.6 May 4, 2018
@WilliButz
Copy link
Member

thank's for your work 👍

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: xidlehook

Partial log (click to expand)

cp: -r not specified; omitting directory 'target/release/.fingerprint'
cp: -r not specified; omitting directory 'target/release/examples'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/gvz1rq7jw93pjxdr14dzxiyb8ck3sm73-xidlehook-0.4.6
shrinking /nix/store/gvz1rq7jw93pjxdr14dzxiyb8ck3sm73-xidlehook-0.4.6/bin/.xidlehook-wrapped
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/gvz1rq7jw93pjxdr14dzxiyb8ck3sm73-xidlehook-0.4.6/bin
patching script interpreter paths in /nix/store/gvz1rq7jw93pjxdr14dzxiyb8ck3sm73-xidlehook-0.4.6
checking for references to /build in /nix/store/gvz1rq7jw93pjxdr14dzxiyb8ck3sm73-xidlehook-0.4.6...
/nix/store/gvz1rq7jw93pjxdr14dzxiyb8ck3sm73-xidlehook-0.4.6

@jD91mZM2
Copy link
Member Author

jD91mZM2 commented May 4, 2018

I should be the one thanking you for your patience! :)

description = "xautolock rewrite in Rust, with a few extra features";
homepage = https://github.com/jD91mZM2/xidlehook;
license = licenses.mit;
maintainer = maintainers.jD91mZM2;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you could fix the borg eval check as follows:

maintainers = [ maintainers.jD91mZM2 ];
platforms = platforms.unix;

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, sorry, didn't notice that it failed. My bad.

@jD91mZM2 jD91mZM2 force-pushed the xidlehook branch 2 times, most recently from bce462a to 8bc6278 Compare May 4, 2018 18:37
@jD91mZM2
Copy link
Member Author

jD91mZM2 commented May 4, 2018

Once again didn't notice the check failed. Sorry.

@matthewbauer matthewbauer merged commit e71c441 into NixOS:master May 7, 2018
@jD91mZM2 jD91mZM2 deleted the xidlehook branch May 7, 2018 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants