* [Patch, Build, libquadmath] PR 47182 Fix BUGURL in libquadmath.texi
@ 2011-01-13 23:33 Tobias Burnus
2011-01-14 7:25 ` Ralf Wildenhues
0 siblings, 1 reply; 3+ messages in thread
From: Tobias Burnus @ 2011-01-13 23:33 UTC (permalink / raw)
To: gcc patches, Ralf Wildenhues
[-- Attachment #1: Type: text/plain, Size: 257 bytes --]
The patch fixes setting the bugurl in libquadmath. The code mimics the
one in in gcc/{configure.ac,Makefile.in,doc/include/gcc-common.texi}
(the latter includes the generated gcc-vers.texi).
Build and regtested on x86-64-linux.
OK for the trunk?
Tobias
[-- Attachment #2: quad-bugurl.diff --]
[-- Type: text/x-patch, Size: 2029 bytes --]
2011-01-14 Tobias Burnus <burnus@net-b.de>
PR fortran/47182
* configure.ac: Use ACX_BUGURL.
* libquadmath.texi: Include libquadmath-vers.texi for BUGURL.
* Makefile.am: Create libquadmath-vers.texi.
* configure.in: Regenerate.
* Makefile.in: Regenerate.
Index: libquadmath/configure.ac
===================================================================
--- libquadmath/configure.ac (Revision 168765)
+++ libquadmath/configure.ac (Arbeitskopie)
@@ -49,6 +49,7 @@ ACX_CHECK_PROG_VER([MAKEINFO], [makeinfo], [--vers
[4.[4-9]*|4.[1-9][0-9]*|[5-9]*|[1-9][0-9]*])
AM_CONDITIONAL(BUILD_INFO, test $gcc_cv_prog_makeinfo_modern = "yes")
+ACX_BUGURL([http://gcc.gnu.org/bugs.html])
# Configure libtool
AM_PROG_LIBTOOL
Index: libquadmath/libquadmath.texi
===================================================================
--- libquadmath/libquadmath.texi (Revision 168765)
+++ libquadmath/libquadmath.texi (Arbeitskopie)
@@ -272,6 +272,9 @@ int main ()
@c Reporting Bugs
@c ---------------------------------------------------------------------
+@c For BUGURL
+@include libquadmath-vers.texi
+
@node Reporting Bugs
@chapter Reporting Bugs
Index: libquadmath/Makefile.am
===================================================================
--- libquadmath/Makefile.am (Revision 168765)
+++ libquadmath/Makefile.am (Arbeitskopie)
@@ -126,7 +126,7 @@ stamp-geninsrc: libquadmath.info
cp -p $(top_builddir)/libquadmath.info $(srcdir)/libquadmath.info
@touch $@
-stamp-build-info: libquadmath.texi
+stamp-build-info: libquadmath.texi $(libquadmath_TEXINFOS)
$(MAKEINFO) $(AM_MAKEINFOFLAGS) $(MAKEINFOFLAGS) -I $(srcdir) -o libquadmath.info $(srcdir)/libquadmath.texi
@touch $@
@@ -150,3 +150,7 @@ TEXINFO_TEX = ../gcc/doc/include/texinfo.tex
# Defines info, dvi, pdf and html targets
MAKEINFOFLAGS = -I $(srcdir)/../gcc/doc/include
info_TEXINFOS = libquadmath.texi
+libquadmath_TEXINFOS = libquadmath-vers.texi
+
+libquadmath-vers.texi:
+ echo "@set BUGURL $(REPORT_BUGS_TEXI)" > $@
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Patch, Build, libquadmath] PR 47182 Fix BUGURL in libquadmath.texi
2011-01-13 23:33 [Patch, Build, libquadmath] PR 47182 Fix BUGURL in libquadmath.texi Tobias Burnus
@ 2011-01-14 7:25 ` Ralf Wildenhues
2011-01-14 7:57 ` Tobias Burnus
0 siblings, 1 reply; 3+ messages in thread
From: Ralf Wildenhues @ 2011-01-14 7:25 UTC (permalink / raw)
To: Tobias Burnus; +Cc: gcc patches
* Tobias Burnus wrote on Thu, Jan 13, 2011 at 10:49:18PM CET:
> The patch fixes setting the bugurl in libquadmath. The code mimics
> the one in in
> gcc/{configure.ac,Makefile.in,doc/include/gcc-common.texi} (the
> latter includes the generated gcc-vers.texi).
>
> Build and regtested on x86-64-linux.
> OK for the trunk?
FWIW the patch looks OK to me.
Thanks,
Ralf
> 2011-01-14 Tobias Burnus <burnus@net-b.de>
>
> PR fortran/47182
> * configure.ac: Use ACX_BUGURL.
> * libquadmath.texi: Include libquadmath-vers.texi for BUGURL.
> * Makefile.am: Create libquadmath-vers.texi.
> * configure.in: Regenerate.
> * Makefile.in: Regenerate.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Patch, Build, libquadmath] PR 47182 Fix BUGURL in libquadmath.texi
2011-01-14 7:25 ` Ralf Wildenhues
@ 2011-01-14 7:57 ` Tobias Burnus
0 siblings, 0 replies; 3+ messages in thread
From: Tobias Burnus @ 2011-01-14 7:57 UTC (permalink / raw)
To: Ralf Wildenhues, gcc patches
Ralf Wildenhues wrote:
> * Tobias Burnus wrote on Thu, Jan 13, 2011 at 10:49:18PM CET:
>> Build and regtested on x86-64-linux.
>> OK for the trunk?
> FWIW the patch looks OK to me.
Well, as the interesting part is the build part - I declared the .texi
part as obvious and have committed it as Rev. 168776.
Thanks for the (partial) review!
Tobias
>> 2011-01-14 Tobias Burnus<burnus@net-b.de>
>>
>> PR fortran/47182
>> * configure.ac: Use ACX_BUGURL.
>> * libquadmath.texi: Include libquadmath-vers.texi for BUGURL.
>> * Makefile.am: Create libquadmath-vers.texi.
>> * configure.in: Regenerate.
>> * Makefile.in: Regenerate.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-01-14 7:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-13 23:33 [Patch, Build, libquadmath] PR 47182 Fix BUGURL in libquadmath.texi Tobias Burnus
2011-01-14 7:25 ` Ralf Wildenhues
2011-01-14 7:57 ` Tobias Burnus
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).