Re: src/backend/Makefile CDPATH Patch

Поиск
Список
Период
Сортировка
От Bruce Momjian
Тема Re: src/backend/Makefile CDPATH Patch
Дата
Msg-id 200104232010.f3NKAH006700@candle.pha.pa.us
обсуждение исходный текст
Ответ на Re: src/backend/Makefile CDPATH Patch  (Jason Tishler <Jason.Tishler@dothill.com>)
Список pgsql-patches
> Bruce,
>
> On Mon, Apr 23, 2001 at 03:25:38PM -0400, Bruce Momjian wrote:
> > > The above is due to the following:
> > >
> > >     $ export CDPATH=.:/home/jt:/home/jt/src:/home/jt/lib
> > >     $ echo `cd parser/ && pwd`
> > >     /home/jt/src/pgsql/src/backend/parser /home/jt/src/pgsql/src/backend/parser
> >
> > Basically, you are telling me that if there are two matching parser
> > directories, pwd gets set to _both_ of them.  Sorry, I don't think we
> > want to work around a bug like that, should we?
>
> I guess that I was not explicit enough, so I will try again...
>
> The extra directory name comes from bash's builtin cd echoing the
> directory name when a match is found in the user's CDPATH.  It is *not*
> coming from pwd.

Oh, I understand now.  Good point.

>
> So, I still content that this is a bug in the Makefile.  The construct
> that is suspect is the following:
>
>     prereqdir=`cd $(dir $<) && pwd`

Yep, I can see it doing that.

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

В списке pgsql-patches по дате отправления:

Предыдущее
От: Jason Tishler
Дата:
Сообщение: Re: src/backend/Makefile CDPATH Patch
Следующее
От: Peter Eisentraut
Дата:
Сообщение: Re: src/backend/Makefile CDPATH Patch