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

Re: [PATCH] Adjust for the new way of enabling the default post-update hook

From
Petr Baudis <pasky@suse.cz>
Date
Aug 11, 2008, 23:05 UTC
Message-ID
<20080811230540.GG10151@machine.or.cz>
In-Reply-To
<7vfxpbuplr.fsf@gitster.siamese.dyndns.org>
On Mon, Aug 11, 2008 at 03:57:36PM -0700, Junio C Hamano wrote:
Show 10 quoted lines
> Petr Baudis <pasky@suse.cz> writes:
> 
> > This patch emphasizes this change in the release notes (since
> > I believe this is rather noticeable backwards-incompatible change).
> 
> Anybody who tries to chmod an nonexisting file would notice, so I
> personally think you are making a mountain out of a molehill.
> 
> Nevertheless, mentioning it in the release notes would be a good finishing
> touch.

Well, my only point is that you might want to know that you have to modify your scripts _before_ they break.

Show 10 quoted lines
> > +The standard way of making repositories friendly to dumb protocols
> > +(i.e. HTTP) changed: instead of making hooks/post-update executable,
> > +hooks/post-update.sample is to be moved to hooks/post-update.
> 
> I'm actually not very happy with the above wording that makes it sound as
> if any of the samples are part of any "standard way".
> 
> The only standard thing is to make sure update-server-info runs.  You do
> not have to start from the sample hook to do so, even though it may be one
> of the easiest ways.

Then the question is what is "standard" defined as, and why shouldn't this be the standard way? This is how our user documentation (both the User Manual and the Git Core Tutorial) teaches the users to do it.

> The rest of the patches look good, but have you actually made sure
> AsciiDoc markups are correct (I am not complaining, just asking, as I
> haven't)?

I haven't - actually, for various technical reasons, I don't even have AsciiDoc installed on my notebook right now. I plan to get around to investigate the problem, but didn't yet and I didn't want to hold this off.

-- 
				Petr "Pasky" Baudis
The next generation of interesting software will be done
on the Macintosh, not the IBM PC.  -- Bill Gates
Previous: Junio C HamanoNext: Junio C Hamano
Message 3 of 4 in “Adjust for the new way of enabling the default post-update hook”
  1. Adjust for the new way of enabling the default post-update hookPetr Baudis, Aug 11, 2008
  2. Junio C HamanoAug 11, 2008
  3. Petr BaudisAug 11, 2008
  4. Junio C HamanoAug 11, 2008

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.