* [Bug libquadmath/55821] Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported
2012-12-28 2:19 [Bug libquadmath/55821] New: Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported gerald at pfeifer dot com
@ 2012-12-28 10:21 ` burnus at gcc dot gnu.org
2012-12-28 10:59 ` schwab@linux-m68k.org
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: burnus at gcc dot gnu.org @ 2012-12-28 10:21 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55821
Tobias Burnus <burnus at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |burnus at gcc dot gnu.org
--- Comment #1 from Tobias Burnus <burnus at gcc dot gnu.org> 2012-12-28 10:21:13 UTC ---
Let me try to understand:
* The release tar balls contain the pre-build .info files of all front ends and
libraries.
* Building the source and running "make install" installs the compilers
including the .info of all build FEs/libraries but also unconditionally
libquadmath's .info file (if it is present), even on systems where libquadmath
itself has not been build.
Currently, libquadmath is always configured (unless --disable-libquadmath has
been used) and the configure script checks whether __float128 is available.
Only if it is, the library is build. It uses BUILD_LIBQUADMATH for that
purpose.
It seems as if libquadmath/Makefile.am has to be modified. However, that might
be not as simple, if one believes the comment:
...
endif BUILD_LIBQUADMATH
# Unconditionally override this target, so that automake's definition
# does not wrongly interfere.
libquadmath.info: $(STAMP_BUILD_INFO)
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug libquadmath/55821] Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported
2012-12-28 2:19 [Bug libquadmath/55821] New: Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported gerald at pfeifer dot com
2012-12-28 10:21 ` [Bug libquadmath/55821] " burnus at gcc dot gnu.org
@ 2012-12-28 10:59 ` schwab@linux-m68k.org
2014-10-08 16:38 ` fxcoudert at gcc dot gnu.org
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: schwab@linux-m68k.org @ 2012-12-28 10:59 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55821
--- Comment #2 from Andreas Schwab <schwab@linux-m68k.org> 2012-12-28 10:58:39 UTC ---
That override is only needed so that libquadmath.info is built in the build
directory instead of the source directory. If you want to suppress installing
the info file don't set info_TEXINFOS.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug libquadmath/55821] Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported
2012-12-28 2:19 [Bug libquadmath/55821] New: Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported gerald at pfeifer dot com
2012-12-28 10:21 ` [Bug libquadmath/55821] " burnus at gcc dot gnu.org
2012-12-28 10:59 ` schwab@linux-m68k.org
@ 2014-10-08 16:38 ` fxcoudert at gcc dot gnu.org
2014-10-09 7:28 ` fxcoudert at gcc dot gnu.org
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: fxcoudert at gcc dot gnu.org @ 2014-10-08 16:38 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55821
Francois-Xavier Coudert <fxcoudert at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
Last reconfirmed| |2014-10-08
CC| |fxcoudert at gcc dot gnu.org
Ever confirmed|0 |1
--- Comment #3 from Francois-Xavier Coudert <fxcoudert at gcc dot gnu.org> ---
Would this work? Gerald, can you try it?
Index: Makefile.am
===================================================================
--- Makefile.am (revision 215645)
+++ Makefile.am (working copy)
@@ -158,6 +158,8 @@ libquadmath.info: $(STAMP_BUILD_INFO)
# the relative path from the current `Makefile.am' to `texinfo.tex'.
TEXINFO_TEX = ../gcc/doc/include/texinfo.tex
+if BUILD_LIBQUADMATH
+
# Defines info, dvi, pdf and html targets
MAKEINFOFLAGS = -I $(srcdir)/../gcc/doc/include
info_TEXINFOS = libquadmath.texi
@@ -165,3 +167,5 @@ libquadmath_TEXINFOS = libquadmath-vers.
libquadmath-vers.texi:
echo "@set BUGURL $(REPORT_BUGS_TEXI)" > $@
+
+endif BUILD_LIBQUADMATH
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug libquadmath/55821] Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported
2012-12-28 2:19 [Bug libquadmath/55821] New: Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported gerald at pfeifer dot com
` (2 preceding siblings ...)
2014-10-08 16:38 ` fxcoudert at gcc dot gnu.org
@ 2014-10-09 7:28 ` fxcoudert at gcc dot gnu.org
2014-10-09 7:41 ` fxcoudert at gcc dot gnu.org
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: fxcoudert at gcc dot gnu.org @ 2014-10-09 7:28 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55821
Francois-Xavier Coudert <fxcoudert at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |patch
URL| |https://gcc.gnu.org/ml/gcc-
| |patches/2014-10/msg00736.ht
| |ml
--- Comment #4 from Francois-Xavier Coudert <fxcoudert at gcc dot gnu.org> ---
Updated patch proposed:
https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00736.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug libquadmath/55821] Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported
2012-12-28 2:19 [Bug libquadmath/55821] New: Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported gerald at pfeifer dot com
` (3 preceding siblings ...)
2014-10-09 7:28 ` fxcoudert at gcc dot gnu.org
@ 2014-10-09 7:41 ` fxcoudert at gcc dot gnu.org
2014-10-09 8:06 ` fxcoudert at gcc dot gnu.org
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: fxcoudert at gcc dot gnu.org @ 2014-10-09 7:41 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55821
--- Comment #5 from Francois-Xavier Coudert <fxcoudert at gcc dot gnu.org> ---
Author: fxcoudert
Date: Thu Oct 9 07:40:39 2014
New Revision: 216027
URL: https://gcc.gnu.org/viewcvs?rev=216027&root=gcc&view=rev
Log:
PR libquadmath/55821
* Makefile.am: Conditionally define info_TEXINFOS and
libquadmath_TEXINFOS.
* Makefile.in: Regenerate.
Modified:
trunk/libquadmath/ChangeLog
trunk/libquadmath/Makefile.am
trunk/libquadmath/Makefile.in
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug libquadmath/55821] Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported
2012-12-28 2:19 [Bug libquadmath/55821] New: Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported gerald at pfeifer dot com
` (4 preceding siblings ...)
2014-10-09 7:41 ` fxcoudert at gcc dot gnu.org
@ 2014-10-09 8:06 ` fxcoudert at gcc dot gnu.org
2014-10-20 2:28 ` sandra at codesourcery dot com
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: fxcoudert at gcc dot gnu.org @ 2014-10-09 8:06 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55821
Francois-Xavier Coudert <fxcoudert at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
Target Milestone|--- |5.0
--- Comment #6 from Francois-Xavier Coudert <fxcoudert at gcc dot gnu.org> ---
Fixed on trunk
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug libquadmath/55821] Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported
2012-12-28 2:19 [Bug libquadmath/55821] New: Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported gerald at pfeifer dot com
` (5 preceding siblings ...)
2014-10-09 8:06 ` fxcoudert at gcc dot gnu.org
@ 2014-10-20 2:28 ` sandra at codesourcery dot com
2014-10-20 16:57 ` sandra at codesourcery dot com
2014-10-21 9:02 ` fxcoudert at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: sandra at codesourcery dot com @ 2014-10-20 2:28 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55821
Sandra Loosemore <sandra at codesourcery dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |sandra at codesourcery dot com
--- Comment #7 from Sandra Loosemore <sandra at codesourcery dot com> ---
Trying to build an arm-none-linux-gnueabi cross from mainline head, I'm getting
this error now:
/scratch/sandra/arm-fsf/src/gcc-mainline/libquadmath/libquadmath.texi:369:
@include `libquadmath-vers.texi': No such file or directory.
/scratch/sandra/arm-fsf/src/gcc-mainline/libquadmath/libquadmath.texi:374:
warning: undefined flag: BUGURL.
Reverting the patch from r216027 makes it work again.
I don't see anything named BUILD_LIBQUADMATH coming out of configure, but in
config.log I do see:
BUILD_LIBQUADMATH_FALSE=''
BUILD_LIBQUADMATH_TRUE='#'
Are you sure that this patch was actually tested with a clean build directory
where libquadmath-vers.texi was not already present from a previous build?
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug libquadmath/55821] Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported
2012-12-28 2:19 [Bug libquadmath/55821] New: Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported gerald at pfeifer dot com
` (6 preceding siblings ...)
2014-10-20 2:28 ` sandra at codesourcery dot com
@ 2014-10-20 16:57 ` sandra at codesourcery dot com
2014-10-21 9:02 ` fxcoudert at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: sandra at codesourcery dot com @ 2014-10-20 16:57 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55821
--- Comment #9 from Sandra Loosemore <sandra at codesourcery dot com> ---
Yes, that patch (with regenerated Makefile.in) did the trick. Thanks.
config.log says my configure line is:
$ /scratch/sandra/arm-fsf/src/gcc-mainline/libquadmath/configure --srcdir=/scr
atch/sandra/arm-fsf/src/gcc-mainline/libquadmath --cache-file=./config.cache
--e
nable-multilib --with-cross-host=i686-pc-linux-gnu --enable-threads
--disable-li
bmudflap --disable-libstdcxx-pch --with-gnu-as --with-gnu-ld --enable-shared
--e
nable-lto --enable-symvers=gnu --enable-__cxa_atexit --with-glibc-version=2.19
-
-disable-nls --prefix=/scratch/sandra/arm-fsf/install
--with-sysroot=/scratch/sa
ndra/arm-fsf/install/arm-none-linux-gnueabi/libc
--with-host-libstdcxx=-static-l
ibgcc -Wl,-Bstatic,-lstdc++,-Bdynamic -lm --enable-libgomp --enable-libitm
--ena
ble-libatomic --disable-libssp --enable-poison-system-directories
--with-build-t
ime-tools=/scratch/sandra/arm-fsf/install/arm-none-linux-gnueabi/bin
--enable-la
nguages=c,c++,fortran,lto --program-transform-name=s&^&arm-none-linux-gnueabi-&
--disable-option-checking --with-target-subdir=arm-none-linux-gnueabi
--build=i6
86-pc-linux-gnu --host=arm-none-linux-gnueabi --target=arm-none-linux-gnueabi
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Bug libquadmath/55821] Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported
2012-12-28 2:19 [Bug libquadmath/55821] New: Release tarballs (unconditionally) install libquadmath.info when libquadmath is not supported gerald at pfeifer dot com
` (7 preceding siblings ...)
2014-10-20 16:57 ` sandra at codesourcery dot com
@ 2014-10-21 9:02 ` fxcoudert at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: fxcoudert at gcc dot gnu.org @ 2014-10-21 9:02 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55821
--- Comment #10 from Francois-Xavier Coudert <fxcoudert at gcc dot gnu.org> ---
Author: fxcoudert
Date: Tue Oct 21 08:59:17 2014
New Revision: 216503
URL: https://gcc.gnu.org/viewcvs?rev=216503&root=gcc&view=rev
Log:
PR libquadmath/55821
* Makefile.am: Unconditionally define libquadmath_TEXINFOS.
* Makefile.in: Regenerate.
Modified:
trunk/libquadmath/ChangeLog
trunk/libquadmath/Makefile.am
trunk/libquadmath/Makefile.in
^ permalink raw reply [flat|nested] 10+ messages in thread