Page MenuHomeFreeBSD
Differential D19772

x11/terminology: Update to 1.4.0
ClosedPublic
Actions

Authored by cem on Mar 31 2019, 9:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 20, 2:48 PM
Unknown Object (File)
Mar 20 2024, 9:06 PM
Unknown Object (File)
Mar 20 2024, 9:02 PM
Unknown Object (File)
Dec 22 2023, 10:57 PM
Unknown Object (File)
Nov 21 2023, 10:10 AM
Unknown Object (File)
Nov 8 2023, 1:12 PM
Unknown Object (File)
Nov 5 2023, 10:35 PM
Unknown Object (File)
Nov 5 2023, 6:25 AM
View All 36 Files
Subscribers
0mp
jrm
mat

Details

Reviewers
gblach
0mp
jrm
Group Reviewers
Ports Committers
Commits
rP497493: x11/terminology: Update to 1.4.0

Diff Detail

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

Event Timeline

cem created this revision.Mar 31 2019, 9:21 PM
Herald added a subscriber: mat. · View Herald TranscriptMar 31 2019, 9:21 PM
Harbormaster completed remote builds in B23426: Diff 55657.Mar 31 2019, 9:21 PM
cem added a reviewer: Ports Committers.Mar 31 2019, 9:21 PM
jrm added a subscriber: jrm.Mar 31 2019, 9:30 PM

portlint asks for INSTALLS_ICONS.

x11/terminology/Makefile
5

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

17–18

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.

cem updated this revision to Diff 55661.Apr 1 2019, 6:06 AM

Fix existing style issues in the port

Harbormaster completed remote builds in B23429: Diff 55661.Apr 1 2019, 6:06 AM
0mp accepted this revision.Apr 1 2019, 7:29 AM
0mp added a subscriber: 0mp.

LGTM

x11/terminology/Makefile
13–14

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
jrm accepted this revision.Apr 1 2019, 11:39 AM

lgtm

cem updated this revision to Diff 55679.Apr 1 2019, 4:44 PM
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
Harbormaster completed remote builds in B23441: Diff 55679.Apr 1 2019, 4:44 PM
jrm accepted this revision as: jrm.Apr 1 2019, 4:45 PM
This revision is now accepted and ready to land.Apr 1 2019, 4:45 PM
Closed by commit rP497493: x11/terminology: Update to 1.4.0 (authored by cem). · Explain WhyApr 1 2019, 4:52 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
x11/
terminology/
5 lines
6 lines

Diff 55679

View Options

x11/terminology/Makefile

Loading...
View Options

x11/terminology/distinfo

Loading...