[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#612502: texinfo: FTBFS due to running configure+build twice, fix attached



Hilmar Preu�e dixit:

>Hm. I tried to build on Debian unstable, one time w/, one time w/o the
>patch. The build logs did not show a difference. No, I didn't found
>an evidence in the log, that something has been done twice.

Interesting. See below…

>I used i386, is the problem limited to m86k and if yes, could you
>provide a build log?

I do not think the problem is limited to m68k, but the FTBFS happened
there (though I believe it MAY happen on any architecture, and that it
doesn’t always happen on m68k). Hmpf of course I deleted the logs after
I made the patch and had it working ☹

But let’s take some release architecture’s build logs:
https://buildd.debian.org/fetch.cgi?pkg=texinfo&arch=powerpc&ver=4.13a.dfsg.1-6&stamp=1287048302&file=log&as=raw

I’m downloading this to make sure we have the correct line numbers to
compare (MD5 = 72786a7e2414bba2f30794cc0f314050).

Please look at lines 1410 ff.

There you see 「touch build-stamp」 (end of the build target), followed
by 「# some man scripts」… which is a repeat of lines 275 ff. which
follow after stamp-patched. This goes up to line 1829 (note that this
repeat, IF it works at all, takes quite a while, which is why this
patch is in general a good idea). From line 1830 onwards, we continue
with the binary-arch target.

Now, lines 1411 to 1829 (and onwards) run as root or under fakeroot,
and it just happened that some configure result differed, leading to
an FTBFS I’ve seen before (in BSD) because, IIRC, malloc was wrongly
not detected correctly and defined to rpl_malloc in config.h, but not
built in gnulib (since it worked in the first instance).

As you can see in the following build log:
http://buildd.debian-ports.org/fetch.php?&pkg=texinfo&ver=4.13a.dfsg.1-5&arch=m68k&stamp=1271634700&file=log&as=raw
This doesn’t _always_ lead to an FTBFS even on m68k. But it definiti-
vely _is_ a bad idea and a broken debian/rules file, so please just
apply the patch for your next upload.

Thanks in advance,
//mirabilos
-- 
22:20⎜<asarch> The crazy that persists in his craziness becomes a master
22:21⎜<asarch> And the distance between the craziness and geniality is
only measured by the success 18:35⎜<asarch> "Psychotics are consistently
inconsistent. The essence of sanity is to be inconsistently inconsistent



Reply to: