Created attachment 164442 [details]
E17gtk is a dark Gtk theme with sharp corners, which is designed for use in Enlightenment and gives the elegant look of Enlightenment to Gtk3, Gtk2 and Qt widgets. Of course, it can be used with any GTK based environment too, but it is not tuned to Gnome or any of its apps.
Comment on attachment 164442 [details]
New ports dont (yet) have a maintainer to approve
@Matthew, can you confirm this passes QA please? (portlint, poudriere). Logs as attachments are not necessary
Created attachment 164454 [details]
add port x11-themes/gtk-E17-theme
The mention of poudriere testing prompted me to realize this should have NO_ARCH set. I assume it was an oversight that it is not set in the other theme ports I was cribbing from when putting this port together.
/usr/ports/x11-themes/gtk-E17-theme # portlint -AC
WARN: Makefile: Consider defining LICENSE.
0 fatal errors and 1 warning found.
Portlint is happy except for license, which I can do nothing about. There is no mention of license in the distributed files at all. The website simply says "License: GPL", and while LICENSE=GPL makes portlint happy, it causes make to fail. Leaving it blank rather than trying to guess a GPL version.
I guess some clarification is needed regarding the maintainer flag. I was previously told it should be set on ports without a maintainer, i.e. MAINTAINER=ports@FreeBSD.org. A new port also has no current maintainer so logically the same should apply.
Created attachment 164455 [details]
(In reply to matthew from comment #3)
maintainer-approval only applies to existing ports without a maintainer (since by definition a maintainer cant approve the change), and new port doesn't yet have a maintainer, until it's committed
This change is nonetheless good to go.
(In reply to Kubilay Kocak from comment #5)
Since a new port has no maintainer until its committed, by definition there is no maintainer to approve, exactly the same as an existing port without maintainer. I'm still waiting some logical explanation for the difference in handling, and that should include an explanation of why maintainer-approval exists but is not used in such a way to be useful filter criteria. This dead simple theme port sits for weeks. Is that because it doesn't show up in Maintainer Approved saved search that committers with a free moment might look at first?
Matthew has waited too long, and this port doesn't depend on an entire X/Desktop stack (contrary to what I assumed)
A commit references this bug:
Date: Tue Feb 2 17:51:52 UTC 2016
New revision: 407831
[NEW] x11-themes/gtk-E17-theme: Dark GTK theme for Enlightenment
E17gtk is a dark Gtk theme with sharp corners, which is designed for use in
Enlightenment and gives the elegant look of Enlightenment to Gtk3, Gtk2 and
Qt widgets. Of course, it can be used with any GTK based environment too,
but it is not tuned to Gnome or any of its apps.
Submitted by: Matthew <matthew reztek cz>
Committed, apologies for the delay Matthew