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

Re: pre-commit not working at server side push

From
JBJ. Bakshi <joydeep@infoservices.in>
Date
Jul 20, 2011, 07:21 UTC
Message-ID
<20110720125124.3c746a1a@shiva.selfip.org>
In-Reply-To
<20110720071807.GA10104@LK-Perkele-VI.localdomain>

On Wed, 20 Jul 2011 10:18:07 +0300 Ilari Liusvaara <ilari.liusvaara@elisanet.fi> wrote:

Show 18 quoted lines
> On Wed, Jul 20, 2011 at 12:13:56PM +0530, J. Bakshi wrote:
> > Hello list,
> > 
> > I have this bash script to check php syntax error and prevent the commit if there is any
>  
> <snip>
>  
> > Running fine when placed it at local copy.
> > 
> > I have placed the hook at the server under hooks directory and provided the execute permission also
> > 
> > $ chmod a+x hooks/pre-commit
> > 
> > But even with wrong php syntax I am able to push the file. Have I missed something ?
> 
> Pre-commit runs on commit, not on push. And push is not commit.
> 
> -Ilari
Thanks, already figured it out, it would be pre-receive on server; but still not working.
Previous: Ilari Liusvaara
Message 7 of 7 in “pre-commit not working at server side push”
  1. J. BakshiJul 20, 2011
  2. J. BakshiJul 20, 2011
  3. Ilari LiusvaaraJul 20, 2011
  4. J. BakshiJul 20, 2011
  5. J. BakshiJul 20, 2011
  6. Ilari LiusvaaraJul 20, 2011
  7. J. BakshiJul 20, 2011

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.