Re: Bug in /etc/profile.d/lilypond.sh (with fix!)

[Mats Bengtsson]

With your patch, the script will not work for people who already
have the TEXMF variable set for other reasons.
I cannot see that the current solution is any major problem, since
it should not occur very often that people start so many shells
inside each other that the variable length breaks any limits.
It will still work even if the same path happens to occur several
times. Therefore, I suggest not to include this patch.

   /Mats

Reuben Thomas wrote:
> The Lilypond RPM contains the following stanza:
>
>         # Add the installation directory to the teTeX system tree,
>         # see Documentation/misc/fontinstallation
>         TEXMF="{$datadir,"`kpsexpand  \\$TEXMF`"}"
>         export TEXMF
>
> This isn't quite right, as every time you start a shell inside another
> one, TEXMF gets another copy of the path shoved on to the front. The
> following works:
>
>         # Add the installation directory to the teTeX system tree,
>         # see Documentation/misc/fontinstallation
>         if [ -z `echo $TEXMF | grep "$datadir"` ]; then
>                 TEXMF="{$datadir,"`kpsexpand  \\$TEXMF`"}"
>                 export TEXMF
>         fi
>
> The same goes for the commented-out lines exporting GS_LIB and
> GS_FONTPATH; may as well correct those too.

--
=============================================
        Mats Bengtsson
        Signal Processing
        Signals, Sensors and Systems
        Royal Institute of Technology
        SE-100 44  STOCKHOLM
        Sweden
        Phone: (+46) 8 790 8463                         
        Fax:   (+46) 8 790 7260
        Email: address@hidden
        WWW: http://www.s3.kth.se/~mabe
=============================================