| Summary: | sys-apps/hal-0.5.11-r7 fails 'make test', patch available | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Hans de Graaff <graaff> |
| Component: | New packages | Assignee: | Tony Vroon (RETIRED) <chainsaw> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | brant, coldwind, esqualante, freedesktop-bugs, kanelxake, rhill, thecoop, zlin |
| Priority: | High | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
|
Description
Hans de Graaff
2007-04-30 08:14:01 UTC
Could this be the same problem as bug 176380? (In reply to comment #1) > Could this be the same problem as bug 176380? > Not even remotely close. Thanks for the report Hans. Thanks. Fixed. It looks like this got broken again in hal 0.5.10: make[6]: Entering directory `/var/tmp/portage/sys-apps/hal-0.5.10/work/hal-0.5.10/fdi' make[7]: Entering directory `/var/tmp/portage/sys-apps/hal-0.5.10/work/hal-0.5.10/fdi' make[7]: Nothing to be done for `install-exec-am'. test -z "/usr/share/hal/fdi" || /bin/mkdir -p "/var/tmp/portage/sys-apps/hal-0.5.10/work/hal-0.5.10/hald/.local-fdi-test/usr/share/hal/fdi" /usr/bin/install -c -m 644 'fdi.dtd' '/var/tmp/portage/sys-apps/hal-0.5.10/work/hal-0.5.10/hald/.local-fdi-test/usr/share/hal/fdi/fdi.dtd' make[7]: Leaving directory `/var/tmp/portage/sys-apps/hal-0.5.10/work/hal-0.5.10/fdi' make[6]: Leaving directory `/var/tmp/portage/sys-apps/hal-0.5.10/work/hal-0.5.10/fdi' make[5]: Leaving directory `/var/tmp/portage/sys-apps/hal-0.5.10/work/hal-0.5.10/fdi' 06:44:28.519 [E] create_cache.c:540: Cannot scan '.local-fdi-test/share/hal/fdi/preprobe': No such file or directory 06:44:28.519 [E] create_cache.c:701: Error generating fdi cache FAIL: hald-cache-test.sh =============================== 1 of 1 tests failed Please report to david@fubar.dk =============================== make[4]: *** [check-TESTS] Error 1 make[4]: Leaving directory `/var/tmp/portage/sys-apps/hal-0.5.10/work/hal-0.5.10/hald' make[3]: *** [check-am] Error 2 We need a hand rolled patch for every revision since the make test paths are all hardcoded in upstream. compnerd, I'd just restrict test and be done with it. Unless you want to constantly re-roll these patches. It also probably would have been better to open a new bug. Since the original one was against 0.5.9 Just for the record that this isn't fixed in 0.5.11. It won't be. Like I've said, upstream uses hardcoded paths for this stuff. The whole test setup needs to be rewritten. No one upstream is working on it and there are no plans to work on it since I've pretty much quit HAL. Rest assured though, David Z is writing a completely new HAL replacement called DeviceKit which doesn't share any code. It might be fixed then. If it is clear that the tests are broken and won't be fixed then I think a RESTRICT="test" would be really useful. Still not fixed in 0.5.11-r1. The tests should only be broken in the case when you change where /var/tmp/portage resides, otherwise they should be fine. Unless of course someone dropped a patch I wrote a while back.. The hald_cache_test_path.patch is not included in the 0.5.11 patchset. Manually adding it from ${FILESDIR}/0.5.9/98_hald_cache_test_path.patch works without modification and allows the tests to complete.
ping. could I please get you guys to add this patch or restrict tests? +*hal-0.5.11-r8 (04 Feb 2009) + + 04 Feb 2009; <chainsaw@gentoo.org> +hal-0.5.11-r8.ebuild: + Revive a patch by Cardoe to fix test failures, closes bug #176535. Enable + support for MacBook/MacBook Pro if USE="laptop", closes bug #200643. Patch + from Ambroz Bizjak <ambro@b4ever.net> removes ill-conceived UID checks so + NTFS volumes can be automounted, closes bug #205901. Upstream commits + scavenged by Ricardo Salveti <ricardo.salveti@openbossa.org> allow + mac80211-based wireless devices to be detected properly, closes bug + #246026. Reverted partition unhiding from bug #247025, it was already + controversial but has been proven to automount recovery partitions. |