Opened 8 years ago
Closed 8 years ago
#52068 closed defect (fixed)
darkice needs pkgconfig dependency
Reported by: | ryandesign (Ryan Carsten Schmidt) | Owned by: | nielsd (Niels Dettenbach) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.3.4 |
Keywords: | haspatch | Cc: | kurthindenburg (Kurt Hindenburg) |
Port: | darkice |
Description
darkice 1.3 fails to build:
https://build.macports.org/builders/ports-10.11_x86_64-builder/builds/1812
checking for cc_r... /usr/bin/clang checking for lame library at /opt/local ... found at /opt/local checking for VORBIS... no checking for OPUS... no checking for faac library at /usr ... configure: WARNING: not found, building without faac checking for AACPLUS... no checking for TWOLAME... no checking for ALSA... no checking for PULSEAUDIO... no checking for JACK... no configure: error: in `/opt/local/var/macports/build/_opt_bblocal_var_buildworker_ports_build_dports_audio_darkice/darkice/work/darkice-1.3': configure: error: The pkg-config script could not be found or is too old. Make sure it is in your PATH or set the PKG_CONFIG environment variable to the full path to pkg-config. Alternatively, you may set the environment variables JACK_CFLAGS and JACK_LIBS to avoid the need to call pkg-config. See the pkg-config man page for more details. To get pkg-config, see <http://pkg-config.freedesktop.org/>. See `config.log' for more details
Evidently, darkice needs pkgconfig to find jack, so a build dependency on port:pkgconfig should be added to the port somewhere. But where?
If only the jack variant needs pkgconfig, then put the dependency in the jack variant.
It looks lame was detected without pkgconfig. What about the other variants, which are not default variants? If some of them require pkgconfig, then the pkgconfig dependency should be made in a conditional outside of a variant directive (to avoid each variant declaring the same dependency). For example, if only jack and faac need pkgconfig, you could do:
if {[variant_isset faac] || [variant_isset jack]} { depends_build-append port:pkgconfig }
Attachments (1)
Change History (3)
Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)
Attachment: | darkice.diff added |
---|
comment:1 Changed 8 years ago by ryandesign (Ryan Carsten Schmidt)
Keywords: | haspatch added |
---|
comment:2 Changed 8 years ago by kurthindenburg (Kurt Hindenburg)
Cc: | khindenburg@… added |
---|---|
Resolution: | → fixed |
Status: | new → closed |
looks like this was committed
r152311 | ryandesign@macports.org | 2016-09-03 09:54:11 -0400 (Sat, 03 Sep 2016) | 2 lines darkice: add pkgconfig dependency (#52051; maintainer)
The discussion moved to the mailing list, where we decided that everything except lame requires pkgconfig, so it's ok to just add a general pkgconfig dependency, as in the attached patch.