Problem compiling helm-git on ArchLinux

[yvonnicknoel]

Hello all,

Trying to compile the (very nice) helm synth under Arch Linux (after migrating from Ubuntu), I get the following:

Code: Select all

Compiling juce_audio_formats.cpp
In file included from ../../../JUCE/modules/juce_audio_formats/codecs/juce_FlacAudioFormat.cpp:78:0,
                 from ../../../JUCE/modules/juce_audio_formats/juce_audio_formats.cpp:100:
../../../JUCE/modules/juce_audio_formats/codecs/flac/libFLAC/lpc_flac.c: In function ‘long int juce::FlacNamespace::lround(double)’:
../../../JUCE/modules/juce_audio_formats/codecs/flac/libFLAC/lpc_flac.c:60:39: error : ‘long int juce::FlacNamespace::lround(double)’ conflicts with a previous declaration
 static inline long int lround(double x) {

Did anybody get that too?

Yvonnick

[sysrqer]

Is that from the aur package? I compiled that fairly recently and it was ok.

[CrocoDuck]

The AUR package is failing to build also on my system, very same error. It could be related to this bug upstream.

[CrocoDuck]

I have just raised the issue on the AUR, but probably we will have to submit an issue to the helm developers. I am gonna try to see if I can make some more troubleshooting.

[CrocoDuck]

From the package maintainer:

Yes, this error affects all software that uses JUCE and is compiled with GCC 6.1. It's up to upstream to fix this.

Seems like there is plenty of software around that still have to cope with the new gcc. Nothing to do here until the developers fix it upstream.

[CrocoDuck]

uhm... now that I think about it you could just downgrade gcc and see if by that you can build the packages you need... I am not sure if downgrading something like the compiler is a good practice... Maybe it would be better to set up a minimal Arch virtual machine with no gui you just use to build the binaries until the developers fix the bug. Or boot Manjaro or Antergos live, install the old gcc and build the package there. Then you should be able to install the binaries on every system.

[witchspace]

Where is that other declaration of lround? No problems compiling helm git but I've only done so on Ubuntu.

[CrocoDuck]

I have tried to build this on Manjaro 15.12. I was successful in building sooperlooper and then I decided to give it a go. Unfortunately, it stopped at the same point of the other user in the AUR page:

Code: Select all

Compiling juce_graphics.cpp
In file included from ../../../JUCE/modules/juce_graphics/juce_graphics.cpp:129:0:
../../../JUCE/modules/juce_graphics/image_formats/juce_PNGLoader.cpp: In function ‘void juce::PNGHelpers::errorCallback(juce::pnglibNamespace::png_structp, juce::pnglibNamespace::png_const_charp)’:
../../../JUCE/modules/juce_graphics/image_formats/juce_PNGLoader.cpp:323:31: error: invalid use of incomplete type ‘juce::pnglibNamespace::png_struct {aka struct juce::pnglibNamespace::png_struct_def}’
         longjmp (*(jmp_buf*) p->error_ptr, 1);
                               ^
In file included from ../../../JUCE/modules/juce_graphics/image_formats/juce_PNGLoader.cpp:289:0,
                 from ../../../JUCE/modules/juce_graphics/juce_graphics.cpp:129:
/usr/include/png.h:458:16: note: forward declaration of ‘juce::pnglibNamespace::png_struct {aka struct juce::pnglibNamespace::png_struct_def}’
 typedef struct png_struct_def png_struct;
                ^
In file included from ../../../JUCE/modules/juce_graphics/juce_graphics.cpp:129:0:
../../../JUCE/modules/juce_graphics/image_formats/juce_PNGLoader.cpp: In function ‘juce::Image juce::PNGHelpers::readImage(juce::InputStream&, juce::pnglibNamespace::png_structp, juce::pnglibNamespace::png_infop)’:
../../../JUCE/modules/juce_graphics/image_formats/juce_PNGLoader.cpp:446:101: error: invalid use of incomplete type ‘juce::pnglibNamespace::png_info {aka struct juce::pnglibNamespace::png_info_def}’
 geFromData ((colorType & PNG_COLOR_MASK_ALPHA) != 0 || pngInfoStruct->num_trans
                                                                     ^
In file included from ../../../JUCE/modules/juce_graphics/image_formats/juce_PNGLoader.cpp:289:0,
                 from ../../../JUCE/modules/juce_graphics/juce_graphics.cpp:129:
/usr/include/png.h:472:16: note: forward declaration of ‘juce::pnglibNamespace::png_info {aka struct juce::pnglibNamespace::png_info_def}’
 typedef struct png_info_def png_info;
                ^
Makefile:765: recipe for target 'build/intermediate/Release/juce_graphics_a34b2885.o' failed
make[1]: *** [build/intermediate/Release/juce_graphics_a34b2885.o] Error 1
make[1]: Leaving directory '/home/manjaro/Downloads/helm-git/src/helm/standalone/builds/linux'
Makefile:25: recipe for target 'standalone' failed
make: *** [standalone] Error 2
==> ERROR: A failure occurred in build().
    Aborting...

On Manjaro 15.12 gcc appears to be at version 5.3 tho:

Code: Select all

local/gcc-libs-multilib 5.3.0-3
    Runtime libraries shipped by GCC for multilib
local/gcc-multilib 5.3.0-3 (multilib-devel)
    The GNU Compiler Collection - C and C++ frontends for multilib
local/lib32-gcc-libs 5.3.0-3
    Runtime libraries shipped by GCC (32-bit)

Guess I will notify the maintainer about this...

[davephillips]

Greetings,

Building from a fresh pull this morning, everything runs along smoothly until the same point reported in this thread.

Fedora 21 x86_64, GCC 4.9.2

Best,

dp

[CrocoDuck]

GCC 4.9.2

Weird. It should build with your gcc If I am not wrong. Are you including the VST SDK?

[davephillips]

GCC 4.9.2

Are you including the VST SDK?

Yes indeed. The problem seems to involve libpng, I think.

Best,

dp

[CrocoDuck]

The problem seems to involve libpng, I think.

Oh right. There is an issue opened on github.

[CrocoDuck]

Just in case you have to see the good new yet, the AUR package builds now.