Page MenuHomeFreeBSD
Differential D14660

deskutils/znotes: Add Qt5 OPTION and support
ClosedPublic
Actions

Authored by rigoletto on Mar 11 2018, 9:47 PM.
Tags
None
Referenced Files
F141186587: D14660.id40275.diff
Fri, Jan 2, 2:04 AM
Unknown Object (File)
Thu, Dec 11, 1:52 AM
Unknown Object (File)
Tue, Dec 9, 8:51 AM
Unknown Object (File)
Sat, Dec 6, 1:52 PM
Unknown Object (File)
Nov 15 2025, 1:28 AM
Unknown Object (File)
Nov 14 2025, 9:59 PM
Unknown Object (File)
Nov 14 2025, 9:32 PM
Unknown Object (File)
Nov 14 2025, 4:57 PM
View All Files
Subscribers
mat
Contributor Reviewers (ports)

Details

Reviewers
tcberner
adridg
mat
Commits
rP468608: Flavorize deskutils/znotes
Summary
  • adopt it
  • add Qt5 support

Diff Detail

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

Event Timeline

rigoletto created this revision.Mar 11 2018, 9:47 PM
Herald added subscribers: Contributor Reviewers (ports), mat. · View Herald TranscriptMar 11 2018, 9:47 PM
Harbormaster completed remote builds in B15505: Diff 40182.Mar 11 2018, 9:47 PM
rigoletto added reviewers: tcberner, adridg.Mar 11 2018, 9:48 PM
tcberner added inline comments.Mar 12 2018, 5:54 AM
deskutils/znotes/Makefile
27

how about flavors?

tcberner added inline comments.Mar 12 2018, 6:18 AM
deskutils/znotes/Makefile
4–5

instead of bumping you should maybe change the version number to the something like
0.4.5.20150825

19–30

^ is this really needed? It should find LRELEASE on its own.

tcberner added inline comments.Mar 12 2018, 6:22 AM
deskutils/znotes/Makefile
4–5

See:
https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/makefile-distfiles.html#makefile-master_sites-github-ex2

tcberner added inline comments.Mar 12 2018, 7:06 AM
deskutils/znotes/Makefile
35–36

^ propbably you want either to get rid of the NLS option, or move the linguist* deps to NLS_USE=QT[45]=...

mat added inline comments.Mar 12 2018, 10:23 AM
deskutils/znotes/Makefile
4–5

it's not a question of "should maybe change" it is a question of must. The upstream version is not the same, so the port version must be updated.

27

Yes, please.

rigoletto updated this revision to Diff 40217.Mar 12 2018, 8:10 PM
rigoletto marked 7 inline comments as done.

Done. :-)

Harbormaster completed remote builds in B15523: Diff 40217.Mar 12 2018, 8:10 PM
rigoletto added a comment.Mar 13 2018, 1:16 AM

Should I set Qt5 as default FLAVOR?

Thanks!

mat added a comment.Mar 13 2018, 10:50 AM

This looks good. We currently do not accept flavors that add new packages though, so this change will be able to go in when ports-mgmt/poudriere is updated to 3.3.

rigoletto updated this revision to Diff 40275.Mar 14 2018, 4:55 AM

Updating new maintainer (me) e-mail address.

Thanks! :-D

Harbormaster completed remote builds in B15558: Diff 40275.Mar 14 2018, 4:55 AM
tcberner added a comment.Mar 14 2018, 5:56 AM

I guess I will commit this after poudriere 3.3 has landed then, ok?

rigoletto added a comment.Mar 14 2018, 5:01 PM
In D14660#308589, @tcberner wrote:

I guess I will commit this after poudriere 3.3 has landed then, ok?

Fine. 😄

mat accepted this revision.Apr 18 2018, 11:12 AM

accept as portmgr for the flavors part.

This revision is now accepted and ready to land.Apr 18 2018, 11:12 AM
Closed by commit rP468608: Flavorize deskutils/znotes (authored by tcberner). · Explain WhyApr 29 2018, 7:45 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
deskutils/
znotes/
32 lines
5 lines
17 lines

Diff 40217

View Options

deskutils/znotes/Makefile

Loading...
View Options

deskutils/znotes/distinfo

Loading...
View Options

deskutils/znotes/files/

Loading...
View Options

deskutils/znotes/files/patch-single__inst_qtlocalpeer.cpp

Loading...
View Options

deskutils/znotes/pkg-plist

Loading...