Bug 242717

Summary: graphics/opencv: mark IGNOREd if MFX option selected on !x86
Product: Ports & Packages Reporter: Mark Linimon <linimon>
Component: Individual Port(s)Assignee: Jan Beich <jbeich>
Status: Closed Overcome By Events    
Severity: Affects Some People CC: pkubaj
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch to graphics/opencv/Makefile none

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.