Bug#812034: klick: FTBFS with GCC 6: needed for in-class initialization

2016-07-11 Thread James Cowgill
On Mon, 2016-07-11 at 20:01 +0200, Jaromír Mikeš wrote:
> 2016-01-20 5:41 GMT+01:00 Martin Michlmayr :
> > Package: klick
> > Version: 0.12.2-2
> > Severity: important
> > User: debian-...@lists.debian.org
> > Usertags: ftbfs-gcc-6
> > 
> > This package fails to build with GCC 6.  GCC 6 has not been
> > released
> > yet, but it's expected that GCC 6 will become the default compiler
> > for
> > stretch.
> > 
> > Note that only the first error is reported; there might be
> > more.  You
> > can find a snapshot of GCC 6 in experimental.  To build with GCC 6,
> > you can set CC=gcc-6 CXX=g++-6 explicitly.
>
> I'm trying fix this issue
> I added gcc-6 and g++-6 as build-deps and these lines to d/rules:
> export CC = gcc-6
> export CXX = g++-6
> 
> Not sure if I still missing something but package build fine this
> way...

scons seems to ignore the CC and CXX environment variables completely.
If I build klick with gcc/g++ from experimental I can reproduce the
FTBFS.

James

signature.asc
Description: This is a digitally signed message part


Bug#812034: klick: FTBFS with GCC 6: needed for in-class initialization

2016-07-11 Thread Jaromír Mikeš
2016-01-20 5:41 GMT+01:00 Martin Michlmayr :
> Package: klick
> Version: 0.12.2-2
> Severity: important
> User: debian-...@lists.debian.org
> Usertags: ftbfs-gcc-6
>
> This package fails to build with GCC 6.  GCC 6 has not been released
> yet, but it's expected that GCC 6 will become the default compiler for
> stretch.
>
> Note that only the first error is reported; there might be more.  You
> can find a snapshot of GCC 6 in experimental.  To build with GCC 6,
> you can set CC=gcc-6 CXX=g++-6 explicitly.

Hi,

I'm trying fix this issue
I added gcc-6 and g++-6 as build-deps and these lines to d/rules:
export CC = gcc-6
export CXX = g++-6

Not sure if I still missing something but package build fine this way...

best regards

mira



Bug#812034: klick: FTBFS with GCC 6: needed for in-class initialization

2016-01-19 Thread Martin Michlmayr
Package: klick
Version: 0.12.2-2
Severity: important
User: debian-...@lists.debian.org
Usertags: ftbfs-gcc-6

This package fails to build with GCC 6.  GCC 6 has not been released
yet, but it's expected that GCC 6 will become the default compiler for
stretch.

Note that only the first error is reported; there might be more.  You
can find a snapshot of GCC 6 in experimental.  To build with GCC 6,
you can set CC=gcc-6 CXX=g++-6 explicitly.

You may be able to find out more about this issue at
https://gcc.gnu.org/gcc-6/changes.html

> sbuild (Debian sbuild) 0.67.0 (26 Dec 2015) on dl580gen9-02.hlinux
...
> g++ -o src/klick.o -c -Wdate-time -D_FORTIFY_SOURCE=2 -g -W -Wall 
> -DVERSION=\"0.12.2\" -DDATA_DIR=\"/usr/share/klick\" -DHAVE_SNDFILE_OGG 
> -DENABLE_OSC -DENABLE_TERMINAL -DENABLE_RUBBERBAND src/klick.cc
> In file included from src/klick.cc:26:0:
> src/metronome_map.hh:51:42: error: 'constexpr' needed for in-class 
> initialization of static data member 'const double 
> MetronomeMap::TICKS_PER_BEAT' of non-integral type [-fpermissive]
>  static double const TICKS_PER_BEAT = 1920.0;
>   ^~
> 
> In file included from src/klick.cc:28:0:
> src/metronome_simple.hh:62:38: error: 'constexpr' needed for in-class 
> initialization of static data member 'const float 
> MetronomeSimple::MAX_TAP_AGE' of non-integral type [-fpermissive]
>  static float const MAX_TAP_AGE = 3.0;
>   ^~~
> 
> src/metronome_simple.hh:63:35: error: 'constexpr' needed for in-class 
> initialization of static data member 'const float MetronomeSimple::TAP_DIFF' 
> of non-integral type [-fpermissive]
>  static float const TAP_DIFF = 0.2;
>^~~
> 
> scons: *** [src/klick.o] Error 1
> scons: building terminated because of errors.

-- 
Martin Michlmayr
Linux for HPE Helion, Hewlett Packard Enterprise