Re: [PATCH 5.10 00/79] 5.10.151-rc1 review

[Florian Fainelli]

On 10/27/22 12:39, Guenter Roeck wrote:
> On 10/27/22 12:27, Pavel Machek wrote:
> > On Thu 2022-10-27 21:25:54, Greg Kroah-Hartman wrote:
> > > On Thu, Oct 27, 2022 at 11:10:18AM -0700, Guenter Roeck wrote:
> > > > On 10/27/22 09:55, Greg Kroah-Hartman wrote:
> > > > > This is the start of the stable review cycle for the 5.10.151 release.
> > > > > There are 79 patches in this series, all will be posted as a response
> > > > > to this one.  If anyone has any issues with these being applied, 
> > > > > please
> > > > > let me know.
> > > > >
> > > > > Responses should be made by Sat, 29 Oct 2022 16:50:35 +0000.
> > > > > Anything received after that time might be too late.
> > > >
> > > > Building arm64:allmodconfig ... failed
> > > > --------------
> > > > Error log:
> > > > /bin/sh: scripts/pahole-flags.sh: Permission denied
> > > >
> > > > Indeed:
> > > >
> > > > $ ls -l scripts/pahole-flags.sh
> > > > -rw-rw-r-- 1 groeck groeck 530 Oct 27 11:08 scripts/pahole-flags.sh
> > > >
> > > > Compared to upstream:
> > > >
> > > > -rwxrwxr-x 1 groeck groeck 585 Oct 20 11:31 scripts/pahole-flags.sh
> > >
> > > Yeah, this is going to be an odd one.  I have to do this by hand as
> > > quilt and git quilt-import doesn't like setting the mode bit.
> > >
> > > I wonder if I should just make a single-commit release with this file in
> > > it, set to the proper permission, to get past this hurdle.  I'll think
> > > about it in the morning...
> >
> > Alternatively you can modify the caller to do /bin/sh /scripts/... so
> > it does not need a +x bit...
>
> That should be done in mainline, though.

This is the second time this is reported unfortunately, so while we 
could change things in mainline to avoid being dependent upon the file 
permissions stored in git, this really seems to be a workflow issue 
involving quilt.

Any chance you can run a fixup while you apply a patch Greg?
--
Florian