Page MenuHomeFreeBSD

Qt5: Drop post-configure target from most QT_DIST ports
ClosedPublic

Authored by rakuco on Dec 24 2016, 5:55 PM.
Tags
None
Referenced Files
F81681655: D8899.id.diff
Fri, Apr 19, 8:59 PM
F81679081: D8899.diff
Fri, Apr 19, 8:00 PM
Unknown Object (File)
Sat, Apr 6, 4:53 AM
Unknown Object (File)
Sat, Apr 6, 4:53 AM
Unknown Object (File)
Sat, Apr 6, 4:53 AM
Unknown Object (File)
Sat, Apr 6, 4:51 AM
Unknown Object (File)
Thu, Mar 28, 8:40 AM
Unknown Object (File)
Mar 12 2024, 11:39 AM
Subscribers

Details

Summary

Qt's configure script already creates all the necessary Makefiles for the directories we need to build, so there is no need to run qmake again in post-configure.

Test Plan

Tested the ports on Poudriere (9.3-i386 and 12-amd64) and everything worked fine.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

rakuco retitled this revision from to Qt5: Drop post-configure target from most QT_DIST ports.
rakuco updated this object.
rakuco edited the test plan for this revision. (Show Details)
rakuco added reviewers: kde, tcberner, adridg.
rakuco set the repository for this revision to rP FreeBSD ports repository.

Tested qt5-network on 10.3-x64. It's ok; there are some stage-qa warnings, but those aren't related to this review.

Tested qt5-printsupport on 10.3-x64, with and without option CUPS. It's ok; same stage-qa warnings.

adridg edited edge metadata.

Tested qt5-buildtools on 10.3-x64 and dbus-qt5 on 10.3-x64, they're OK.

This revision is now accepted and ready to land.Dec 24 2016, 8:27 PM
tcberner edited edge metadata.

There seems to be a difference in the Makefiles with and without the target (for example in dbus-qt5's src/tools/qdbusxml2cpp) -- but it still seems to work as expected.

This revision was automatically updated to reflect the committed changes.