Hi Marc,

Marc Espie wrote on Sat, Feb 09, 2019 at 10:03:20PM +0100:

> hey, your commit to install(1) broke something.
> 
> Specifically lang/go-boostrap now produces a broken package which can't
> be used to build go.
> 
> All the go/bootstrap/pkg/tool/openbsd_amd64/*
> have lost their x bit
> 
> Relevant fake install information, it definitely looks like the last line
> is now a no-op.
[...]
> install -c -m 755 -p [...]

Sorry for the disruption.

It looks like the bug was not caused, but merely exposed by my commit.
I reverted my commit anyway such than we can first fix the bug
without a hurry, then reapply my change.

I did not revert the manual page change to minimize churn, expecting
that the change will eventually be put back.


A preliminary analysis indicates that

1. install       -m 755 foo bar   always applies 755 to bar
2. install -S    -m 755 foo bar   always applies 755 to bar
3. install -S -p -m 755 foo bar   only applies 755 if the file content changes

Item 3 looks like a bug to me.  Similar effects may or may not apply
to the owner and the flags.  I will look in more detail later.

Yours,
  Ingo

Reply via email to