[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [Gnu-arch-users] Re: patch: clean up some cacherev mess
From: |
Derek Zhou |
Subject: |
RE: [Gnu-arch-users] Re: patch: clean up some cacherev mess |
Date: |
Sat, 19 Nov 2005 16:40:26 -0800 |
I am a little bit confused here.
Neither my home dir nor /tmp are g+s?
The only thing so special about /tmp is it is w+t, which has nothing to
do the permission of the dir within?
An what do you mean by non-transient tmp_dir?
Derek
> -----Original Message-----
> From: address@hidden
> [mailto:address@hidden On
> Behalf Of Mikhael Goikhman
> Sent: Saturday, November 19, 2005 3:43 PM
> To: address@hidden
> Subject: Re: [Gnu-arch-users] Re: patch: clean up some cacherev mess
>
> On 19 Nov 2005 12:50:12 +0100, Matthieu Moy wrote:
> >
> > Derek Zhou <address@hidden> writes:
> >
> > > --- orig/libarch/archive-cache.c
> > > +++ mod/libarch/archive-cache.c
> > > @@ -27,7 +27,7 @@
> > > t_uchar * tmp_dir = 0;
> > > t_uchar * pristine_dir = 0;
> > >
> > > - tmp_dir = tmp_file_name (".", ",,archive-cache-revision");
> > > + tmp_dir = tmp_file_name ("/tmp", ",,archive-cache-revision");
> > > pristine_dir = file_name_in_vicinity (0, tmp_dir, revision);
> > >
> > > safe_mkdir (tmp_dir, 0777);
> >
> > "/tmp/" should be something like tmp_dir(...) (with the appropriate
> > "free") to obey TMPDIR.
>
> Please don't blindly change "." to "/tmp" if this has a side effect of
> loosing inherited g+s permission bit (and group name) in non-transient
> subdirectories created by tla.
>
> Regards,
> Mikhael.
>
>
> _______________________________________________
> Gnu-arch-users mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/gnu-arch-users
>
> GNU arch home page:
> http://savannah.gnu.org/projects/gnu-arch/
>