Re: [PATCH] Introduce --with-gcc-major-version-only configure option (take 2)

2017-01-20 Thread Jakub Jelinek
On Fri, Jan 20, 2017 at 12:12:17AM +0100, Matthias Klose wrote: > On 19.01.2017 13:26, Jakub Jelinek wrote: > > On Thu, Jan 19, 2017 at 01:13:14PM +0100, Franz Sirl wrote: > >> Am 2017-01-12 um 21:16 schrieb Jakub Jelinek: > >>> libmpx/ > >>> * configure.ac: Add GCC_BASE_VER. > >>> * Makefile.a

Re: [PATCH] Introduce --with-gcc-major-version-only configure option (take 2)

2017-01-19 Thread Matthias Klose
On 19.01.2017 13:26, Jakub Jelinek wrote: > On Thu, Jan 19, 2017 at 01:13:14PM +0100, Franz Sirl wrote: >> Am 2017-01-12 um 21:16 schrieb Jakub Jelinek: >>> libmpx/ >>> * configure.ac: Add GCC_BASE_VER. >>> * Makefile.am (gcc_version): Use @get_gcc_base_ver@ instead of cat to >>> get ve

Re: [PATCH] Introduce --with-gcc-major-version-only configure option (take 2)

2017-01-19 Thread Jakub Jelinek
On Thu, Jan 19, 2017 at 01:13:14PM +0100, Franz Sirl wrote: > Am 2017-01-12 um 21:16 schrieb Jakub Jelinek: > > libmpx/ > > * configure.ac: Add GCC_BASE_VER. > > * Makefile.am (gcc_version): Use @get_gcc_base_ver@ instead of cat to > > get version from BASE-VER file. > > * configure

Re: [PATCH] Introduce --with-gcc-major-version-only configure option (take 2)

2017-01-19 Thread Franz Sirl
Am 2017-01-12 um 21:16 schrieb Jakub Jelinek: libmpx/ * configure.ac: Add GCC_BASE_VER. * Makefile.am (gcc_version): Use @get_gcc_base_ver@ instead of cat to get version from BASE-VER file. * configure: Regenerated. Hi, it seems libmpx/configure.ac is missing th

Re: [PATCH] Introduce --with-gcc-major-version-only configure option

2017-01-18 Thread Andris Pavenis
Just one suggestion: What about configure option like --with-version-convert where one could specify actual conversion? I have used conversion sed -e 's:\.::2g' for DJGPP to leave only the first dot in version for complying with MS-DOS file name restrictions. The implementation of that was not

Re: [PATCH] Introduce --with-gcc-major-version-only configure option (take 2)

2017-01-13 Thread Richard Biener
On Thu, Jan 12, 2017 at 9:16 PM, Jakub Jelinek wrote: > Hi! > > On Tue, Jan 10, 2017 at 07:56:36AM +0100, Jakub Jelinek wrote: >> As I said on IRC, I think -dumpversion of 7 in this configuration is the >> right thing to do. In GCC sources, we have 3 uses of -dumpversion, >> two of them look like

[PATCH] Introduce --with-gcc-major-version-only configure option (take 2)

2017-01-12 Thread Jakub Jelinek
Hi! On Tue, Jan 10, 2017 at 07:56:36AM +0100, Jakub Jelinek wrote: > As I said on IRC, I think -dumpversion of 7 in this configuration is the > right thing to do. In GCC sources, we have 3 uses of -dumpversion, > two of them look like: > gcc_version := $(shell $(GOC) -dumpversion) > ... > toolexe

Re: [PATCH] Introduce --with-gcc-major-version-only configure option

2017-01-09 Thread Jakub Jelinek
On Tue, Jan 10, 2017 at 12:15:41AM +0100, Matthias Klose wrote: > On 09.01.2017 21:43, Jakub Jelinek wrote: > > On Fri, Jan 06, 2017 at 01:48:26PM +0100, Jakub Jelinek wrote: > >> Yet another option is introduce AC_ARG_ENABLE into all those configure > >> scripts (some macro in config/*.m4) and do

Re: [PATCH] Introduce --with-gcc-major-version-only configure option

2017-01-09 Thread Matthias Klose
On 09.01.2017 21:43, Jakub Jelinek wrote: > Hi! > > On Fri, Jan 06, 2017 at 01:48:26PM +0100, Jakub Jelinek wrote: >> Yet another option is introduce AC_ARG_ENABLE into all those configure >> scripts (some macro in config/*.m4) and do the sed conditionally. > > Here is a patch to do that. > Boots

[PATCH] Introduce --with-gcc-major-version-only configure option

2017-01-09 Thread Jakub Jelinek
Hi! On Fri, Jan 06, 2017 at 01:48:26PM +0100, Jakub Jelinek wrote: > Yet another option is introduce AC_ARG_ENABLE into all those configure > scripts (some macro in config/*.m4) and do the sed conditionally. Here is a patch to do that. Bootstrapped/regtested on x86_64-linux (without --with-gcc-ma