On Sat,  3 Dec 2016 12:51:14 +0100
Mattias Andrée <[email protected]> wrote:

Hey Mattias,

> The -s flag previously called strip(1) on the installed file.
> This patch changes install(1)'s behaviour to ignore -s.
> 
> Many makefiles use the -s flag, so it has to be recognised for
> compatibility, however it does not have to do anything because
> symbols do not negatively affect the functionallity of binaries.
> 
> Ignoring -s have the added benefit that the user do not need
> to edit makefiles if they want the symbols that are useful for
> debugging. If the user wants to strip away symbols, it can be
> done manually or automatically by the package manager.

thanks, applied! I favor just not documenting the s-flag in xinstall.1,
but understand both sides of the argument. We have many examples in
sbase, like sort -m, but also as we can see install -c and others.

Maybe we can discuss this here. Another alternative would be to put
that in a NOTES section. I really don't want to see the manpages filled
up with ignored options, making them harder to read. On the other hand,
a manpage should reflect the code and no matter how we put it, we do
handle the c and s flags specially here.

Cheers

Laslo

-- 
Laslo Hunhold <[email protected]>

Reply via email to