From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8D3DA197 for ; Fri, 24 Feb 2023 16:55:45 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9D758C4339B; Fri, 24 Feb 2023 16:55:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1677257745; bh=WkAsswgR4+hQzfKBDFB+xhU49P2+yeEhH/3Zu2LBbuw=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ccq6bD1bO3PnEyrOl8qScPAxPBtyoI6VJ2DipjQKQVqOVvREVpfkOCd8I/7C0Cz0S p4Zo0JbTP8VrOHZwGTtPuKCcZrHGvKMpddppxMj4GyT3hPtnXMQ3J5Eux0VKbf8Us/ OraJ8mb5s9GTJUJPU0GH09a/GpgQiHHT4IWdsSDo= Date: Fri, 24 Feb 2023 17:55:41 +0100 From: Greg Kroah-Hartman To: Linus Torvalds Cc: Nathan Chancellor , Guenter Roeck , stable@vger.kernel.org, patches@lists.linux.dev, linux-kernel@vger.kernel.org, akpm@linux-foundation.org, shuah@kernel.org, patches@kernelci.org, lkft-triage@lists.linaro.org, pavel@denx.de, jonathanh@nvidia.com, f.fainelli@gmail.com, sudipm.mukherjee@gmail.com, srw@sladewatkins.net, rwarsow@gmx.de Subject: Re: [PATCH 5.15 00/37] 5.15.96-rc2 review Message-ID: References: <20230223141542.672463796@linuxfoundation.org> Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Fri, Feb 24, 2023 at 11:25:37AM +0100, Greg Kroah-Hartman wrote: > On Thu, Feb 23, 2023 at 01:39:42PM -0800, Linus Torvalds wrote: > > On Thu, Feb 23, 2023 at 1:24 PM Greg Kroah-Hartman > > wrote: > > > > > > Quilt should be depending on patch here, I think it's something in my > > > set of "turn this series of patches into a mbox, split the mbox up into > > > patches" sequence that is causing the problem. > > > > Well, it might also be that quilt just re-generates the patch with > > 'diff', and then in the process loses the information again. > > > > We can happily continue to support the "we lost the executable bit", > > since it sounds like Nathan is willing to cook up a patch. > > > > I'm just too lazy and personally unaffected to care (and too busy with > > merges - excuses, excuses). > > Ok, I figured it out. git is doing the right thing, my scripts are > doing the right thing, patch is doing the right thing, but quilt is > not looking at the permissions on the file at all when refreshing a > patch that has been applied to a tree. > > This might be a configuration setting on my end for quilt, or a bug in > quilt, I'll try to track this down, but in the meantime, I've fixed up > the stable queue of patches and pushed out a -rc3 with this hopefully > all fixed up. Ok, it's not quilt's fault, it's GNU diff's fault from what I can tell. quilt relies on diff to generate the patch, and I can't figure out how to get diff to notice file permissions at all. Am I just missing an option to 'diff' somewhere that I can't find in the manual? What tool originally generated diffs with the file permission settings that patch can read if it wasn't 'diff'? Anyway, quilt can handle replacing what it uses for 'diff', so I'll just replace it with 'git diff' and that seems to solve the problem for me! thanks, greg k-h