trinity-devel@lists.pearsoncomputing.net

Message: previous - next
Month: April 2012

Re: [trinity-devel] Request help revising starttde

From: Slávek Banko <slavek.banko@...>
Date: Wed, 11 Apr 2012 19:10:02 +0200
Dne st 11. dubna 2012 Darrell Anderson napsal(a):
> > Lastly I looked at the patch - it looks good. Just one note:
> > $TDEDIR/share/man is "added" to the MANPATH, even when empty. But
> > instead of direct testing MANPATH probably should be checked output
> > from call `manpath`.
>
> MANPATH test added.
>
> Merged upstream in GIT hash 398ef116.
>
> Thank you!
>
> Darrell
>

Oops, setting MANPATH is wrong! MANPATH is not only independent variable. 
He has a relationship with the configuration file /etc/manpath.config - 
see: man manpath

The problem was not in the colon at end, when value is not set. The colon 
is desirable there. But, instead in the variable MANPATH must be searched 
in the result of call 'manpath'.

I add the patch to bug 675.

Slávek
--