Comment 1 John Marino 2014-08-07 19:05:41 UTC

My favorite, a -devel port.

Notifying maintainer in CC about the problem.

Created attachment 145572 [details]
Update for clang 3.4

Clang 3.4 now stops with an error on unknown options (or is it just some options?)

I don't see any problems removing the -fno-implement-inlines option. It shouldn't cause a problem with other compiler or older versions of clang.

Attaching an update patch that removes the -fno-implement-inlines from CMAKE_CXX_FLAGS

(In reply to FreeBSD from comment #3)
> Clang 3.4 now stops with an error on unknown options (or is it just some
> options?)
> 
> I don't see any problems removing the -fno-implement-inlines option. It
> shouldn't cause a problem with other compiler or older versions of clang.
> 
> Attaching an update patch that removes the -fno-implement-inlines from
> CMAKE_CXX_FLAGS

after the removal of the flag compilation "-fno-implement-inlines" - I have the following problem:

make build deinstall install
===>   hydrogen-devel-0.9.6r2 depends on package: ladspa>=0 - found
===>   hydrogen-devel-0.9.6r2 depends on file: /usr/local/bin/cmake - found
===>   hydrogen-devel-0.9.6r2 depends on executable: update-desktop-database - found
===>   hydrogen-devel-0.9.6r2 depends on executable: pkgconf - found
===>   hydrogen-devel-0.9.6r2 depends on file: /usr/local/lib/qt4/libQtCore.so - found
===>   hydrogen-devel-0.9.6r2 depends on file: /usr/local/lib/qt4/libQtGui.so - found
===>   hydrogen-devel-0.9.6r2 depends on file: /usr/local/bin/linguist-qt4 - found
===>   hydrogen-devel-0.9.6r2 depends on file: /usr/local/bin/moc-qt4 - found
===>   hydrogen-devel-0.9.6r2 depends on file: /usr/local/lib/qt4/libQtNetwork.so - found
===>   hydrogen-devel-0.9.6r2 depends on file: /usr/local/bin/qmake-qt4 - found
===>   hydrogen-devel-0.9.6r2 depends on file: /usr/local/lib/qt4/libQt3Support.so - found
===>   hydrogen-devel-0.9.6r2 depends on file: /usr/local/bin/rcc - found
===>   hydrogen-devel-0.9.6r2 depends on file: /usr/local/lib/qt4/libQtSql.so - found
===>   hydrogen-devel-0.9.6r2 depends on file: /usr/local/bin/uic-qt4 - found
===>   hydrogen-devel-0.9.6r2 depends on file: /usr/local/lib/qt4/libQtXml.so - found
===>   hydrogen-devel-0.9.6r2 depends on file: /usr/local/bin/ccache - found
===>   hydrogen-devel-0.9.6r2 depends on shared library: libsndfile.so - found (/usr/local/lib/libsndfile.so.1.0.25)
===>   hydrogen-devel-0.9.6r2 depends on shared library: libtar.so - found (/usr/local/lib/libtar.so.0)
===>   hydrogen-devel-0.9.6r2 depends on shared library: libasound.so - found (/usr/local/lib/libasound.so.2.0.0)
===>   hydrogen-devel-0.9.6r2 depends on shared library: libjack.so - found (/usr/local/lib/libjack.so.0.0.28)
===>   hydrogen-devel-0.9.6r2 depends on shared library: liblash.so - found (/usr/local/lib/liblash.so.1.1.1)
===>   hydrogen-devel-0.9.6r2 depends on shared library: libarchive.so - found (/usr/lib/libarchive.so.6) - found (/usr/local/lib/libarchive.so.13.1.2)
===>   hydrogen-devel-0.9.6r2 depends on shared library: libportaudio.so.2 - found (/usr/local/lib/portaudio2/libportaudio.so.2.0.0)
===>   hydrogen-devel-0.9.6r2 depends on shared library: liblrdf.so - found (/usr/local/lib/liblrdf.so.2.0.0)
===>   hydrogen-devel-0.9.6r2 depends on shared library: librubberband.so - found (/usr/local/lib/librubberband.so.2.1.0) - found (/usr/local/lib/ardour2/librubberband.so)
===>  Configuring for hydrogen-devel-0.9.6r2
===>  Performing out-of-source build
/bin/mkdir -p /media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/.build
-- The C compiler identification is Clang 3.4.1
-- The CXX compiler identification is Clang 3.4.1
-- Check for working C compiler: /usr/local/libexec/ccache/cc
-- Check for working C compiler: /usr/local/libexec/ccache/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/local/libexec/ccache/c++
-- Check for working CXX compiler: /usr/local/libexec/ccache/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Found ZLIB: /usr/lib/libz.so (found version "1.2.8") 
-- Looking for include file pthread.h
-- Looking for include file pthread.h - found
-- Found Threads: TRUE  
-- Checking SSCANF useability - TRUE
-- Checking RTCLOCK useability - TRUE
-- Looking for include file sys/types.h
-- Looking for include file sys/types.h - found
-- Looking for include file sys/stat.h
-- Looking for include file sys/stat.h - found
-- Looking for include file libtar.h
-- Looking for include file libtar.h - not found
-- Looking for tar_open in tar
-- Looking for tar_open in tar - not found
-- Looking for tar_close in tar
-- Looking for tar_close in tar - not found
-- Looking for tar_extract_all in tar
-- Looking for tar_extract_all in tar - not found
-- Found PkgConfig: /usr/local/bin/pkg-config (found version "0.28") 
-- checking for module 'tar'
--   package 'tar' not found
-- Found LIBTAR: /usr/local/lib/libtar.so  
-- checking for module 'archive'
--   package 'archive' not found
-- Found LIBARCHIVE: /usr/local/lib/libarchive.so  
-- checking for module 'sndfile'
--   found sndfile, version 1.0.25
-- Found LIBSNDFILE: sndfile  
-- checking for module 'alsa'
--   found alsa, version 1.0.27.2
-- Found ALSA: asound  
-- Checking for module 'LADSPA'
Warning: You do not have a LADSPA_PATH environment variable set.
-- Found LADSPA: /usr/local/lib/ladspa  
-- checking for module 'oss'
--   package 'oss' not found
-- Found OSS: /usr/lib/libc.so  
-- checking for module 'jack'
--   found jack, version 0.121.3
-- Found JACK: jack  
-- checking for module 'jack'
--   found jack, version 0.121.3
-- Found JACKSESSION: jack  
-- checking for module 'portaudio-2.0'
--   found portaudio-2.0, version 19
-- Found PORTAUDIO: portaudio;m;pthread  
-- checking for module 'portmidi'
--   package 'portmidi' not found
-- Could NOT find PORTMIDI (missing:  PORTMIDI_LIBRARIES PORTMIDI_INCLUDE_DIR) 
-- checking for module 'pulseaudio'
--   package 'pulseaudio' not found
-- Found PULSEAUDIO: /usr/local/lib/libpulse.so  
-- checking for module 'lash-1.0'
--   found lash-1.0, version 0.5.4
-- Found LASH: lash;jack  
-- checking for module 'lrdf'
--   found lrdf, version 0.5.0
-- Found LRDF: lrdf  
-- checking for module 'rubberband'
--   found rubberband, version 1.8.1
-- Found RUBBERBAND: rubberband  
-- checking for module 'cppunit'
--   found cppunit, version 1.12.1
-- Found CPPUNIT: cppunit  
-- Looking for Q_WS_X11
-- Looking for Q_WS_X11 - found
-- Looking for Q_WS_WIN
-- Looking for Q_WS_WIN - not found
-- Looking for Q_WS_QWS
-- Looking for Q_WS_QWS - not found
-- Looking for Q_WS_MAC
-- Looking for Q_WS_MAC - not found
-- Found Qt4: /usr/local/bin/qmake-qt4 (found suitable version "4.8.6", minimum required is "4.3.0") 
-- Could NOT find Doxygen (missing:  DOXYGEN_EXECUTABLE) 
Installation Summary
--------------------
* Install Directory            : /usr/local
* User data path               : .hydrogen-devel/data
* System data path             : /usr/local/share/hydrogen-devel/data
* core library build as        : SHARED
* debug capabilities           : TRUE
* macosx bundle                : FALSE

Main librarires
* libQt                        : 4.8.6
* libsndfile                   : + used 1.0.25 ( sndfile )
*                                libsndfile supports FLAC and OGG sound file formats
* libtar                       : ? available but not desired  ( /usr/local/lib/libtar.so )
* libarchive                   : + used  ( /usr/local/lib/libarchive.so )
* ladspa                       : + used  ( /usr/local/lib/ladspa )

Supported audio interfaces
--------------------------
* ALSA                         : + used 1.0.27.2 ( asound )
* OSS                          : + used  ( /usr/lib/libc.so )
* JACK                         : + used 0.121.3 ( jack )
* JACKSESSION                  : + used 0.121.3 ( jack )
* CoreAudio                    : - not found and not desired
* CoreMidi                     : - not found and not desired
* PortAudio                    : + used 19 ( portaudio;m;pthread )
* PortMidi                     : - not found and not desired
* PulseAudio                   : + used  ( /usr/local/lib/libpulse.so )

Usefull extensions
-----------------------------------------
* LASH                         : + used 0.5.4 ( lash;jack )
* LRDF                         : + used 0.5.0 ( lrdf )
* RUBBERBAND                   : + used 1.8.1 ( rubberband )
*                                The use of librubberband2 is marked as experimental.
*				 Because the current implementation produce wrong timing!
*				 So long this bug isn't solved, please disable this option.
*				 If rubberband-cli is installed, the hydrogen rubberband-fuction
*				 will work properly as expected.

Miscellaneous capabilities
-----------------------------------------
* realtime clock               : TRUE
* working sscanf               : TRUE
* unit tests                   : + used 1.12.1 ( cppunit )

-----------------------------------------------------------------
IMPORTANT:
  after installing missing packages, remove /media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/.build/CMakeCache.txt before
  running cmake again!
-----------------------------------------------------------------

-----------------------------------------------------------------
IMPORTANT:
  previously installed hydrogen headers found in /usr/local/include
  you should uninstall these files before building hydrogen unless you know what you are doing.
-----------------------------------------------------------------

-- Configuring done
-- Generating done
CMake Warning:
  Manually-specified variables were not used by the project:

    CMAKE_C_FLAGS_DEBUG
    CMAKE_C_FLAGS_RELEASE
    CMAKE_MODULE_LINKER_FLAGS


-- Build files have been written to: /media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/.build
===>  Building for hydrogen-devel-0.9.6r2
Scanning dependencies of target hydrogen-core-0.9.6
[  1%] Building CXX object src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/alsa_midi_driver.cpp.o
[  1%] Building CXX object src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/alsa_audio_driver.cpp.o
[  1%] Building CXX object src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/coremidi_driver.cpp.o
[  1%] Building CXX object src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/coreaudio_driver.cpp.o
[  2%] Building CXX object src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/disk_writer_driver.cpp.o
[  2%] Building CXX object src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/fake_driver.cpp.o
[  3%] Building CXX object src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/jack_midi_driver.cpp.o
/media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/hydrogen-music-hydrogen-2ff9010/src/core/src/IO/alsa_midi_driver.cpp:121:34: warning: adding 'int' to a string does not append to the string [-Wstring-plus-int]
        __INFOLOG( "MIDI addr client: " +  m_dest_addr_client );
                   ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
/usr/local/include/hydrogen/object.h:129:65: note: expanded from macro '__INFOLOG'
#define __INFOLOG(x)    __LOG_OBJ( H2Core::Logger::Info,       (x) );
                                                                ^
/usr/local/include/hydrogen/object.h:111:147: note: expanded from macro '__LOG_OBJ'
#define __LOG_OBJ(      lvl, msg )  if( __object->logger()->should_log( (lvl) ) )       { __object->logger()->log( (lvl), 0, __PRETTY_FUNCTION__, msg ); }
                                                                                                                                                  ^
/media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/hydrogen-music-hydrogen-2ff9010/src/core/src/IO/alsa_midi_driver.cpp:121:34: note: use array indexing to silence this warning
        __INFOLOG( "MIDI addr client: " +  m_dest_addr_client );
                                        ^
                   &                    [
/usr/local/include/hydrogen/object.h:129:65: note: expanded from macro '__INFOLOG'
#define __INFOLOG(x)    __LOG_OBJ( H2Core::Logger::Info,       (x) );
                                                                ^
/usr/local/include/hydrogen/object.h:111:147: note: expanded from macro '__LOG_OBJ'
#define __LOG_OBJ(      lvl, msg )  if( __object->logger()->should_log( (lvl) ) )       { __object->logger()->log( (lvl), 0, __PRETTY_FUNCTION__, msg ); }
                                                                                                                                                  ^
/media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/hydrogen-music-hydrogen-2ff9010/src/core/src/IO/alsa_midi_driver.cpp:122:32: warning: adding 'int' to a string does not append to the string [-Wstring-plus-int]
        __INFOLOG( "MIDI addr port: " + m_dest_addr_port );
                   ~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~
/usr/local/include/hydrogen/object.h:129:65: note: expanded from macro '__INFOLOG'
#define __INFOLOG(x)    __LOG_OBJ( H2Core::Logger::Info,       (x) );
                                                                ^
/usr/local/include/hydrogen/object.h:111:147: note: expanded from macro '__LOG_OBJ'
#define __LOG_OBJ(      lvl, msg )  if( __object->logger()->should_log( (lvl) ) )       { __object->logger()->log( (lvl), 0, __PRETTY_FUNCTION__, msg ); }
                                                                                                                                                  ^
/media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/hydrogen-music-hydrogen-2ff9010/src/core/src/IO/alsa_midi_driver.cpp:122:32: note: use array indexing to silence this warning
        __INFOLOG( "MIDI addr port: " + m_dest_addr_port );
                                      ^
                   &                  [
/usr/local/include/hydrogen/object.h:129:65: note: expanded from macro '__INFOLOG'
#define __INFOLOG(x)    __LOG_OBJ( H2Core::Logger::Info,       (x) );
                                                                ^
/usr/local/include/hydrogen/object.h:111:147: note: expanded from macro '__LOG_OBJ'
#define __LOG_OBJ(      lvl, msg )  if( __object->logger()->should_log( (lvl) ) )       { __object->logger()->log( (lvl), 0, __PRETTY_FUNCTION__, msg ); }
                                                                                                                                                  ^
2 warnings generated.
[  3%] Building CXX object src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/jack_output.cpp.o
[  3%] Building CXX object src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/midi_input.cpp.o
[  4%] Building CXX object src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/midi_output.cpp.o
[  4%] Building CXX object src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/null_driver.cpp.o
/media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/hydrogen-music-hydrogen-2ff9010/src/core/src/IO/jack_output.cpp:767:12: error: no member named 'get_filename' in 'Playlist'
                if ( PL->get_filename().isEmpty() ) PL->set_filename( "untitled.h2playlist" );
                     ~~  ^
/media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/hydrogen-music-hydrogen-2ff9010/src/core/src/IO/jack_output.cpp:767:43: error: no member named 'set_filename' in 'Playlist'
                if ( PL->get_filename().isEmpty() ) PL->set_filename( "untitled.h2playlist" );
                                                    ~~  ^
/media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/hydrogen-music-hydrogen-2ff9010/src/core/src/IO/jack_output.cpp:769:37: error: no member named 'get_filename' in 'Playlist'
                QString FileName = baseName ( PL->get_filename() );
                                              ~~  ^
/media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/hydrogen-music-hydrogen-2ff9010/src/core/src/IO/jack_output.cpp:778:30: error: no member named 'getPath' in 'H2Core::SongReader'
                        QString SongPath = reader.getPath ( H->m_PlayList[i].m_hFile );
                                           ~~~~~~ ^
/media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/hydrogen-music-hydrogen-2ff9010/src/core/src/IO/jack_output.cpp:791:14: error: no member named 'save' in 'Playlist'
                if ( ! PL->save ( jackSessionDirectory + FileName ) )
                       ~~  ^
5 errors generated.
--- src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/jack_output.cpp.o ---
*** [src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/jack_output.cpp.o] Error code 1

make[4]: stopped in /media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/.build
/media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/hydrogen-music-hydrogen-2ff9010/src/core/src/IO/midi_input.cpp:114:5: error: use of undeclared identifier 'handleProgramChangeMessage'; did you mean 'handleControlChangeMessage'?
                                handleProgramChangeMessage( msg );
                                ^~~~~~~~~~~~~~~~~~~~~~~~~~
                                handleControlChangeMessage
/usr/local/include/hydrogen/IO/MidiInput.h:52:7: note: 'handleControlChangeMessage' declared here
        void handleControlChangeMessage( const MidiMessage& msg );
             ^
/media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/hydrogen-music-hydrogen-2ff9010/src/core/src/IO/midi_input.cpp:184:17: error: out-of-line definition of 'handleProgramChangeMessage' does not match any declaration in 'H2Core::MidiInput'
void MidiInput::handleProgramChangeMessage( const MidiMessage& msg )
                ^~~~~~~~~~~~~~~~~~~~~~~~~~
/media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/hydrogen-music-hydrogen-2ff9010/src/core/src/IO/midi_input.cpp:191:28: error: no member named 'getPCAction' in 'MidiMap'; did you mean 'getCCAction'?
        MidiAction *pAction = mM->getPCAction();
                                  ^~~~~~~~~~~
                                  getCCAction
/usr/local/include/hydrogen/midi_map.h:56:16: note: 'getCCAction' declared here
                MidiAction * getCCAction( int parameter );
                             ^
/media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/hydrogen-music-hydrogen-2ff9010/src/core/src/IO/midi_input.cpp:191:40: error: too few arguments to function call, single argument 'parameter' was not specified
        MidiAction *pAction = mM->getPCAction();
                              ~~~~~~~~~~~~~~~ ^
/usr/local/include/hydrogen/midi_map.h:56:3: note: 'getCCAction' declared here
                MidiAction * getCCAction( int parameter );
                ^
4 errors generated.
--- src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/midi_input.cpp.o ---
*** [src/core/CMakeFiles/hydrogen-core-0.9.6.dir/src/IO/midi_input.cpp.o] Error code 1

make[4]: stopped in /media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/.build
2 errors

make[4]: stopped in /media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/.build
--- src/core/CMakeFiles/hydrogen-core-0.9.6.dir/all ---
*** [src/core/CMakeFiles/hydrogen-core-0.9.6.dir/all] Error code 2

make[3]: stopped in /media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/.build
1 error

make[3]: stopped in /media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/.build
--- all ---
*** [all] Error code 2

make[2]: stopped in /media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/.build
1 error

make[2]: stopped in /media/da0s1/obj/usr/ports/audio/hydrogen-devel/work/.build
===> Compilation failed unexpectedly.
Try to set MAKE_JOBS_UNSAFE=yes and rebuild before reporting the failure to
the maintainer.
*** Error code 1

Stop.
make[1]: stopped in /usr/ports/audio/hydrogen-devel
*** Error code 1

Stop.
make: stopped in /usr/ports/audio/hydrogen-devel

You have the previous version installed and it conflicts. There was meant to be a notice printed in this situation but it wasn't included in the commit, that's what threw me off.

You can either uninstall hydrogen before building or rename /usr/local/include/hydrogen during the build.

Comment 6 John Marino 2014-08-12 19:53:38 UTC

Or build your ports in a clean poudriere environment and install them with pkg from the resulting local reposititory

Comment 7 Carlo Strub 2014-08-29 19:40:28 UTC

There seems to be another PR that resolves this bug: add a dependency