Bug 277603 - Mk: Move man to share/man
Summary: Mk: Move man to share/man
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Ports Framework (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Port Management Team
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-03-09 21:28 UTC by Muhammad Moinur Rahman
Modified: 2024-04-12 15:52 UTC (History)
2 users (show)

See Also:
bofh: merge-quarterly-
arrowd: exp-run?


Attachments
Change PREFIX/man to PREFIX/share/man (5.38 KB, patch)
2024-03-09 21:28 UTC, Muhammad Moinur Rahman
no flags Details | Diff
Move man to share/man and fix mtree (5.79 KB, patch)
2024-03-10 10:16 UTC, Muhammad Moinur Rahman
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Muhammad Moinur Rahman freebsd_committer freebsd_triage 2024-03-09 21:28:59 UTC
Created attachment 249058 [details]
Change PREFIX/man to PREFIX/share/man

This is the final patch that should go through an exp-run as this involves mass changes in Templates/BSD.local.dist.

The exp-run should be initiated after all the ports have been converted or at least a week before the 14.1-RELEASE build starts and should be committed before the release of 14.1-RELEASE.

Current list of failed and skipped ports are available here:
https://people.freebsd.org/~bofh/dropzone/manprefix-fail.maintainer.txt

This file is regularly updated however this update will stop on 13th March as I will need to dedicate my builder on some other build works.
Comment 1 Antoine Brodin freebsd_committer freebsd_triage 2024-03-09 22:52:09 UTC
Templates/BSD.local.dist does not look good,  both before and after this patch.

1) there are some unneeded uname=root and /set uname=root (unneeed because the owner is already root)

2) since there is no catman anymore and man is no longer setuid,  are all the cat directories actually used?
Comment 2 Muhammad Moinur Rahman freebsd_committer freebsd_triage 2024-03-10 10:16:30 UTC
Created attachment 249065 [details]
Move man to share/man and fix mtree

Hi Antoine,

I just remembered our discussion and have fixed the issues with the uname.

I have only added the cat directories which were required during my testing and moving man pages. So these specific directories of cat are actually required in share/man.
Comment 3 Gleb Popov freebsd_committer freebsd_triage 2024-03-28 19:25:34 UTC
Autotools ports seem to be dealt with. However, I'd not request exp-run yet but rather start fixing meson ones to reduce the total amount of exp-runs needed.
Comment 4 Gleb Popov freebsd_committer freebsd_triage 2024-03-29 07:27:27 UTC
Apparently, meson doesn't need to be fixed, as it already correctly defaults to --mandir=share/man

I'd say, this patch is now ready for an exp-run.
Comment 5 Muhammad Moinur Rahman freebsd_committer freebsd_triage 2024-03-29 09:06:46 UTC
@arrowd I have stopped updating the file at freefall.

And thanks for the great work. Hopefully the exp-run will be uneventful.
Comment 6 Antoine Brodin freebsd_committer freebsd_triage 2024-03-29 09:42:13 UTC
Before the exp-run,  can you check if those ports need fixes?

databases/xrootd/pkg-plist
devel/papi/pkg-plist
devel/dcd-client/pkg-plist
devel/gearmand/pkg-plist
dns/dnsmasq-devel/pkg-plist
lang/icc/files/exclude
sysutils/e2fsprogs-core/files/unwanted
sysutils/desktop-installer/pkg-plist
sysutils/bacula15-server/pkg-plist
sysutils/bacula15-server/pkg-plist.client
x11-wm/emwm-utils/pkg-plist
x11-wm/emwm/pkg-plist
Comment 7 Gleb Popov freebsd_committer freebsd_triage 2024-04-04 17:35:14 UTC
(In reply to Antoine Brodin from comment #6)
The list has been dealt with.

Before launching an exp-run, am I understanding it right that it will highlight all ports that install manpages to the old location (${PREFIX}/man) ?
Comment 8 Muhammad Moinur Rahman freebsd_committer freebsd_triage 2024-04-04 17:40:17 UTC
(In reply to Gleb Popov from comment #7)
Yes it should. This is the same patch that I used to run my mini exp-runs. The exact logics that actually detects the failures are actually not int the patch as it just iterates over the MANDIRS variables. As we are removing the ${PREFIX}/man from the MANDIRS so what will happen is that no man pages will be actually be gzipped anymore if they are not in PREFIX/share/man and eventually will fail with pkg-plist errors.
Comment 9 Gleb Popov freebsd_committer freebsd_triage 2024-04-08 15:12:02 UTC
I just figured I haven't actually requested an exp-run.
Comment 12 Gleb Popov freebsd_committer freebsd_triage 2024-04-11 16:08:57 UTC
The fallout has been dealt with. Moin, would you push this yourself?
Comment 13 Muhammad Moinur Rahman freebsd_committer freebsd_triage 2024-04-11 16:10:59 UTC
Not at all. :P

Shoot at will please. :)
Comment 14 Muhammad Moinur Rahman freebsd_committer freebsd_triage 2024-04-11 16:20:01 UTC
On another contrary arrowd@ can you discuss with portmgr@ whether if this is actually a right time for this push or not? We have just split quarterly and some stupid people like me will be removing the share/man hacks and in case those needs to be merged quarterly it might break things. So waiting and doing this sometimes near the next quarterly makes sense to me. But yeah I leave it upon portmgr@ to decide and work on this.
Comment 15 commit-hook freebsd_committer freebsd_triage 2024-04-11 18:55:07 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=9fd7ee1cdf6cbd5528159b58a966e1a40e2909a9

commit 9fd7ee1cdf6cbd5528159b58a966e1a40e2909a9
Author:     Muhammad Moinur Rahman <bofh@freebsd.org>
AuthorDate: 2024-04-11 12:32:07 +0000
Commit:     Gleb Popov <arrowd@FreeBSD.org>
CommitDate: 2024-04-11 18:51:09 +0000

    Framework: Finalize switch to share/man for manpages

    PR:             277603
    Approved by:    portmgr

 Mk/bsd.port.mk           |   6 +-
 Templates/BSD.local.dist | 226 +----------------------------------------------
 2 files changed, 4 insertions(+), 228 deletions(-)
Comment 16 Gleb Popov freebsd_committer freebsd_triage 2024-04-11 18:55:39 UTC
Pushed, thanks everyone.