git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Odd results writing a Git pre-receive hook to syntax check PHP files.

From
CPChris Patti <cpatti@gmail.com>
Date
Aug 30, 2010, 15:00 UTC
Message-ID
<AANLkTimFx0DQb0Bq=00O5jSwbFpxU6TE53fMoiJd4Rdy@mail.gmail.com>
In-Reply-To
<4C7BC068.7030909@drmicha.warpmail.net>

On Mon, Aug 30, 2010 at 10:30 AM, Michael J Gruber <git@drmicha.warpmail.net> wrote:

Show 19 quoted lines
> Jonathan Nieder venit, vidit, dixit 30.08.2010 16:16:
>> Chris Patti wrote:
>>
>>> That's great, thanks!  That gets me the file names.  Now, how do I get
>>> to the contents of each file?
>>
>> Why not "git diff --raw" (or its plumbing equivalent "git diff-tree")?
>
> That's what I meant by "diff family" ;)
> In fact, "git diff --raw --name-only oldrev newrev" gives the nicest
> listing.
>
> It seems Chris has to check full file contents, i.e. he'll have to loop
> over this list (as before), but I guess he'll need to replace his "show
> $blob" by "show newrev:$filename" (or, alternatively, parse the output
> of "diff --raw" without "--name-only" for the blob sha1).
>
> Michael
>

Exactly, I'm writing a regex to parse the diff --raw output and extract the needed bits. I was already doing this with ls-tree so I don't expect any difficulties.

-Chris
-- 
Christopher Patti - Geek At Large | GTalk: cpatti@gmail.com | AIM:
chrisfeohpatti | P: (260) 54PATTI
"Technology challenges art, art inspires technology." - John Lasseter, Pixar
Previous: Michael J GruberNext: Chris Patti
Message 11 of 20 in “Odd results writing a Git pre-receive hook to syntax check PHP files.”
  1. Chris PattiAug 27, 2010
  2. Michael J GruberAug 30, 2010
  3. Chris PattiAug 30, 2010
  4. Jonathan NiederAug 30, 2010
  5. Chris PattiAug 30, 2010
  6. Example git hooks (Re: Odd results writing a Git pre-receive hook to syntax check PHP files.)Jonathan Nieder, Aug 30, 2010
  7. Todd ZullingerAug 31, 2010
  8. Chris PattiAug 31, 2010
  9. Chris PattiAug 31, 2010
  10. Michael J GruberAug 30, 2010
  11. Chris PattiAug 30, 2010
  12. Chris PattiAug 30, 2010
  13. Sverre RabbelierAug 30, 2010
  14. Jonathan NiederAug 30, 2010
  15. Chris PattiAug 30, 2010
  16. Michael J GruberAug 30, 2010
  17. Chris PattiAug 31, 2010
  18. Junio C HamanoAug 31, 2010
  19. Chris PattiSep 1, 2010
  20. Chris PattiSep 2, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.