Opened 4 years ago
Closed 4 years ago
#62738 closed defect (fixed)
gmic 2.9.7.x: OpenCV support no longer enabled by default
Reported by: | mascguy (Christopher Nielsen) | Owned by: | mascguy (Christopher Nielsen) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | Schamschula (Marius Schamschula) | |
Port: | gmic |
Description
It looks like gmic
changed the default build options starting with 2.9.7, such that OpenCV is no longer enabled by default.
The portfile now needs to include the following configure option:
-DENABLE_OPENCV=ON
I'll take care of fixing this, since I'm updating opencv4
to 4.5.2 anyway. That update is covered by the following pull request:
Change History (9)
comment:1 Changed 4 years ago by Schamschula (Marius Schamschula)
comment:2 Changed 4 years ago by mascguy (Christopher Nielsen)
While it sounds like OpenCV isn't used for much, I'd hate to potentially affect any of our users by switching to a variant. Up to you though.
comment:3 Changed 4 years ago by mascguy (Christopher Nielsen)
Pull request created:
https://github.com/macports/macports-ports/pull/10771
We can go with the variant approach if you want, but let's discuss via the PR.
comment:4 Changed 4 years ago by Schamschula (Marius Schamschula)
Understood. However, I'm following upstream.
No need for a PR, I've already updated the Portfile locally and will push it to GitHub shortly.
comment:5 Changed 4 years ago by mascguy (Christopher Nielsen)
Marius, can you include my fix, to eliminate the need to patch the opencv library path?
comment:6 Changed 4 years ago by mascguy (Christopher Nielsen)
Specifically the following:
# insert path to opencv4 libraries, as cmake fails to do so configure.ldflags-append -L${prefix}/lib/opencv4
comment:7 Changed 4 years ago by mascguy (Christopher Nielsen)
Plus enabling parallel builds, which I also included in my PR.
comment:8 Changed 4 years ago by Schamschula (Marius Schamschula)
I'm test building with both of your changes.
comment:9 Changed 4 years ago by Schamschula (Marius Schamschula)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Indeed. That's an upstream change. See: https://discuss.pixls.us/t/on-the-road-to-3-0/17226 ( In the From 2.9.6 to 2.9.7 section)
opencv support should probably be included as a variant.