Bug 87374 - incorrect use of WITH_THEORA vs. WITH_LIBTHEORA in some ports
Summary: incorrect use of WITH_THEORA vs. WITH_LIBTHEORA in some ports
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Michael Johnson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-10-13 17:20 UTC by Zahemszky@, Gábor
Modified: 2005-11-05 07:30 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Zahemszky@, Gábor 2005-10-13 17:20:17 UTC
Many ports in the audio and multimedia categories (and maybe others) have
optional dependencies to special libraries (eg. libogg, libvorbis). It looks
like in the FreeBSD ports-tree, there isn't a correct way to tell the libtheora
dependency.  Eg, in multimedia, there are libxine, vlc and vlc-devel, which use
the WITH_LIBTHEORA (or WITHOUT_LIBTHEORA) Makefile variable, but ffmpeg-devel,
mplayer and transcode use the WITH_THEORA (ot WITHOUT_THEORA) variables.
So if somebody'd like to include (exclude) libtheora, she has to define two
variables in /etc/make.conf: WITH_THEORA and WITH_LIBTHEORA (or w/o). It
would be better to use one; and after the other similar variable's name, I
think WITH_THEORA is the correct name.

Fix: 

In /usr/ports/multimedia/{libxine,vlc,vlc-devel}/Makefile, in the next (or
similar lines), change the WITH(OUT)?_LIBTHEORA strings to WITH(OUT)?_THEORA.

==
if !defined(WITHOUT_LIBTHEORA) && (exists(${LOCALBASE}/lib/libtheora.so.1) ||
defined (WITH_LIBTHEORA))
==
How-To-Repeat: 	egrep -R 'WITH(OUT)?_(LIB)?THEORA' /usr/ports
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2005-10-14 11:23:58 UTC
Responsible Changed
From-To: freebsd-ports-bugs->ahze

Over to maintainer of vlc{,-devel} for further reasoning. 

Ahze, can you afterwards assign it to nobutaka?
Comment 2 Michael Johnson freebsd_committer freebsd_triage 2005-10-15 05:33:00 UTC
State Changed
From-To: open->analyzed

- Over to nobutaka 


Comment 3 Michael Johnson freebsd_committer freebsd_triage 2005-10-15 05:33:00 UTC
Responsible Changed
From-To: ahze->nobutaka

Mark this analyzed
Comment 4 MANTANI Nobutaka freebsd_committer freebsd_triage 2005-10-23 00:40:43 UTC
Responsible Changed
From-To: nobutaka->ahze

Commit to libxine done. Hand this PR over to the maintainer of vlc and vlc-devel.
Comment 5 Michael Johnson freebsd_committer freebsd_triage 2005-11-05 07:29:04 UTC
State Changed
From-To: analyzed->closed

- close this pr since we now have a ports/KNOBS file. vlc 
and vlc-devel knob names will be updated on the next  
release.