On Mon, Oct 18, 2021 at 4:57 AM Gregory Nutt wrote:
>
> Wow, that change has been in the Makefile for 20 years.
>
> Here is the original bugfix:
> https://github.com/chexum/genromfs/commit/6d1b1e46bb7dbc25ca521c5dfcdc37d17dca8615
>
> The bug was:
>
>     -mkdir -p $(PREFIX)$(mandir)
>     +mkdir -p $(PREFIX)$(mandir)/man8
>      install -m 644 genromfs.8 $(PREFIX)$(mandir)/man8/
>
> Which installed the manpage in a regular file called man8 (the use of
> bindir instead of mandir was also a bug it should have done nothing).
>
> And this one makes the code look like it does today:
> https://github.com/chexum/genromfs/commit/427b121e23cd21d9b9ddb6caee3eb1064369358d
>
> Which looks wrong to me:  Why does the change use bindir?  Must be some
> other issue that I can't see???

Probably a "fix" for bad install by 0.5 :-)

https://github.com/chexum/genromfs/blob/4fdb9d00df13dfe52c28e8af058d38ce8bea89a1/Makefile

install-man:
    mkdir -p $(PREFIX)/$(bindir)
    install -m 644 genromfs.8 $(PREFIX)/$(bindir)/man8

I would keep that file instead removing anything from $envvar /
$clivar / usr as root, this is part of base OS, messing with /usr is
wrong, this place is sacred, /usr/local is for local packages :-)

Also prefix and PREFIX can be provided as command line and environment
parameter so its not a constant location, is user controlled, and may
differ between installations :-)

-- 
CeDeROM, SQ7MHZ, http://www.tomek.cedro.info

Reply via email to