Comment 3 for bug 334576

Revision history for this message
vitjok (vitjok-gmx) wrote :

I solved the Problem for me.

#------I do not know whether this is important -----------------
#Just by the way: there is a difference between tuxpaint-import on my ubuntu 8.10 ang the sourceforge cvs repository:
#line 146 on my ubuntu box:
# i=$(eval echo \$\{$x\})
#
#line 146 on sourceforge cvs:
# i="${!x}"
#
#If I use the line from sourceforge cvs, i get an error: "Bad substitution", so I still use the line from my ubuntu version.
#-------------------------------------------------------------------------

So, now to my problem with white spaces.
I replaced the line 148:
    if [ -e $i ]; then
by this line:
    if [ -e "$i" ]; then

and the line 154:
    anytopnm $i | pnmscale -xysize $width $height > $TMPDIR/saved/$NEWFILENAME.ppm
by this line:
    anytopnm "$i" | pnmscale -xysize $width $height > $TMPDIR/saved/$NEWFILENAME.ppm

just replace $i by "$i".

And now it works for me, i can now place a starter on the desktop and by drag and drop import *all* Images in tuxpaint.

I am not sure, whether this change is OK for all and whether other user has this bug too.
So, it would be great if somebody (with more experience in shell programming) check this.

PS: /bin/sh points to /bin/dash

Oh no, that is great :-(, after I solved the problem for me, I founded this bug report:
http://sourceforge.net/tracker/index.php?func=detail&aid=1527884&group_id=66938&atid=516295
This resolve was founded over 2 years ago, but the problem was never fixed, or am i mistaken?