GNU bug report logs - #39974
Guile fails to build without makeinfo

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 39974 in the body.
You can then email your comments to 39974 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Fabrice Fontaine <fontaine.fabrice <at> gmail.com>
To: bug-guile <at> gnu.org
Subject: Guile fails to build without makeinfo
Date: Sat, 7 Mar 2020 10:53:33 +0100

[Message part 1 (text/plain, inline)]

guile fails to build without makeinfo:

make[4]: Entering directory
'/nvmedata/autobuild/instance-7/output-1/build/host-guile-2.0.14/doc/ref'
  MAKEINFO guile.info
/nvmedata/autobuild/instance-7/output-1/build/host-guile-2.0.14/build-aux/missing:
line 81: makeinfo: command not found
WARNING: 'makeinfo' is missing on your system.
         You should only need it if you modified a '.texi' file, or
         any other file indirectly affecting the aspect of the manual.
         You might want to install the Texinfo package:
         <http://www.gnu.org/software/texinfo/>
         The spurious makeinfo call might also be the consequence of
         using a buggy 'make' (AIX, DU, IRIX), in which case you might
         want to install GNU make:
         <http://www.gnu.org/software/make/>

To fix this build failure, the attached patch don't build the doc
subdir if makeinfo is not available.

Fixes:
 - http://autobuild.buildroot.org/results/9605aac6f760bfff190d0ab95fa50f65486ffe90

[0001-Makefile.am-fix-build-without-makeinfo.patch (text/x-patch, attachment)]

Message #8 received at 39974 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Fabrice Fontaine <fontaine.fabrice <at> gmail.com>
Cc: 39974 <at> debbugs.gnu.org
Subject: Re: bug#39974: Guile fails to build without makeinfo
Date: Sat, 07 Mar 2020 17:04:51 +0100

Hello,

Fabrice Fontaine <fontaine.fabrice <at> gmail.com> skribis:

> guile fails to build without makeinfo:
>
> make[4]: Entering directory
> '/nvmedata/autobuild/instance-7/output-1/build/host-guile-2.0.14/doc/ref'
>   MAKEINFO guile.info
> /nvmedata/autobuild/instance-7/output-1/build/host-guile-2.0.14/build-aux/missing:
> line 81: makeinfo: command not found

AFAIK, ‘makeinfo’ is needed if and only if one modifies the .texi files,
or perhaps the ‘configure’ file.

Could it be what happened here?

BTW, 2.0 is an old series that is no longer maintained (the current
stable series is 3.0, and the previous one is 2.2).

Thanks,
Ludo’.

Message #11 received at 39974 <at> debbugs.gnu.org (full text, mbox):

From: Fabrice Fontaine <fontaine.fabrice <at> gmail.com>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 39974 <at> debbugs.gnu.org
Subject: Re: bug#39974: Guile fails to build without makeinfo
Date: Sat, 7 Mar 2020 17:35:58 +0100

Hello,

Le sam. 7 mars 2020 à 17:04, Ludovic Courtès <ludo <at> gnu.org> a écrit :
>
> Hello,
>
> Fabrice Fontaine <fontaine.fabrice <at> gmail.com> skribis:
>
> > guile fails to build without makeinfo:
> >
> > make[4]: Entering directory
> > '/nvmedata/autobuild/instance-7/output-1/build/host-guile-2.0.14/doc/ref'
> >   MAKEINFO guile.info
> > /nvmedata/autobuild/instance-7/output-1/build/host-guile-2.0.14/build-aux/missing:
> > line 81: makeinfo: command not found
>
> AFAIK, ‘makeinfo’ is needed if and only if one modifies the .texi files,
> or perhaps the ‘configure’ file.
>
> Could it be what happened here?
Yes, this build failure is raised by one of our buidlroot autobuilders.
We're running autoreconf as we have a patch that updates configure.ac:
https://git.buildroot.net/buildroot/tree/package/guile/0002-calculate-csqrt_manually.patch.
If your interested in applying this patch upstream, I can send it to
the mailing list.
>
> BTW, 2.0 is an old series that is no longer maintained (the current
> stable series is 3.0, and the previous one is 2.2).
Our current version of guile is indeed pretty old, one of our users
sent a patch to bump it to 3.0:
https://patchwork.ozlabs.org/patch/1246994.
However, I assume that this build failure won't be fixed in 3.0 as
we'll still have to update configure.ac.

So, in the end, I saw that HAVE_MAKEINFO was added quite a while ago (in 2001):
http://git.savannah.gnu.org/cgit/guile.git/commit/?id=a9d78896d1737c429ca8640d5e2fe0814a2ea20c
However, this variable was never used so it seemed right to me to use
it to fix this build failure
>
> Thanks,
> Ludo’.
Best Regards,

Fabrice

Message #14 received at 39974 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Fabrice Fontaine <fontaine.fabrice <at> gmail.com>
Cc: 39974 <at> debbugs.gnu.org
Subject: Re: bug#39974: Guile fails to build without makeinfo
Date: Sat, 07 Mar 2020 21:40:34 +0100

Hi,

Fabrice Fontaine <fontaine.fabrice <at> gmail.com> skribis:

>> Fabrice Fontaine <fontaine.fabrice <at> gmail.com> skribis:
>>
>> > guile fails to build without makeinfo:
>> >
>> > make[4]: Entering directory
>> > '/nvmedata/autobuild/instance-7/output-1/build/host-guile-2.0.14/doc/ref'
>> >   MAKEINFO guile.info
>> > /nvmedata/autobuild/instance-7/output-1/build/host-guile-2.0.14/build-aux/missing:
>> > line 81: makeinfo: command not found
>>
>> AFAIK, ‘makeinfo’ is needed if and only if one modifies the .texi files,
>> or perhaps the ‘configure’ file.
>>
>> Could it be what happened here?
> Yes, this build failure is raised by one of our buidlroot autobuilders.
> We're running autoreconf as we have a patch that updates configure.ac:

OK.  From Guile’s perspective, it’s not a bug that ‘makeinfo’ is needed
in this case.

> https://git.buildroot.net/buildroot/tree/package/guile/0002-calculate-csqrt_manually.patch.
> If your interested in applying this patch upstream, I can send it to
> the mailing list.

I don’t think we would apply the patch because it’s uClibc-specific,
AIUI.

However, instead of patching ‘configure.ac’, you can run:

  ./configure guile_cv_use_csqrt=no

to achieve the same effect as the patch.

Let me know if it works for you!

Thanks,
Ludo’.

Message #17 received at 39974 <at> debbugs.gnu.org (full text, mbox):

From: Fabrice Fontaine <fontaine.fabrice <at> gmail.com>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 39974 <at> debbugs.gnu.org
Subject: Re: bug#39974: Guile fails to build without makeinfo
Date: Sat, 7 Mar 2020 23:07:18 +0100

Hi,

Le sam. 7 mars 2020 à 21:40, Ludovic Courtès <ludo <at> gnu.org> a écrit :
>
> Hi,
>
> Fabrice Fontaine <fontaine.fabrice <at> gmail.com> skribis:
>
> >> Fabrice Fontaine <fontaine.fabrice <at> gmail.com> skribis:
> >>
> >> > guile fails to build without makeinfo:
> >> >
> >> > make[4]: Entering directory
> >> > '/nvmedata/autobuild/instance-7/output-1/build/host-guile-2.0.14/doc/ref'
> >> >   MAKEINFO guile.info
> >> > /nvmedata/autobuild/instance-7/output-1/build/host-guile-2.0.14/build-aux/missing:
> >> > line 81: makeinfo: command not found
> >>
> >> AFAIK, ‘makeinfo’ is needed if and only if one modifies the .texi files,
> >> or perhaps the ‘configure’ file.
> >>
> >> Could it be what happened here?
> > Yes, this build failure is raised by one of our buidlroot autobuilders.
> > We're running autoreconf as we have a patch that updates configure.ac:
>
> OK.  From Guile’s perspective, it’s not a bug that ‘makeinfo’ is needed
> in this case.
OK, then instead of this patch, do you think that adding a
--disable-doc option could be acceptable?
>
> > https://git.buildroot.net/buildroot/tree/package/guile/0002-calculate-csqrt_manually.patch.
> > If your interested in applying this patch upstream, I can send it to
> > the mailing list.
>
> I don’t think we would apply the patch because it’s uClibc-specific,
> AIUI.
>
> However, instead of patching ‘configure.ac’, you can run:
>
>   ./configure guile_cv_use_csqrt=no
>
> to achieve the same effect as the patch.
>
> Let me know if it works for you!
OK, I'll update buildroot with your proposal as we try to avoid
carrying out non-upstreamable patch.
>
> Thanks,
> Ludo’.
Best Regards,

Fabrice

Message #20 received at 39974 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Fabrice Fontaine <fontaine.fabrice <at> gmail.com>
Cc: 39974 <at> debbugs.gnu.org
Subject: Re: bug#39974: Guile fails to build without makeinfo
Date: Sun, 08 Mar 2020 12:09:59 +0100

Hello,

Fabrice Fontaine <fontaine.fabrice <at> gmail.com> skribis:

> Le sam. 7 mars 2020 à 21:40, Ludovic Courtès <ludo <at> gnu.org> a écrit :

[...]

>> > Yes, this build failure is raised by one of our buidlroot autobuilders.
>> > We're running autoreconf as we have a patch that updates configure.ac:
>>
>> OK.  From Guile’s perspective, it’s not a bug that ‘makeinfo’ is needed
>> in this case.
> OK, then instead of this patch, do you think that adding a
> --disable-doc option could be acceptable?

No because again, when building from a tarball, ‘makeinfo’ is not needed
(the Info files are included in the tarball).  I hope that makes sense!

Thanks,
Ludo’.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.