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

Re: [PATCH] Make core.sharedRepository more generic (version 2)

From
HOHeikki Orsila <shdl@zakalwe.fi>
Date
Apr 15, 2008, 01:22 UTC
Message-ID
<20080415012249.GJ31039@zakalwe.fi>
In-Reply-To
<20080415004310.GI31039@zakalwe.fi>
On Tue, Apr 15, 2008 at 03:43:10AM +0300, Heikki Orsila wrote:
Show 14 quoted lines
> On Mon, Apr 14, 2008 at 05:08:03PM -0700, Junio C Hamano wrote:
> > > +		 * of git. Note, we use octal numbers.
> > >  		 */
> > > -		sprintf(buf, "%d", shared_repository);
> > > +		sprintf(buf, "0%o", shared_repository);
> > 
> > Unconditionally doing this makes the resulting repository unusable by git
> > 1.5.5 and older, even when the user wanted to use the bog standard "git
> > init --shared".  You can limit the extent of damage if you continue
> > writing PERM_GROUP and PERM_EVERYBODY out as 1 and 2, and use the new
> > octal notation only when the user used the settings allowed only with new
> > git.
> 
> I submitted a new patch that should address all the points.

Submitted another version that really fixes the o+w case. Forgot the 0664 mask there. Now it's 0666.

-- 
Heikki Orsila
heikki.orsila@iki.fi
http://www.iki.fi/shd
Previous: Heikki Orsila
Message 4 of 4 in “Make core.sharedRepository more generic (version 2)”
  1. Make core.sharedRepository more generic (version 2)Heikki Orsila, Apr 12, 2008
  2. Junio C HamanoApr 15, 2008
  3. Heikki OrsilaApr 15, 2008
  4. Heikki OrsilaApr 15, 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.