[PATCH] Add solaris path for docbook COLLATEINDEX

Started by Zdenek Kotalaover 16 years ago6 messageshackers
Jump to latest
#1Zdenek Kotala
Zdenek.Kotala@Sun.COM

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

Attachments:

docbook.patchtext/x-patch; CHARSET=US-ASCII; name=docbook.patchDownload+1-1
#2Bruce Momjian
bruce@momjian.us
In reply to: Zdenek Kotala (#1)
Re: [PATCH] Add solaris path for docbook COLLATEINDEX

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

#3Zdenek Kotala
Zdenek.Kotala@Sun.COM
In reply to: Bruce Momjian (#2)
Re: [PATCH] Add solaris path for docbook COLLATEINDEX

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

#4Peter Eisentraut
peter_e@gmx.net
In reply to: Zdenek Kotala (#3)
Re: [PATCH] Add solaris path for docbook COLLATEINDEX

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.

#5Zdenek Kotala
Zdenek.Kotala@Sun.COM
In reply to: Peter Eisentraut (#4)
Re: [PATCH] Add solaris path for docbook COLLATEINDEX

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

Attachments:

docbook_02.patchtext/x-patch; CHARSET=US-ASCII; name=docbook_02.patchDownload+1-1
#6Peter Eisentraut
peter_e@gmx.net
In reply to: Zdenek Kotala (#5)
Re: [PATCH] Add solaris path for docbook COLLATEINDEX

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.