Bug 191963 - x11-toolkits/fltk-devel: resurrect audio/alsa-lib dependency
Summary: x11-toolkits/fltk-devel: resurrect audio/alsa-lib dependency
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Pietro Cerutti
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-07-19 02:27 UTC by Koichiro Iwao
Modified: 2014-07-21 12:13 UTC (History)
1 user (show)

See Also:

Attachments
builg lod with EXAMPLES option (37.98 KB, text/plain)
2014-07-19 02:27 UTC, Koichiro Iwao 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Koichiro Iwao freebsd_committer freebsd_triage 2014-07-19 02:27:47 UTC 
Created attachment 144788 [details]
builg lod with EXAMPLES option

Why did you remove alsa-lib dependency at r361516?
It is actually required by EXAMPLES option.

Please see attached build log.
Comment 1 John Marino freebsd_committer freebsd_triage 2014-07-20 14:25:11 UTC 
assign to maintainer (he was only on CC before)
Comment 2 Pietro Cerutti freebsd_committer freebsd_triage 2014-07-21 08:15:34 UTC 
The CMake configure system should figure out that alsa is not available, and do without. The reason why I removed it is that it wasn't strictly necessary, as it's only used by sample apps.

We should try to figure out why your configuration tries to use the alsa libraries even when they're not installed (see the top of test/CMakeLists.txt).

Are you using any local patches or anything else that might screw up the configure system?
Comment 3 Pietro Cerutti freebsd_committer freebsd_triage 2014-07-21 12:13:31 UTC 
Ok I figured it out. This happens after my change if you have alsa-lib installed. The reason is the CMake locates alsa/asoundlib.h but doesn't have the correct linker flags to locate libasound.so.

I fixed this in commit r362429, which updates the port to the snapshot 10221.

Thanks,