Re: pkg/30920: audio/mpg321 is missing a dependency on audio/libao-oss

["Julio M. Merino Vidal" <jmmv84%gmail.com@localhost>]

On 8/19/05, Rui-Xiang Guo <rxg%myrealbox.com@localhost> wrote:
> The following reply was made to PR pkg/30920; it has been noted by GNATS.
>  Hi,
>  please try this patch -
[...]
>  .if !empty(PKG_OPTIONS:Mlibao-oss)
>  DEPENDS+=              libao-oss>=0.8.6:../../audio/libao-oss
>  .endif
> 
>  .if !empty(PKG_OPTIONS:Mlibao-sun)
>  DEPENDS+=              libao-sun>=0.8.6:../../audio/libao-sun
>  .endif

The whole purpose of the original split of libao was to avoid
having build-time options, so this is just going backwards.

wiz@'s suggestion (merging libao-sun and libao-oss again into
libao), is a possible solution to have everything working out
of the box again, although I don't like it much.  I can live with it
because it doesn't add extra dependencies on most systems,
but I like to not install oss stuff if possible.

Though... I just had another idea.  In libao's buildlink file, we'd
add an extra dependency of the form libao-*:../../audio/libao-oss
(not sure if the wildcard there could DTRT, but something else
probably will).  That is, a dependency that accepts any plugin
and defaults to oss.  This way, whenever you install an app that
uses libao, libao-oss will be installed iff you hadn't installed any
other plugin by hand before.  This is binary friendly and avoids
options.

Do you think this'd be appropriate?

-- 
Julio M. Merino Vidal <jmmv84%gmail.com@localhost>
http://www.livejournal.com/users/jmmv/
The NetBSD Project - http://www.NetBSD.org/