Re: [PATCH] checkkconfigsymbols.py: make it Git aware

From: Valentin Rothberg <valentinrothberg@gmail.com>
To: Paul Bolle <pebolle@tiscali.nl>
Cc: Greg KH <gregkh@linuxfoundation.org>,
	hengelein Stefan <stefan.hengelein@fau.de>,
	linux-kernel@vger.kernel.org,
	Andreas Ruprecht <rupran@einserver.de>
Subject: Re: [PATCH] checkkconfigsymbols.py: make it Git aware
Date: Wed, 11 Mar 2015 15:33:46 +0100	[thread overview]
Message-ID: <CAD3Xx4Kp8griLJrR3H7G2AU0_YZeV-xvrdnxV-dTr=YuDmop9w@mail.gmail.com> (raw)
In-Reply-To: <CAD3Xx4LR8pUQnFp-h5BRG08Ujkj5hKzF2G9TJRD5-C+jaNsPTQ@mail.gmail.com>

On Wed, Mar 11, 2015 at 3:19 PM, Valentin Rothberg
<valentinrothberg@gmail.com> wrote:
> On Wed, Mar 11, 2015 at 1:04 PM, Paul Bolle <pebolle@tiscali.nl> wrote:
>> On Wed, 2015-03-11 at 12:16 +0100, Valentin Rothberg wrote:
>>> Note that both options require to 'git reset --hard' the user's Git
>>> tree, which can lead to the loss of uncommitted data.
>>
>> My local "800 line perl monster" basically does
>>     git ls-tree -r $commit_or_tag
>>
>> which allows you to generate a list of files and their corresponding
>> hashes (it also helps with filtering out symlinks by the way).
>>
>> The you can do
>>     git cat-file blob $commit:$path
>>
>> or just
>>     git cat-file blob $hash
>>
>> for every file you're interested in and parse the output of that file
>> (in memory, as it were). None of that messes with the current state of
>> the tree you're working on.
>
> That's a good point.  I used cat-file once in another script but
> totally forgot its existence, thanks : )  The patch is already queued,
> so I will change the behavior to your suggestion soon.

Sorry for confusion.  This patch is not queued yet.  The one that
filters toos/ has been queued : )

Valentin