Bug 242595 - [patch] multimedia/qt5-multimedia: fix unconditional openal dependency despite OPTIONS
Summary: [patch] multimedia/qt5-multimedia: fix unconditional openal dependency despit...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: freebsd-kde (Team)
Depends on:
Reported: 2019-12-12 11:41 UTC by Piotr Smyrak
Modified: 2020-01-01 18:56 UTC (History)
2 users (show)

See Also:
tcberner: maintainer-feedback+

Makefile patch (592 bytes, patch)
2019-12-12 11:41 UTC, Piotr Smyrak
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Piotr Smyrak 2019-12-12 11:41:01 UTC
Created attachment 209889 [details]
Makefile patch

The multimedia/qt5-multimedia port has an OPTIONS knob to build with or without openal dependency, but the option is bogus, since Makefile does not take it into account and hardcodes unconditional dependency via USES.

The attached patch fixes the problem
Comment 1 Tobias C. Berner freebsd_committer 2019-12-13 17:35:11 UTC
Thanks for the patch, that sounds reasonable.
Comment 2 Adriaan de Groot freebsd_committer 2019-12-31 14:04:00 UTC
 - I expect that this needs a PORTREVISION bump.
 - I think OPENAL should be in OPTIONS_DEFAULT so that the change to the package is as small (and unsurprising) as possible. Yes, the option is default off right now, but the feature (openal) is on in all existing packages (because the option is broken).
Comment 3 commit-hook freebsd_committer 2020-01-01 00:01:54 UTC
A commit references this bug:

Author: adridg
Date: Wed Jan  1 00:01:43 UTC 2020
New revision: 521666
URL: https://svnweb.freebsd.org/changeset/ports/521666

  Fix OPENAL knob in multimedia/qt5-multimedia

  The knob was already there, but didn't have any machinery attached.

   - Move USES=openal to the knob
   - Introduce a patch to actually make openal configurable
     (instead of only following autodetect, so you can still
     turn the knob off with OpenAL installed)
   - Tidy the port a little
   - Add OPENAL to default options.

  Previously, setting or unsetting OPENAL had no effect, and OpenAL
  was always a dependency and used. Now, it does have an effect.
  I've made it default so that the default options still match
  what the port previously did (that is, depend on OpenAL).
  Bump PORTREVISION because of that.

  (Based very loosely on a patch submitted by Piotr Smyrak)

  PR:		242595
  Reported by:	Piotr Smyrak