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

In tests where libtree is expected to fail the tests fail for real #65

Closed
artyom-poptsov opened this issue Jan 13, 2022 · 4 comments
Closed

Comments

@artyom-poptsov
Copy link
Contributor

Hello!

I'm trying to package libtree for GNU Guix and there's a problem with the tests.

In some tests libtree is expected to fail (and that's fine) but the failure brings down the whole build as a non-zero exit status returned to the operating system.

I tried to call libtree on the test auto-generated program called exe after the fail and it failed indeed with the code 28 as you can see:

$ libtree /tmp/guix-build-libtree-3.0.1.drv-0/source/tests/07_origin_is_relative_to_symlink_location_not_realpath/exe 
exe 
└── libg.so [runpath]
    └── libf.so not found
        ┊ Paths considered in this order:
        ┊ 1. rpath is skipped because runpath was set
        ┊ 2. LD_LIBRARY_PATH was not set
        ┊ 3. runpath:
        ┊    /tmp/guix-build-libtree-3.0.1.drv-0/source/tests/07_origin_is_relative_to_symlink_location_not_realpath/b
        ┊ 4. ld config files:
        ┊    /usr/lib/x86_64-linux-gnu/libfakeroot
        ┊    /usr/local/lib
        ┊    /usr/local/lib/x86_64-linux-gnu
        ┊    /lib/x86_64-linux-gnu
        ┊    /usr/lib/x86_64-linux-gnu
        ┊    /lib32
        ┊    /usr/lib32
        ┊ 5. Standard paths:
        ┊    /lib
        ┊    /lib64
        ┊    /usr/lib
        ┊    /usr/lib64
Error [/tmp/guix-build-libtree-3.0.1.drv-0/source/tests/07_origin_is_relative_to_symlink_location_not_realpath/exe]: Not all dependencies were found
$ echo $?
28

Please re-write the tests in such way that tests with expected failures will return zero status to the OS.

Thanks!

@artyom-poptsov artyom-poptsov changed the title In tests where libtree is expected to failt he tests fail for real In tests where libtree is expected to failt the tests fail for real Jan 13, 2022
@artyom-poptsov artyom-poptsov changed the title In tests where libtree is expected to failt the tests fail for real In tests where libtree is expected to fail the tests fail for real Jan 13, 2022
@haampie
Copy link
Owner

haampie commented Jan 14, 2022

You're right, I haven't had time to improve the tests yet.

Also I think in guix you need #61, right? Or does guix only have one system glibc?

@haampie
Copy link
Owner

haampie commented Jan 14, 2022

Fixed in 3.0.2

@haampie haampie closed this as completed Jan 14, 2022
@artyom-poptsov
Copy link
Contributor Author

Hello,

FWIW, my patch that adds libtree to GNU Guix package manager is finaly accepted:
https://git.savannah.gnu.org/cgit/guix.git/commit/?id=fcebc4aa12300544a2d62eab5ed06d1cb9eeed43

Also:
https://guix.gnu.org/en/packages/libtree-3.0.2/

Happy hacking,
avp

@haampie
Copy link
Owner

haampie commented Mar 10, 2022

Thanks a lot! When I have time I'll try to get #61 fixed so libtree is more usable on guix and nix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants