Issue 5950 in pharo: Whitespace in directory names for the OneClick script

Status: Accepted
Owner: [hidden email]
Labels: Type-Bug

New issue 5950 by [hidden email]: Whitespace in directory names for  
the OneClick script
http://code.google.com/p/pharo/issues/detail?id=5950

Reported by Thierry Goubier:

it seems the script shell does not handle paths with spaces in their names  
under Linux.

Replacing in Pharo-1.4-14438-OneClick.sh

ROOT = `readlink -f $(dirname $0)`

by

DIR=`dirname "$0"`
ROOT=`readlink -f "$DIR"`

seems to solve the problem.


Thierry
--


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixReviewNeeded

Comment #1 on issue 5950 by [hidden email]: Whitespace in directory  
names for the OneClick script
http://code.google.com/p/pharo/issues/detail?id=5950

Gastón Dall' Oglio:

Another little fix. In the linux I use, zenity ignore the directory change  
in Pharo-1.4 one-click.sh (cd "$RESOURCES" ), showing my home directory for  
default when open the dialog. Maybe in Ubuntu work fine, but I don't have  
one for test.

I changed de logic, putting a value in variable "image" in advance, before  
open the dialog, and then zenity show the Resource directory, with the  
default image selected if it exists.

I attached my version of script.

Attachments:
        Pharo-1.4-one-click.sh  761 bytes

_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Labels: Milestone-1.4 Milestone-2.0

Comment #2 on issue 5950 by [hidden email]: Whitespace in directory  
names for the OneClick script
http://code.google.com/p/pharo/issues/detail?id=5950

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: NoSourcesAvailable

Comment #3 on issue 5950 by [hidden email]: Whitespace in directory  
names for the OneClick script
http://code.google.com/p/pharo/issues/detail?id=5950#c3

This Issue has been checked by Ulysse the Monkey, but no sources are  
available

Someone should have a look either to check the validity of the source or to  
report a strange behavior of a crazy monkey.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: HumanReviewNeeded

Comment #4 on issue 5950 by [hidden email]: Whitespace in directory  
names for the OneClick script
http://code.google.com/p/pharo/issues/detail?id=5950

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixToInclude

Comment #5 on issue 5950 by [hidden email]: Whitespace in directory  
names for the OneClick script
http://code.google.com/p/pharo/issues/detail?id=5950

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Integrated

Comment #6 on issue 5950 by [hidden email]: Whitespace in directory  
names for the OneClick script
http://code.google.com/p/pharo/issues/detail?id=5950

(No comment was entered for this change.)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker