Created attachment 224954 [details] patch for x11-themes/kde-icons-black-and-white This port effectively has no LICENSE. It also has no MAINTAINER. I already maintain at least one of this set and want to improve these ports. As they all share a common file which I want to modify x11-themes/kde-icons-noia/Makefile.icons So I'm just going to take MAINTAINER for all those (unmaintained) so as to add an actual LICENSE to those that come with one, and add an appropriate LICENSE && TEXT for those that don't. Changes Makefile (LICENSE, LICENSE_NAME, LICENSE_TEXT, LICENSE_PERMS, MAINTAINER) That's it. Builds/Works as intended on 12/amd64 13/amd64 Thanks! :-) --Chris
It already inherits all the LICENSE* from ../kde-icons-noia/Makefile.icons
Basically I don't understand what you want to change in Makefile.icons and why. It might be better to submit a big change once to see the bigger picture.
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=b309895b3544ffba9e8df8786062ec6013c752ff commit b309895b3544ffba9e8df8786062ec6013c752ff Author: Chris Hutchinson <portmaster@bsdforge.com> AuthorDate: 2021-05-17 06:32:36 +0000 Commit: Nuno Teixeira <eduardo@FreeBSD.org> CommitDate: 2021-05-17 06:32:36 +0000 x11-themes/kde-icons-black-and-white: add LICENSE, take MAINTAINER - Add LICENSE - Submitter becomes maintainer PR: 255891 Approved by: dbaio, garga (mentors, implicit) x11-themes/kde-icons-black-and-white/Makefile | 8 ++++++++ 1 file changed, 8 insertions(+)
Hello Chis! Thanks for adopting this port. Committed!
(In reply to Tobias Kortkamp from comment #2) Hello! Sorry I have comitted it because I didn't see this messages. Should I revert?
(In reply to Nuno Teixeira from comment #5) I'll make a "mega patch" today. As Tobias suggests. That incorporates all the icon ports that depend on x11-themes/kde-icons-noia/Makefile.icons and modify x11-themes/kde-icons-noia/Makefile.icons accordingly. Thanks for your attention to this, Nuno. --Chris
(In reply to Chris Hutchinson from comment #6) To be clear; I won't include this one. As it's already done. :-)
(In reply to Chris Hutchinson from comment #7) See bug #255846 for the MEGA DIFF I'm talking about. :-)
Triage: * patch and patch-ready keywords are deprecated. (The dashboard showed this ports bug as ready to be taken by a committer.)
To submitter: is there anything further that needs to be done with this PR?
^Triage: assign to committer.
What's the status of this PR?
(In reply to Nuno Teixeira from comment #12) It appears that nothing but discussion has happened since 2021-05-17 06:32:36 +0000. If I'm understanding this correctly, I'll just close it as bugmeister.