Smalltalk › Pharo › Pharo Smalltalk Developers

pharo script doesn't work on paths that contain whitespaces

_‹ Previous Topic Next Topic _›

Classic

List

Threaded

3 messages Options

tomo

pharo script doesn't work on paths that contain whitespaces

Hello,

I found my Jenkins jobs with scripts like below fails to start headless pharo process these couple of nights.

---

wget --quiet -O - get.pharo.org/50+vm | bash

./pharo Pharo.image save Pharo5.0 --delete-old

---

The problem is in the last line of the downloaded pharo script;

LD_LIBRARY_PATH=$DIR/pharo-vm:$LD_LIBRARY_PATH $DIR/pharo-vm/pharo --nodisplay "$@"

should be quoted lilke

LD_LIBRARY_PATH="$DIR/pharo-vm:$LD_LIBRARY_PATH" "$DIR/pharo-vm/pharo" --nodisplay "$@"

It seems the script for Pharo6 also has the same issue.

Best Regards,

---

tomo

Ben Coman

Re: pharo script doesn't work on paths that contain whitespaces

Thanks for the report Tomo.

cheers -ben

On Fri, Mar 17, 2017 at 10:22 AM, Tomohiro Oda <[hidden email]> wrote:

Hello,

I found my Jenkins jobs with scripts like below fails to start headless pharo process these couple of nights.
---
wget --quiet -O - get.pharo.org/50+vm | bash
./pharo Pharo.image save Pharo5.0 --delete-old
---

The problem is in the last line of the downloaded pharo script;

LD_LIBRARY_PATH=$DIR/pharo-vm:$LD_LIBRARY_PATH $DIR/pharo-vm/pharo --nodisplay "$@"

should be quoted lilke
LD_LIBRARY_PATH="$DIR/pharo-vm:$LD_LIBRARY_PATH" "$DIR/pharo-vm/pharo" --nodisplay "$@"

It seems the script for Pharo6 also has the same issue.

Best Regards,
---
tomo

EstebanLM

Re: pharo script doesn't work on paths that contain whitespaces

ouch!

I’m sorry :)

anyway I’m reverting that change because I’m transmitting that configuration to general script (by using osvm script).

Esteban

On 17 Mar 2017, at 04:35, Ben Coman <[hidden email]> wrote:

Thanks for the report Tomo.
cheers -ben

On Fri, Mar 17, 2017 at 10:22 AM, Tomohiro Oda <[hidden email]> wrote:
Hello,

I found my Jenkins jobs with scripts like below fails to start headless pharo process these couple of nights.
---
wget --quiet -O - get.pharo.org/50+vm | bash
./pharo Pharo.image save Pharo5.0 --delete-old
---

The problem is in the last line of the downloaded pharo script;

LD_LIBRARY_PATH=$DIR/pharo-vm:$LD_LIBRARY_PATH $DIR/pharo-vm/pharo --nodisplay "$@"

should be quoted lilke
LD_LIBRARY_PATH="$DIR/pharo-vm:$LD_LIBRARY_PATH" "$DIR/pharo-vm/pharo" --nodisplay "$@"

It seems the script for Pharo6 also has the same issue.

Best Regards,
---
tomo