public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [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).