Bug 233696 - Porter's handbook: Makes clearer the official policy about port signatures
Summary: Porter's handbook: Makes clearer the official policy about port signatures
Status: Closed Overcome By Events
Alias: None
Product: Documentation
Classification: Unclassified
Component: Books & Articles (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Gordon Bergling
URL: https://reviews.freebsd.org/D23589
Keywords: dogfood
Depends on:
Blocks:
 
Reported: 2018-12-01 19:04 UTC by Lorenzo Salvadore
Modified: 2023-08-29 12:25 UTC (History)
2 users (show)

See Also:


Attachments
Makefile signatures patch (892 bytes, patch)
2018-12-01 19:06 UTC, Lorenzo Salvadore
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lorenzo Salvadore freebsd_committer freebsd_triage 2018-12-01 19:04:47 UTC
This bug report is similar to bug #228715, but slightly different.

If I understood it well, old ports are allowed to keep "Created by:..." lines but new ports are not allowed to include one. This is based on my experience.

It still happens that new ports creators try to insert "Created by: ..." lines (as a new FreeBSD contributor, I did it on my first new port less than 6 months ago): the handbook discourages it, but the ports tree includes many "bad" examples. Such examples create confusion and might make new contributors feel bad when seeing that they cannot sign their work while others did (I felt a bit bad at first, before understanding the policy).

Furthermore, if someone interprets this fact as "once signatures were allowed, now they are not any more", one could think that (s)he should remove existing lines from existing ports (like I recently did in bug #233662), which seems to be against the official policy (unless the creator has been asked).

Specifying the correct policy once and for all in the porter's handbook is the best way to avoid similar problem in the future: if what I think to be the official policy is wrong, feel free to correct it.

(My personal opinion is that we should remove all signatures, both old and new: ports change over time and creators ends up with a signature that stays on a Makefile that can be very different to what they wrote when they left maintainership. Credit for creation can still be recognized through other means like Bugzilla or SVN. This policy also has the advantage to be fair between old and new contributors.)
Comment 1 Lorenzo Salvadore freebsd_committer freebsd_triage 2018-12-01 19:06:07 UTC
Created attachment 199719 [details]
Makefile signatures patch

This is the patch I propose for the handbook.
Comment 2 Fernando Apesteguía freebsd_committer freebsd_triage 2023-08-28 07:06:16 UTC
Hi Gordon,

There are no "Created by" lines anymore in ports Makefiles. They have been deleted over time (mainly by tcberner@):

2c54d261a63d7af9bef550ad62b7a47e20c7b05f
48318794ca7c06285786d2d7f4cbe2c1acf08c13
...

etc.

I would like to close this PR.