Bug 242717 - graphics/opencv: mark IGNOREd if MFX option selected on !x86
Summary: graphics/opencv: mark IGNOREd if MFX option selected on !x86
Status: Closed Overcome By Events
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Jan Beich
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-12-19 02:05 UTC by Mark Linimon
Modified: 2021-01-30 18:34 UTC (History)
1 user (show)

See Also:

Attachments
patch to graphics/opencv/Makefile (503 bytes, patch)
2019-12-19 02:05 UTC, Mark Linimon 		no flags Details | Diff
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 Mark Linimon freebsd_committer freebsd_triage 2019-12-19 02:05:47 UTC 
Created attachment 210050 [details]
patch to graphics/opencv/Makefile

ports r520396 added option MFX.  This depends on multimedia/intel-media-sdk which is only avaiable on x86.

I do not know enough about group OPTIONS to construct a patch to exclude it based on ARCH.  This patch merely warns the user that they have selected something that is not supported.  This is merely a foot-shooting prevention measure.
Comment 1 Mark Linimon freebsd_committer freebsd_triage 2019-12-19 02:06:58 UTC 
Over to committer of r520396 (the port has no maintainer).
Comment 2 Jan Beich freebsd_committer freebsd_triage 2019-12-19 02:28:34 UTC 
See also ports r520395. Keeping track of ONLY_FOR_ARCHS changes in non-default dependencies doesn't scale and is error-prone. Only truly ARCH-specific options (e.g., SSE, NEON) makes sense to track in current port.

OpenCV itself doesn't limit MFX to x86, so if something else provides the same API it maybe possible to build. For example, VAAPI was initially x86-only.
Comment 3 Jan Beich freebsd_committer freebsd_triage 2019-12-19 02:32:34 UTC 
Ideally, the framework should check ONLY_FOR_ARCHS, NOT_FOR_ARCHS, BROKEN, IGNORE, etc. in dependencies before starting build.
Comment 4 Jan Beich freebsd_committer freebsd_triage 2021-01-30 18:34:31 UTC 
MFX is gone after ports r563430.