Bug 266068 - portlint: Prepare for WWW in Makefile instead of pkg-descr
Summary: portlint: Prepare for WWW in Makefile instead of pkg-descr
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Joe Marcus Clarke
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-08-27 13:52 UTC by Stefan Eßer
Modified: 2022-09-27 17:40 UTC (History)
1 user (show)

See Also:


Attachments
Check for WWW in Makefile, reject it in pkg-descr (4.07 KB, patch)
2022-08-27 13:52 UTC, Stefan Eßer
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Eßer freebsd_committer freebsd_triage 2022-08-27 13:52:50 UTC
Created attachment 236149 [details]
Check for WWW in Makefile, reject it in pkg-descr

There have been multople requests to move the project URL from the last line of the pkg-descr file into the Makefile (see review D2949 and review D34979, and the proposed documentation change review D36369).

In coordination with PortMgr I'm currently working on this change, and one of the pre-requisites is that portlint is adapted to this change.

Please see the attached patch.

This patch should not be committed before all the other preparations have been completed (e.g. documentation changes) and the sweeping update of all ports can be committed.

Please let me know which time constraints there are with regard to an update of the portlint port, to allow it to be committed in close sequence to the update of the port Makefiles and pkg-descr files.

If you approve this patch, I could commit it together with the move of WWW to the port Makefiles - this might reduce the time window where portlint does not match the new convention of WWW= in the Makefile.
Comment 1 Joe Marcus Clarke freebsd_committer freebsd_triage 2022-09-06 14:24:58 UTC
Hey, Stefan.  I have two changes in my private git repo ready to go.  I can add this as well and commit when you're ready.
Comment 2 Stefan Eßer freebsd_committer freebsd_triage 2022-09-06 15:15:40 UTC
(In reply to Joe Marcus Clarke from comment #1)

Hi Joe,

I'm technically ready to go - portlint has been the biggest issue since it would flag the new style (WWW in Makefile instead of pkg-descr file) as error.

I do fully agree that the time window between the portlint update and the implementation of the planned change of all Makefiles should be as short as possible.

The patch provided with this PR does flag deviations from the new configuration as warnings, not as error, for this reason. (A later change to an error might be appropriate for some of the tests.)

I'll discuss this conversion with members of the PortMgr team later today and I'd appreciate if you'd update portlint at an agreed upon time within the next 48 hours.

The best time of day (low commit frequency) for the large commit to >30000 each Makefiles and pkg-descr files seems to be between 2300 and 0700 UTC (corresponding to late afternoon to night in the US?). But feel free to suggest any time that is appropriate for you for the portlint commit - I'll then try to push the commit to the ports tree with as little delay as possible.

Thank you for your support!
Comment 3 Joe Marcus Clarke freebsd_committer freebsd_triage 2022-09-06 15:29:46 UTC
I'm good to go when you are.  You just tell me when to pull the trigger.
Comment 4 Stefan Eßer freebsd_committer freebsd_triage 2022-09-07 06:20:00 UTC
(In reply to Joe Marcus Clarke from comment #3)

Please just commit the portlint update at any time within the next 16 hours, if possible (between now and 2022-09-07 2200 UTC).

And please notify me by mail to se@freebsd.org about that commit - either after pushing it or (preferred) some time in advance.
Comment 5 Joe Marcus Clarke freebsd_committer freebsd_triage 2022-09-27 17:40:24 UTC
This has been done.