Re: [PATCH] Add solaris path for docbook COLLATEINDEX
От | Zdenek Kotala |
---|---|
Тема | Re: [PATCH] Add solaris path for docbook COLLATEINDEX |
Дата | |
Msg-id | 1259611042.1115.130.camel@localhost обсуждение исходный текст |
Ответ на | Re: [PATCH] Add solaris path for docbook COLLATEINDEX (Peter Eisentraut <peter_e@gmx.net>) |
Ответы |
Re: [PATCH] Add solaris path for docbook COLLATEINDEX
|
Список | pgsql-hackers |
Peter Eisentraut píše v po 30. 11. 2009 v 21:27 +0200: > On mån, 2009-11-30 at 19:53 +0100, Zdenek Kotala wrote: > > Bruce Momjian píše v po 30. 11. 2009 v 12:32 -0500: > > > I am not happy looking in a directory _above_ a specified directory by > > > default: > > > > > > [$DOCBOOKSTYLE/bin $DOCBOOKSTYLE/.. $PATH]) > > > > > > That seems possibly unsafe. I suggest you just add it to the PATH for > > > Solaris builds. > > > > I'm not sure if it is unsafer that searching in $PATH. Anyway I don't > > think that your proposed solution is good way. I'm able to do it but how > > many other people can? Who know where collateindex.pl stays on Solaris? > > configure is here to make live easier for people. If we know how to do > > it automatically we should do it. > > Note that $DOCBOOKSTYLE/bin corresponds to the location of the file in > an unpacked source archive and $PATH is of course where programs > normally go. The $DOCBOOKSTYLE/.. location makes no sense at all, > because that basically says that the installer intentionally moved the > file, but to a completely nonstandard location. I'm not sgml//docbook guru. Do you think that Solaris location of collateindex.pl is wrong? Does exist any recommendation for this? I could log a bug, but I need some link with recommendation. > > If you think that $DOCBOOKSTYLE/.. is not good than I propose to > > use /usr/share/sgml/docbook/ directly. > > That would be the less ugly solution. Patch attached. thanks Zdenek
Вложения
В списке pgsql-hackers по дате отправления: