IMAGE="$DIR/Pharo.image";
if [ ! -z "$IMAGE" ];
�
is a bit superfluous :) Agreed?
Norbert
P.S.: Thanks for picking it up. I need to have a look at the zeroconf stuff.
On 2013-04-02, at 12:02, Camillo Bruni <camillobruni@gmail.com> wrote:
Yes I just haven't fully announced it yet ;) since I only programmed them
this saturday and was waiting for the pharo conf ;)
But I will deprecate it with a warning message in it tomorrow or so.
And then people should rename their scripts.
On 2013-04-02, at 11:58, Norbert Hartl <norbert@hartl.name> wrote:
Well, I would say a
mv vm-ui.sh pharo-ui
would do a good job to get out of deprecation. Honestly if the scripts are deprecated just take them out of the download. I don't know why to leave them there because it is confusing.
Norbert
Am 02.04.2013 um 11:24 schrieb Esteban Lorenzano <estebanlm@gmail.com>:
Hi Norbert,
the vm-ui.sh script is deprecated (yep, Camillo already deprecated something, even if it is brand new).
now the valid scripts are
"pharo" and "pharo-ui"
:)
Esteban
On Apr 2, 2013, at 11:17 AM, Norbert Hartl <norbert@hartl.name> wrote:
Another attempt.
Norbert
<vm-ui.sh>
Am 02.04.2013 um 10:43 schrieb Norbert Hartl <norbert@hartl.name>:
Ah, rats, didn't think it through. It only works for commandline switches that do not have an argument itself. Sad and sorry!
Norbert
Am 02.04.2013 um 10:38 schrieb Norbert Hartl <norbert@hartl.name>:
Very good Cami. I have two remarks. If it should be short I think that
curl -sL get.pharo.org | bash
is shorter than
wget --quiet -O - get.pharo.org | bash
If I see it correct the script is used for CI but as there is a get.pharo.org host I assume it is supposed to be an easy entry point for all kind of users, right?
If this is true I find it somewhat strange to have a download-all-in-one-and-cozy-script that downloads everything needed but to start it I have still to provide the image myself to the shell scripts. So I added this to the script
# parse vm options and image
for arg in "$@"
do
if [[ $arg == -* ]];
then
VMOPTIONS="$VMOPTIONS $arg"
shift
else
break
fi
done
IMAGE=$1
shift
# if no image has been provided use the installed one
if [ -z "$IMAGE" ];
then
IMAGE=$DIR/Pharo.image
fi
which uses the downloaded image in case nothing has been specified on the commandline.
Hope you like it. Usually I would extract the same portion of both scripts to an extra file but then there is the trade-off to have one more file in the download. I'll attach a version of vm-ui.sh for you to try. If you need help I'm happy to chime in for this.
thanks for the effort,
Norbert
<vm-ui.sh>
Am 02.04.2013 um 09:15 schrieb Camillo Bruni <camillobruni@gmail.com>:
we're back up, shinier than ever ;)
check http://get.pharo.org ;)
now you can use:
wget --quiet -O - get.pharo.org | bash
for the current stable release of Pharo and the PharoVM.
On 2013-04-01, at 22:52, Tudor Girba <tudor@tudorgirba.com> wrote:
No stress. In the worst case you can fix it tomorrow during the tutorial :).
Doru
On Apr 1, 2013, at 10:26 PM, Camillo Bruni <camillobruni@gmail.com> wrote:
On 2013-04-01, at 22:09, "Sean P. DeNigris" <sean@clipperadams.com> wrote:
Tudor Girba-2 wrote
Could it have something to do with the latest optimizations of Camillo?
Probably... Cami breaks everything :-P
dammit, I wanted to silently add the new zeroconf scripts
obviously something went horribly wrong there, sorry guys.. working on it!
--
www.tudorgirba.com
"What is more important: To be happy, or to make happy?"