Page MenuHomeFreeBSD

x11/terminology: Update to 1.4.0
ClosedPublic

Authored by cem on Mar 31 2019, 9:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 12, 12:53 PM
Unknown Object (File)
Sun, Oct 12, 12:26 PM
Unknown Object (File)
Wed, Oct 8, 7:26 AM
Unknown Object (File)
Mon, Oct 6, 2:13 PM
Unknown Object (File)
Sat, Oct 4, 8:11 AM
Unknown Object (File)
Sat, Oct 4, 6:09 AM
Unknown Object (File)
Fri, Oct 3, 9:49 AM
Unknown Object (File)
Tue, Sep 23, 1:32 PM
Subscribers

Diff Detail

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

Event Timeline

portlint asks for INSTALLS_ICONS.

x11/terminology/Makefile
5 ↗(On Diff #55657)

Nit-pick: Porter's Handbook prefers DISTVERSION here.

16 ↗(On Diff #55657)

Nit-pick: This should go after MASTER_SITES.

cem planned changes to this revision.Apr 1 2019, 5:53 AM

Thanks @jrm!

Will fix these preexisting issues at the same time.

cem marked 2 inline comments as done.Apr 1 2019, 5:59 AM
In D19772#423875, @jrm wrote:

portlint asks for INSTALLS_ICONS.

It seems this may be a false positive? INSTALLS_ICONS is for USES= GNOME ports; this port does not use Gtk or GNOME at all.

Fix existing style issues in the port

0mp added a subscriber: 0mp.

LGTM

x11/terminology/Makefile
13 ↗(On Diff #55661)

Nit picking: if there is a LICENSE_FILE somewhere there, I'd add it.

This revision is now accepted and ready to land.Apr 1 2019, 7:29 AM
cem marked an inline comment as done.

Tag COPYING as LICENSE_FILE per 0mp@

This revision now requires review to proceed.Apr 1 2019, 4:44 PM
This revision is now accepted and ready to land.Apr 1 2019, 4:45 PM
This revision was automatically updated to reflect the committed changes.