Thread: [PATCH] Add solaris path for docbook COLLATEINDEX

[PATCH] Add solaris path for docbook COLLATEINDEX

From
Zdenek Kotala
Date:
collateindex.pl is stored in /usr/share/sgml/docbook/. Attached fix
modify docbook.m4 to find correct path.

It would be nice also backported the fix back at least to 8.2.

    Thanks Zdenek

Attachment

Re: [PATCH] Add solaris path for docbook COLLATEINDEX

From
Bruce Momjian
Date:
Zdenek Kotala wrote:
> collateindex.pl is stored in /usr/share/sgml/docbook/. Attached fix
> modify docbook.m4 to find correct path.
> 
> It would be nice also backported the fix back at least to 8.2.

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.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + If your life is a hard drive, Christ can be your backup. +


Re: [PATCH] Add solaris path for docbook COLLATEINDEX

From
Zdenek Kotala
Date:
Bruce Momjian píše v po 30. 11. 2009 v 12:32 -0500:
> Zdenek Kotala wrote:
> > collateindex.pl is stored in /usr/share/sgml/docbook/. Attached fix
> > modify docbook.m4 to find correct path.
> > 
> > It would be nice also backported the fix back at least to 8.2.
> 
> 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.

If you think that $DOCBOOKSTYLE/.. is not good than I propose to
use /usr/share/sgml/docbook/ directly. 
Zdenek





Re: [PATCH] Add solaris path for docbook COLLATEINDEX

From
Peter Eisentraut
Date:
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.

> 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.




Re: [PATCH] Add solaris path for docbook COLLATEINDEX

From
Zdenek Kotala
Date:
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

Attachment

Re: [PATCH] Add solaris path for docbook COLLATEINDEX

From
Peter Eisentraut
Date:
On mån, 2009-11-30 at 20:57 +0100, Zdenek Kotala wrote:
> 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.

It's a normal program, so you install it where you would normally
install programs.  Like /usr/bin/ or something similar.