Summary: | =sys-apps/findutils-4.6.0 fails test-mbrtowc[12].sh if certain locales installed | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Jonathan Lovelace <kingjon3377> |
Component: | [OLD] Core system | Assignee: | Gentoo's Team for Core System packages <base-system> |
Status: | RESOLVED FIXED | ||
Severity: | normal | Keywords: | TESTFAILURE |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://git.savannah.gnu.org/cgit/gnulib.git/commit/?id=1f63650823cebf52044df840c81062ccb52163a2 | ||
See Also: | http://bugs.debian.org/812762 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
test suite log showing details of failures
emerge log with non-executable configure script |
Description
Jonathan Lovelace
2016-03-08 21:30:45 UTC
Created attachment 427782 [details]
test suite log showing details of failures
thanks, should be fixed with: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0ca5d09b5a3688ba55daad5cd210569dacf7e72d Actually, the patch in the tree isn't the patch from upstream, and instead is a no-op. The file that needs to be changed in the source tree is gl/m4/mbrtowc.m4; the patch in the Portage tree now only changes the configure script, though the patch header is from the correct patch. (In reply to Jonathan Lovelace from comment #3) m4 files are the inputs to generate the configure file. they are not read at all when running configure. have you actually verified things still fail ? (Sorry for the delay.) Yes, the tests still fail, with apparently-identical results, which was why I commented. As I mentioned, the patch in the tree doesn't actually change the configure file at all; if I run md5sum on the file between the unpack and prepare phases and after the prepare phase, it gives the same hash. If I adjust the patch to match the relevant lines in the upstream patch, the tests pass. Besides being a no-op, this patch breaks stable findutils-4.6.0, due to removing executable permission from configure script (log attached). This one would help here: diff --git a/m4/mbrtowc.m4 b/m4/mbrtowc.m4 -index deb9f06..be2e9d6 100644 +index deb9f06..be2e9d6 100755 --- a/configure +++ configure Created attachment 428722 [details]
emerge log with non-executable configure script
(In reply to Jonathan Lovelace from comment #5) ok, the issue isn't configure-vs-m4, it's that when i was hand munging the patch, i replaced the + line with the same - line as before. fixed here: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=93915c28a70bb0189e5e072a3bd119917b5aa0df (In reply to Michael Haubenwallner from comment #6) fixed that too in the aforementioned change |