Re: [Cs-dev] CMakeLists.txt: PORTMIDI_HEADER not included, wont be found in includepath
Date | 2013-05-04 09:33 |
From | Tim Neumann |
Subject | Re: [Cs-dev] CMakeLists.txt: PORTMIDI_HEADER not included, wont be found in includepath |
Attachments | None None |
(I hope this will be noticed as a proper Re: since gmail wont let me see my own mailinglist entries) Can somebody confirm this? --------------------------------------------------------------------------------------- I just ran into this: Scanning dependencies of target pmidi [ 64%] Building C object InOut/CMakeFiles/pmidi.dir/pmidi.c.obj h:/Builder/Csound6.00rc1.1/InOut/pmidi.c:31:22: fatal error: portmidi.h: No such file or directory compilation terminated. PORTMIDI_HEADER is set correctly in cmake-gui but the actual path/file seems to be ignored. Is there any existing way to include that path with this build target without moving the whole portmidi headers to my /include folder? (portmidi does not provide a make install target) |
Date | 2013-05-04 09:39 |
From | Victor Lazzarini |
Subject | Re: [Cs-dev] CMakeLists.txt: PORTMIDI_HEADER not included, wont be found in includepath |
did you try adding the path to custom.cmake? On 4 May 2013, at 09:33, Tim Neumann wrote: > (I hope this will be noticed as a proper Re: since gmail wont let me see my own mailinglist entries) > > Can somebody confirm this? > > Original message, in case this is not a proper Re: > --------------------------------------------------------------------------------------- > > I just ran into this: > > Scanning dependencies of target pmidi > [ 64%] Building C object InOut/CMakeFiles/pmidi.dir/pmidi.c.obj > h:/Builder/Csound6.00rc1.1/InOut/pmidi.c:31:22: fatal error: portmidi.h: No > such > file or directory > compilation terminated. > > PORTMIDI_HEADER is set correctly in cmake-gui but the actual path/file > seems to be ignored. Is there any existing way to include that path with > this build target without moving the whole portmidi headers to my /include > folder? (portmidi does not provide a make install target) > > ------------------------------------------------------------------------------ > Get 100% visibility into Java/.NET code with AppDynamics Lite > It's a free troubleshooting tool designed for production > Get down to code-level detail for bottlenecks, with <2% overhead. > Download for free and get started troubleshooting in minutes. > http://p.sf.net/sfu/appdyn_d2d_ap2_______________________________________________ > Csound-devel mailing list > Csound-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/csound-devel Dr Victor Lazzarini Senior Lecturer Dept. of Music NUI Maynooth Ireland tel.: +353 1 708 3545 Victor dot Lazzarini AT nuim dot ie ------------------------------------------------------------------------------ Get 100% visibility into Java/.NET code with AppDynamics Lite It's a free troubleshooting tool designed for production Get down to code-level detail for bottlenecks, with <2% overhead. Download for free and get started troubleshooting in minutes. http://p.sf.net/sfu/appdyn_d2d_ap2 _______________________________________________ Csound-devel mailing list Csound-devel@lists.sourceforge.net |
Date | 2013-05-04 09:51 |
From | Tim Neumann |
Subject | Re: [Cs-dev] CMakeLists.txt: PORTMIDI_HEADER not included, wont be found in includepath |
Attachments | None None |
Not yet, I will try that later (booted into Linux...) May I use this mail to kindly point out this unsolved problem, as well?http://sourceforge.net/mailarchive/message.php?msg_id=30795646 Tim 2013/5/4 Victor Lazzarini <Victor.Lazzarini@nuim.ie> did you try adding the path to custom.cmake? |
Date | 2013-05-04 09:57 |
From | Victor Lazzarini |
Subject | Re: [Cs-dev] CMakeLists.txt: PORTMIDI_HEADER not included, wont be found in includepath |
Attachments | None None |
if that ipmidi change is Windows, I would prefer if Mike made and tested this change, as I don't have a windows machine. Victor On 4 May 2013, at 09:51, Tim Neumann wrote:
Dr Victor Lazzarini Senior Lecturer Dept. of Music NUI Maynooth Ireland tel.: +353 1 708 3545 Victor dot Lazzarini AT nuim dot ie |
Date | 2013-05-04 10:02 |
From | Tim Neumann |
Subject | Re: [Cs-dev] CMakeLists.txt: PORTMIDI_HEADER not included, wont be found in includepath |
Attachments | None None |
It is. Please notice that under 64Bit Windows filename is the same, the _32.dll got me confused a little. 2013/5/4 Victor Lazzarini <Victor.Lazzarini@nuim.ie>
|
Date | 2013-05-04 10:21 |
From | Steven Yi |
Subject | Re: [Cs-dev] CMakeLists.txt: PORTMIDI_HEADER not included, wont be found in includepath |
There's a few things here in the this thread. First, there is a questionable set of lines added by Michael in the InOut/CmakeLists.txt: include_directories(${PORTMIDI_INCLUDE_PATH}) include_directories(${PORTTIME_INCLUDE_PATH}) This is the wrong way to do things, IMO. I think the cmake files have two ways of going about it. One is to use check_include_file, using the result of that to determine if the file is on the path. There may be problems if that doesn't work correctly with custom paths. The other way is using find_path, i.e.: find_path(COREMIDI_INCLUDE_PATH CoreMidi.h) Then later, that is used: include_directories(${COREMIDI_INCLUDE_PATH}) So that way, we get CMake to search for it, we can test for it to be true, and we can also use the value. The changes should probably be to remove the two lines Michael added and use find_path/include_directories. 2) This email thread is about "pmidi" and the other one referenced here is about "ipmidi". 3) Regarding ipmidi, the build target does not correct in that it uses ${ipmidi_LIBS} but has nothing defined for it. If on windows it requires libs to link to that aren't required elsewhere, it should have: if(WIN32) list(APPEND ipmidi_LIBS ws2_32) endif() or something along those lines, before the make_plugin line. (Basically, adding to the list of libraries to use when linking ipmidi). There should also be further checks for the existence of ws2_32 and that should be factored into the check to see if the ipmidi library should build. On Sat, May 4, 2013 at 10:02 AM, Tim Neumann |