Page MenuHomeFreeBSD

Update Qt to 5.6.2
ClosedPublic

Authored by adridg on Oct 12 2016, 5:13 PM.
Tags
None
Referenced Files
F80193555: D8228.id21385.diff
Fri, Mar 29, 1:46 AM
F80165858: D8228.id21311.diff
Thu, Mar 28, 7:26 PM
Unknown Object (File)
Sun, Mar 10, 9:48 AM
Unknown Object (File)
Sun, Mar 10, 9:48 AM
Unknown Object (File)
Sun, Mar 10, 9:48 AM
Unknown Object (File)
Sun, Mar 10, 9:48 AM
Unknown Object (File)
Sun, Mar 10, 9:48 AM
Unknown Object (File)
Thu, Mar 7, 4:00 AM
Subscribers

Details

Reviewers
tcberner
emaste
rakuco
mat
Group Reviewers
O5: Ports Framework(Owns No Changed Paths)
portmgr
Commits
rP424842: Update Qt to 5.6.2 [1,2]
Summary

This updates Qt 5.6.1 to Qt 5.6.2 (both of them LTS). Original work by tcberner, adjusted to Qt 5.6.2 final by me.

Test Plan

Needs build of devel/qt5, devel/py-qt5 to get started; after that probably one or more of the bigger Qt5 consumers (qupzilla in particular).

Diff Detail

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

Event Timeline

adridg retitled this revision from to Update Qt to 5.6.2.
adridg updated this object.
adridg edited the test plan for this revision. (Show Details)
adridg added reviewers: rakuco, tcberner, mat.
adridg set the repository for this revision to rP FreeBSD ports repository.
adridg added a subscriber: kde.
devel/qmake5/Makefile
55

I think this is no longer needed. mkspecs/unsupported/freebsd-g++46 no longer exists.

misc/qt5-examples/Makefile
7

this should again be changed to official_releases =)

If you can't use arcanist for some reason to upload your diffs, could you at least upload them the same way it does, with a lot of context (svn diff --diff-cmd=diff -x -U9999 or git diff -U9999)

adridg added inline comments.
misc/qt5-examples/Makefile
7

Also QT5_SUBDIR is not defined anywhere, so I've removed that.

adridg marked an inline comment as done.
adridg edited edge metadata.

Update Qt to 5.6.2 (v2)

  • Addressed tcberner@'s comments
  • This revision done with arc, per mat@'s comment
misc/qt5-examples/Makefile
7–8

There seem to be spaces instead of tabs between on the MASTER_SITES= line.

adridg edited edge metadata.

Update Qt to 5.6.2 (v3)

Addressed tabs-vs-spaces in qt5-examples/Makefile

tcberner edited edge metadata.

The only thing I don't understand is why devel/qmake5/files/patch-mkspecs__features__qt_functions.prf was removed, as the code we were patching is still present upstream.

adridg edited edge metadata.

Update Qt to 5.6.2 (v4)

  • Removed a now-empty directory
  • Restored patch Rakuco pointed out as missing (thanks tcberner@)

Mark things as done that were resolved by v3 and v4 patches.

tcberner edited edge metadata.
rakuco edited edge metadata.

looks good to me too. the next step is filing an exp-run request to test the patch

rP424842 says

Still Importing...
This commit is still importing. Changes will be visible once the import finishes.

rP424842 says

Still Importing...
This commit is still importing. Changes will be visible once the import finishes.

@emaste : Should we simply close it now? Or is there anything else to do?

Yes you can just close this review. The PR will track the Phabricator importer issue.

Hm, it refuses to be closed

You can not close this revision because it has not been accepted. You can only close accepted revisions.
emaste added a reviewer: emaste.

Let me try accepting again

Oh, I think it needs the 'blocking reviewers' to accept

mat edited edge metadata.

That's my cue.

This revision is now accepted and ready to land.Nov 3 2016, 3:43 PM