Page MenuHomeFreeBSD

New port: multimedia/webcamoid: Full featured webcam suite
ClosedPublic

Authored by yuri on Dec 2 2017, 10:22 PM.
Tags
None
Referenced Files
F81657035: D13348.diff
Fri, Apr 19, 1:44 PM
Unknown Object (File)
Fri, Mar 29, 3:19 PM
Unknown Object (File)
Mar 11 2024, 7:49 AM
Unknown Object (File)
Mar 11 2024, 7:49 AM
Unknown Object (File)
Mar 11 2024, 7:49 AM
Unknown Object (File)
Mar 11 2024, 7:49 AM
Unknown Object (File)
Mar 11 2024, 7:49 AM
Unknown Object (File)
Mar 11 2024, 7:49 AM
Subscribers

Details

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 13293
Build 13527: arc lint + arc unit

Event Timeline

yuri retitled this revision from New port: Full featured webcam suite to New port: multimedia/webcamoid: Full featured webcam suite.Dec 2 2017, 10:22 PM
multimedia/webcamoid/Makefile
18

^ depends twice on the same port

yuri marked an inline comment as done.Dec 2 2017, 10:43 PM

Removed qmake_build, it's a duplicate.

multimedia/webcamoid/pkg-plist
86

this should'n these two files be installed to QT_QMLDIR? (which would be lib/qt5/qml).

yuri marked an inline comment as done.Dec 3 2017, 8:24 AM
yuri added inline comments.
multimedia/webcamoid/pkg-plist
86

That's the upstream bug. I also noticed it, but I didn't think it was reasonable to patch it.

https://github.com/webcamoid/webcamoid/issues/102

yuri marked 2 inline comments as done.Dec 3 2017, 8:24 AM

The patch should be pretty simple to replace the fallback by $$[QT_INSTALL_QML] for INSTALLQMLDIR in commons.pri:

-isEmpty(INSTALLQMLDIR): INSTALLQMLDIR = $${LIBDIR}/qt/qml
+isEmpty(INSTALLQMLDIR): INSTALLQMLDIR = $$[QT_INSTALL_QML]

The patch should be pretty simple to replace the fallback by $$[QT_INSTALL_QML] for INSTALLQMLDIR in commons.pri:

-isEmpty(INSTALLQMLDIR): INSTALLQMLDIR = $${LIBDIR}/qt/qml
+isEmpty(INSTALLQMLDIR): INSTALLQMLDIR = $$[QT_INSTALL_QML]

https://github.com/webcamoid/webcamoid/pull/103

I would add the url to the issue at the top of the two patches -- that way, one directly sees that they have been upstreamed.
Also you could simply add your git-commit as a patch (i..e. one patch for the two current ones). If you call it something like "patch-git_XXXXXX" it is also clearer that

  • this patch can probably be dropped in a future update
  • as it will likely be part of the next release
In D13348#278583, @yuri wrote:

The patch should be pretty simple to replace the fallback by $$[QT_INSTALL_QML] for INSTALLQMLDIR in commons.pri:

-isEmpty(INSTALLQMLDIR): INSTALLQMLDIR = $${LIBDIR}/qt/qml
+isEmpty(INSTALLQMLDIR): INSTALLQMLDIR = $$[QT_INSTALL_QML]

https://github.com/webcamoid/webcamoid/pull/103

The upstream rejected the patch. They said that this is an intended behavior.

Fixed Qml location in a different way.

tcberner added inline comments.
multimedia/webcamoid/Makefile
28

^ you can drop the comment, I think :)

This revision is now accepted and ready to land.Dec 3 2017, 5:50 PM
This revision was automatically updated to reflect the committed changes.