Arch Linux User Repository

I've had to remove prepare() to compile this, seems the VST SDK is included in helm sources again? (sed fails because directory structure changed)

Compilation fixed, thanks to @CrocoDuck.

Package now uses steinberg-vst36 instead of including the VST3 SDK in the sources list.

Hi there! Following falkTX advice in this thread (https://linuxmusicians.com/viewtopic.php?f=24&t=15889&p=72486#p72486) I was able to have helm built and installed by changing the build() function to this:

build() {
cd "${srcdir}/${_pkgname}"

CXXFLAGS="${CFLAGS} -DHAVE_LROUND"

make PREFIX="/usr" all vst
}

You know probably better than me whether that is an appropriate PKGBUILD modification. I just skimmed through this: https://wiki.archlinux.org/index.php/Makepkg#Creating_optimized_packages

Thanks SpotlightKid and sorry for the late reply. I tried to build again and it went fine until:

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:60:24: note: candidate: long int juce::FlacNamespace::lround(double)
static inline long int lround(double x) {
^~~~~~
make[1]: *** [Makefile:744: build/intermediate/Release/juce_audio_formats_5c144c69.o] Error 1
make[1]: Leaving directory '/tmp/packerbuild-1000/helm-git/helm-git/src/helm/standalone/builds/linux'
make: *** [Makefile:35: standalone] Error 2
==> ERROR: A failure occurred in build().
Aborting...
The build failed.

Seems like there are many ways this build can go wrong, seems you are right about considering helm git version broken for the time being...

@CrocoDuck: there is a GH issue about the compile error you're getting:

https://github.com/mtytel/helm/issues/76

I've tried the fix suggested there (by patching the Makefile on-the-fly), which means that the libpng version included in the helm sources is used again. The error still occurred, but compilation continued (not sure why?), but then another compiler error occurred:

Compiling juce_VST_Wrapper.cpp
../../../JUCE/modules/juce_audio_plugin_client/VST/juce_VST_Wrapper.cpp:324:29: Warnung: Zeichenkonstante mit mehreren Zeichen [-Wmultichar]
setUniqueID ((int) (JucePlugin_VSTUniqueID));
^
In file included from ../../../JUCE/modules/juce_audio_plugin_client/VST/juce_VST_Wrapper.cpp:72:0:
/home/chris/src/arch/aur/helm-git/src/VST3 SDK/public.sdk/source/vst2.x/audioeffect.cpp: In Elementfunktion »virtual void AudioEffect::int2string(VstInt32, char*, VstInt32)«:
/home/chris/src/arch/aur/helm-git/src/VST3 SDK/public.sdk/source/vst2.x/audioeffect.cpp:534:24: Warnung: verengende Umwandlung von »(((int)((char)digit)) + 48)« von »int« nach »char« in { } [-Wnarrowing]
char temp[2] = {'0' + (char)digit, '\0'};
^

I'm afraid that the git version of helm must be considered broken at the moment. :(

@CrocoDuck: To compile with GCC 5.3, you need to install the "gcc5" AUR package and change the build function accordingly to set the compiler used. If helm/JUCE isn't fixed soon to compile with gcc 6.x, I'll try to incorporate this into this package.

Re the VST SDK: this package was created before the steinberg-vst36 AUR package was available, so I still have to adapt helm-git to use it.

Thanks SpotlightKid. I have tried yo build the package on a live Manjaro 15.12 session, which comes with gcc-multilib 5.3.0-3. It failed at "Compiling juce_graphics.cpp" as for rcrath. I though you might be interested, so I am reporting ;).

By the way, I see that the VST SDK is handled manually here. Unless you did that on purpose, it would probably be better to use this as a build dependence:

https://aur.archlinux.org/packages/steinberg-vst36/

the package contains 2.4 for backward compatibility.

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

Hey there. This could be maybe related to the same error:

http://pastebin.com/zSWPbwRx

This is an error in helm or in JUCE. It has already been reported to the project:

https://github.com/mtytel/helm/issues/76