Opened 13 years ago

Closed 13 years ago

#30251 closed defect (fixed)

[portaudio] [19.20110326] failes to build on OSX 10.7 GM/Final

Reported by: zaphodb@… Owned by: humem (humem)
Priority: Normal Milestone:
Component: ports Version: 2.0.1
Keywords: Cc: g5pw (Aljaž Srebrnič), gaochangjian@…, sjcjonker@…, trottier@…, davmp@…
Port: portaudio

Description

port audio does not build for me on 10.7

:info:build /bin/sh ./libtool --mode=link /Developer/usr/bin/llvm-gcc-4.2 -o bin/patest_stop_playout -I./include -I./src/common -I./src/os/unix -std=c99 -pipe -O2 -arch x86_64 -DPA_LITTLE_ENDIAN     -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\" -DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE_URL=\"\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_DLFCN_H=1 -DLT_OBJDIR=\".libs/\" -DSIZEOF_SHORT=2 -DSIZEOF_INT=4 -DHAVE_NANOSLEEP=1 -DPA_USE_COREAUDIO=1 ./test/patest_stop_playout.c lib/libportaudio.la -framework CoreAudio -framework AudioToolbox -framework AudioUnit -framework Carbon
:info:build In file included from ./test/patest_sine_srate.c:54:
:info:build ./include/pa_mac_core.h:100: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'PaMacCore_GetStreamInputDevice'
:info:build ./include/pa_mac_core.h:109: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'PaMacCore_GetStreamOutputDevice'
:info:build In file included from ./test/patest_sine_channelmaps.c:53:
:info:build ./include/pa_mac_core.h:100: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'PaMacCore_GetStreamInputDevice'
:info:build ./include/pa_mac_core.h:109: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'PaMacCore_GetStreamOutputDevice'
:info:build make: *** [bin/patest_sine_srate] Error 1
:info:build make: *** Waiting for unfinished jobs....
:info:build make: *** [bin/patest_sine_channelmaps] Error 1
:info:build libtool: link: /Developer/usr/bin/llvm-gcc-4.2 -o bin/.libs/patest_stop_playout -I./include -I./src/common -I./src/os/unix -std=c99 -pipe -O2 -arch x86_64 -DPA_LITTLE_ENDIAN -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\" -DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE_URL=\"\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_DLFCN_H=1 -DLT_OBJDIR=\".libs/\" -DSIZEOF_SHORT=2 -DSIZEOF_INT=4 -DHAVE_NANOSLEEP=1 -DPA_USE_COREAUDIO=1 ./test/patest_stop_playout.c  lib/.libs/libportaudio.dylib -L/opt/local/lib -framework Carbon -framework AudioUnit -framework AudioToolbox -framework CoreAudio
:info:build shell command " cd "/opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_ports_audio_portaudio/work/portaudio" && /usr/bin/make -j8 all " returned error 2
:error:build Target org.macports.build returned: shell command failed (see log for details)
:debug:build Backtrace: shell command failed (see log for details)
    while executing
"command_exec build"
    (procedure "portbuild::build_main" line 8)
    invoked from within
"$procedure $targetname"
:info:build Warning: the following items did not execute (for portaudio): org.macports.destroot org.macports.build

Attachments (3)

main.log (14.7 KB) - added by zaphodb@… 13 years ago.
build log
patch-Portfile.diff (323 bytes) - added by gaochangjian@… 13 years ago.
patch for Portfile
patch-include__pa_mac_core.h.diff (325 bytes) - added by gaochangjian@… 13 years ago.
Portfile required patch file

Download all attachments as: .zip

Change History (16)

Changed 13 years ago by zaphodb@…

Attachment: main.log added

build log

comment:1 Changed 13 years ago by g5pw (Aljaž Srebrnič)

Cc: a2piratesoft@… added

Cc Me!

comment:2 Changed 13 years ago by mressl@…

I can confirm the same problem here. I also downloaded the PortAudio latest stable release (pa_stable_v19_20110326.tgz), and can confirm the same problem

comment:3 Changed 13 years ago by gaochangjian@…

Cc: gaochangjian@… added

Cc Me!

comment:4 Changed 13 years ago by sjcjonker@…

Cc: sjcjonker@… added

Cc Me!

comment:5 Changed 13 years ago by davmp@…

Simply need to uncomment the inclusion of AudioToolbox at line 49 of ./include/pa_mac_core.h After this edit, the build and install work for me on Lion.

comment:6 Changed 13 years ago by trottier@…

Cc: trottier@… added

Cc Me!

comment:7 Changed 13 years ago by davmp@…

Cc: davmp@… added

Cc Me!

comment:8 in reply to:  5 ; Changed 13 years ago by gaochangjian@…

Replying to davmp@…:

Simply need to uncomment the inclusion of AudioToolbox at line 49 of ./include/pa_mac_core.h After this edit, the build and install work for me on Lion.

Yeah, I build successfully! I'll submit two patch files.

Changed 13 years ago by gaochangjian@…

Attachment: patch-Portfile.diff added

patch for Portfile

Changed 13 years ago by gaochangjian@…

Portfile required patch file

comment:9 in reply to:  8 ; Changed 13 years ago by sjcjonker@…

Replying to gaochangjian@…:

Replying to davmp@…:

Simply need to uncomment the inclusion of AudioToolbox at line 49 of ./include/pa_mac_core.h After this edit, the build and install work for me on Lion.

Yeah, I build successfully! I'll submit two patch files.

Ehmm an now for the stupid question....

How to include this, I'm familiar with patches, even on linux kernels etc etc. But how to include this in a port package? With ports I never got further then port <install/clean/upgrade> etc?

Is this something like: port clean portaudio port patch portaudio << Here add above patch >>? port configure portaudio port build portaudio port destroot portaudio port install portaudio

Apologies for asking here, besides the mailinglist I didn't know a better place.

comment:10 in reply to:  9 Changed 13 years ago by davmp@…

Replying to sjcjonker@…:

How to include this, I'm familiar with patches, even on linux kernels etc etc. But how to include this in a port package? With ports I never got further then port <install/clean/upgrade> etc?

Sure you can do it manually until a patch gets committed to the macports repo by:

  • sudo port fetch portaudio
  • <manually edit the file now -- see below>
  • sudo port install portaudio

No need to explicitly list every step macports would do!

The file to edit should be in a path like '/opt/local/var/macports/sources/rsync.macports.org/release/tarballs/ports/audio/portaudio/work/portaudio-??/include/pa_mac_core.h

comment:11 in reply to:  9 Changed 13 years ago by gaochangjian@…

Replying to sjcjonker@…:

How to include this, I'm familiar with patches, even on linux kernels etc etc. But how to include this in a port package? With ports I never got further then port <install/clean/upgrade> etc?

  • patch portaudio's Portfile, you could find this file through port file portaudio command
  • put patch-include__pa_mac_core.h.diff file in `port dir portaudio`/files directory

Now you can install portaudio normally.

comment:12 Changed 13 years ago by humem (humem)

Owner: changed from macports-tickets@… to hum@…
Status: newassigned
Version: 1.9.22.0.1

comment:13 Changed 13 years ago by humem (humem)

Resolution: fixed
Status: assignedclosed

Thanks for providing patches. Notice that the patch file for 10.6 (patch-srchostapicoreaudiopa_mac_core.c) is also applied in building on 10.7. Committed in r82196.

Note: See TracTickets for help on using tickets.