Re: [PATCHv2 4/9] save-cwd: reduce dependency

2011-04-27 Thread Eric Blake
On 04/27/2011 10:03 AM, Jim Meyering wrote: > Eric Blake wrote: >> On 04/27/2011 01:37 AM, Jim Meyering wrote: >>> Eric Blake wrote: save-cwd needs either a working fchdir or a working getcwd(NULL). The only platform where getcwd fixes problems not solved by getcwd-lgpl is Linux, but

Re: [PATCHv2 4/9] save-cwd: reduce dependency

2011-04-27 Thread Jim Meyering
Eric Blake wrote: > On 04/27/2011 01:37 AM, Jim Meyering wrote: >> Eric Blake wrote: >>> save-cwd needs either a working fchdir or a working getcwd(NULL). >>> The only platform where getcwd fixes problems not solved by >>> getcwd-lgpl is Linux, but fchdir works there. >>> >> Hi Eric, >> >> Unfortun

Re: [PATCHv2 4/9] save-cwd: reduce dependency

2011-04-27 Thread Eric Blake
On 04/27/2011 01:37 AM, Jim Meyering wrote: > Eric Blake wrote: >> save-cwd needs either a working fchdir or a working getcwd(NULL). >> The only platform where getcwd fixes problems not solved by >> getcwd-lgpl is Linux, but fchdir works there. >> > Hi Eric, > > Unfortunately, that change would in

Re: [PATCHv2 4/9] save-cwd: reduce dependency

2011-04-27 Thread Jim Meyering
Eric Blake wrote: > save-cwd needs either a working fchdir or a working getcwd(NULL). > The only platform where getcwd fixes problems not solved by > getcwd-lgpl is Linux, but fchdir works there. > > * modules/save-cwd (Depends-on): Use getcwd-lgpl. > > Signed-off-by: Eric Blake > --- > ChangeLog

Re: [PATCHv2 4/9] save-cwd: reduce dependency

2011-04-26 Thread Eric Blake
On 04/26/2011 07:03 PM, Bruno Haible wrote: > Eric Blake wrote: >> diff --git a/modules/save-cwd b/modules/save-cwd >> index 02a0723..974b596 100644 >> --- a/modules/save-cwd >> +++ b/modules/save-cwd >> @@ -9,7 +9,7 @@ m4/save-cwd.m4 >> Depends-on: >> chdir-long >> cloexec >> -getcwd >> +getcwd

Re: [PATCHv2 4/9] save-cwd: reduce dependency

2011-04-26 Thread Bruno Haible
Eric Blake wrote: > diff --git a/modules/save-cwd b/modules/save-cwd > index 02a0723..974b596 100644 > --- a/modules/save-cwd > +++ b/modules/save-cwd > @@ -9,7 +9,7 @@ m4/save-cwd.m4 > Depends-on: > chdir-long > cloexec > -getcwd > +getcwd-lgpl > fchdir > stdbool > unistd-safer Here it woul

[PATCHv2 4/9] save-cwd: reduce dependency

2011-04-26 Thread Eric Blake
save-cwd needs either a working fchdir or a working getcwd(NULL). The only platform where getcwd fixes problems not solved by getcwd-lgpl is Linux, but fchdir works there. * modules/save-cwd (Depends-on): Use getcwd-lgpl. Signed-off-by: Eric Blake --- ChangeLog|3 +++ modules/save-c