FS#29576 - [libreoffice-common] tcsh profile breaks if PYTHONPATH is not defined

Attached to Project: Arch Linux
Opened by Sunjae Park (darehanl) - Monday, 23 April 2012, 16:49 GMT
Last edited by Andreas Radke (AndyRTR) - Sunday, 20 May 2012, 10:39 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Andreas Radke (AndyRTR)
Stéphane Gaudreault (stephane)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
The profile file "/etc/profile.d/libreoffice-common.csh" assumes that PYTHONPATH is already defined, and tcsh breaks when encountering an undefined variable.

Additional info:
* package version(s)
libreoffice-common: 3.5.2-1
* config and/or log files etc.
/etc/profile.d/libreoffice-common.csh

Steps to reproduce:
Run tcsh without PYTHONPATH being set anywhere.

To resolve:
Wrap the line with an if-check like the following:

if ( $?PYTHONPATH ) then
setenv PYTHONPATH ${PYTHONPATH}:/usr/lib/libreoffice/program
else
setenv PYTHONPATH /usr/lib/libreoffice/program
endif
This task depends upon

Closed by  Andreas Radke (AndyRTR)
Sunday, 20 May 2012, 10:39 GMT
Reason for closing:  Duplicate
Additional comments about closing:   FS#27257 
Comment by Javier (jevv) - Monday, 23 April 2012, 17:40 GMT
Actually even for SH it doesn't look good supposing the variable exists and is defined... I changed both source scripts in my box to:

% cat /etc/profile.d/libreoffice-common.sh
if [ "$PYTHONPATH" != "" ]
then
export PYTHONPATH="$PYTHONPATH:/usr/lib/libreoffice/program"
else
export PYTHONPATH="/usr/lib/libreoffice/program"
fi

% cat /etc/profile.d/libreoffice-common.csh
if (${?PYTHONPATH}) then
setenv PYTHONPATH ${PYTHONPATH}:/usr/lib/libreoffice/program
else
setenv PYTHONPATH /usr/lib/libreoffice/program
endif
Comment by Andreas Radke (AndyRTR) - Sunday, 29 April 2012, 16:27 GMT
duplicate bug of  FS#27257  ?
Comment by Andreas Radke (AndyRTR) - Sunday, 13 May 2012, 06:15 GMT
For now I've reverted the change until we have a proper solution.

Loading...