[Pharo-project] [3.0] CI build is broken
https://ci.inria.fr/pharo/job/Pharo-3.0-Update-Step-1-Tracker/82/console 2013-04-01 20:54:16 (6.41 MB/s) - `updates30.staged' saved [28/28] + bash + wget --quiet -O - http://files.pharo.org/script/ciPharo30PharoVM.sh Downloading the latest 30 Image: http://files.pharo.org/image/30/latest.zip image.3YiB0/Pharo-30019.image bash: line 70: syntax error near unexpected token `newline' Build step 'Execute shell' marked build as failure Archiving artifacts Finished: FAILURE
So is the Moose build (Pharo 2.0). Could it have something to do with the latest optimizations of Camillo? Doru On Apr 1, 2013, at 9:25 PM, Marcus Denker <marcus.denker@inria.fr> wrote:
https://ci.inria.fr/pharo/job/Pharo-3.0-Update-Step-1-Tracker/82/console
2013-04-01 20:54:16 (6.41 MB/s) - `updates30.staged' saved [28/28]
+ bash + wget --quiet -O - http://files.pharo.org/script/ciPharo30PharoVM.sh Downloading the latest 30 Image: http://files.pharo.org/image/30/latest.zip image.3YiB0/Pharo-30019.image bash: line 70: syntax error near unexpected token `newline' Build step 'Execute shell' marked build as failure Archiving artifacts Finished: FAILURE
-- www.tudorgirba.com "No matter how many recipes we know, we still value a chef."
Tudor Girba-2 wrote
Could it have something to do with the latest optimizations of Camillo?
Probably... Cami breaks everything :-P ----- Cheers, Sean -- View this message in context: http://forum.world.st/3-0-CI-build-is-broken-tp4679211p4679220.html Sent from the Pharo Smalltalk mailing list archive at Nabble.com.
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!
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?"
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?"
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 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?"
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?"
Another attempt. Norbert 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?"
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?"
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?"
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?"
Norbert, I deprecated now the old scripts and I had a look at your script proposal and I extended it a bit: https://gist.github.com/dh83/5363000 what do you think? 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?"
Am 11.04.2013 um 14:44 schrieb Camillo Bruni <camillobruni@gmail.com>:
Norbert, I deprecated now the old scripts and I had a look at your script proposal and I extended it a bit:
https://gist.github.com/dh83/5363000
what do you think?
Looks good to me, although I find ⦠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?"
On 2013-04-11, at 14:56, Norbert Hartl <norbert@hartl.name> wrote:
Am 11.04.2013 um 14:44 schrieb Camillo Bruni <camillobruni@gmail.com>:
Norbert, I deprecated now the old scripts and I had a look at your script proposal and I extended it a bit:
https://gist.github.com/dh83/5363000
what do you think?
Looks good to me, although I find
⦠IMAGE="$DIR/Pharo.image"; if [ ! -z "$IMAGE" ]; â¦
is a bit superfluous :) Agreed?
I guess I can always output the warning ;) I am just not a bit fan of too much auto conversion (besides bash itself is magic enough).
Norbert
P.S.: Thanks for picking it up. I need to have a look at the zeroconf stuff.
On 2013-04-02, at 11:24, Esteban Lorenzano <estebanlm@gmail.com> wrote:
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"
with the same contents ;)
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?"
On 2013-04-02, at 10:38, Norbert Hartl <norbert@hartl.name> wrote:
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
yes definitely! though it seems that I cannot properly install curl under mingw :P otherwise it is definitely nicer ;)
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?
yes
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
humm the thing is, that I usually rename the image on jenkins. So for my common use-case it is somehow not needed. But I see that it would convey more to the all-in-one zeroconf idea ;)
# 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.
I am bit tired so might make mistakes ;). Currently I generate the scripts from the files present here: http://smalltalkhub.com/#!/~Pharo/ZeroConf There are nice smalltalk classes that generate all the scripts and hence duplication is not really an issue :). I like your idea, though I have to think about the side-effects a bit. Maybe later today ;), but I'll look into it
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?"
On 02 Apr 2013, at 09:15, Camillo Bruni <camillobruni@gmail.com> wrote:
we're back up, shinier than ever ;)
check http://get.pharo.org ;)
Beautiful and simple, the mark of a true master. Cool HTML/bash hack ;-) Sven
You should see the demo live. It's so ... how should I put it ... sexy :) Doru On Apr 2, 2013, at 10:43 AM, Sven Van Caekenberghe <sven@stfx.eu> wrote:
On 02 Apr 2013, at 09:15, Camillo Bruni <camillobruni@gmail.com> wrote:
we're back up, shinier than ever ;)
check http://get.pharo.org ;)
Beautiful and simple, the mark of a true master.
Cool HTML/bash hack ;-)
Sven
-- www.tudorgirba.com "Don't give to get. Just give."
On 02 Apr 2013, at 11:12, Tudor Girba <tudor@tudorgirba.com> wrote:
You should see the demo live. It's so ... how should I put it ... sexy :)
Yes, I wish I was there - hope you are recording ...
Doru
On Apr 2, 2013, at 10:43 AM, Sven Van Caekenberghe <sven@stfx.eu> wrote:
On 02 Apr 2013, at 09:15, Camillo Bruni <camillobruni@gmail.com> wrote:
we're back up, shinier than ever ;)
check http://get.pharo.org ;)
Beautiful and simple, the mark of a true master.
Cool HTML/bash hack ;-)
Sven
-- www.tudorgirba.com
"Don't give to get. Just give."
We are. Doru On Apr 2, 2013, at 11:14 AM, Sven Van Caekenberghe <sven@stfx.eu> wrote:
On 02 Apr 2013, at 11:12, Tudor Girba <tudor@tudorgirba.com> wrote:
You should see the demo live. It's so ... how should I put it ... sexy :)
Yes, I wish I was there - hope you are recording ...
Doru
On Apr 2, 2013, at 10:43 AM, Sven Van Caekenberghe <sven@stfx.eu> wrote:
On 02 Apr 2013, at 09:15, Camillo Bruni <camillobruni@gmail.com> wrote:
we're back up, shinier than ever ;)
check http://get.pharo.org ;)
Beautiful and simple, the mark of a true master.
Cool HTML/bash hack ;-)
Sven
-- www.tudorgirba.com
"Don't give to get. Just give."
-- www.tudorgirba.com "Obvious things are difficult to teach."
On 02 Apr 2013, at 11:19, Tudor Girba <tudor@tudorgirba.com> wrote:
We are.
Thx!
Doru
On Apr 2, 2013, at 11:14 AM, Sven Van Caekenberghe <sven@stfx.eu> wrote:
On 02 Apr 2013, at 11:12, Tudor Girba <tudor@tudorgirba.com> wrote:
You should see the demo live. It's so ... how should I put it ... sexy :)
Yes, I wish I was there - hope you are recording ...
Doru
On Apr 2, 2013, at 10:43 AM, Sven Van Caekenberghe <sven@stfx.eu> wrote:
On 02 Apr 2013, at 09:15, Camillo Bruni <camillobruni@gmail.com> wrote:
we're back up, shinier than ever ;)
check http://get.pharo.org ;)
Beautiful and simple, the mark of a true master.
Cool HTML/bash hack ;-)
Sven
-- www.tudorgirba.com
"Don't give to get. Just give."
-- www.tudorgirba.com
"Obvious things are difficult to teach."
participants (7)
-
Camillo Bruni -
Esteban Lorenzano -
Marcus Denker -
Norbert Hartl -
Sean P. DeNigris -
Sven Van Caekenberghe -
Tudor Girba