ProjectLauncher readDocumentAtStartup

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
13 messages Options
Reply | Threaded
Open this post in threaded view
|

ProjectLauncher readDocumentAtStartup

Michael Roberts-2
Hi, this method was changed recently I think probably by the
preferences work (Alain Plantec). It used to be a pref. Now it is hard
coded to false.

Is there a reason?  It prevents scripts being executed on the command
line.  It causes the Mason example to not work in Pharo, and a bunch
of other stuff to break.

Is there a replacement for loading code on the command line? otherwise
this needs changing, perhaps you did not know it is registered in the
auto start mechanism.

thanks,
Mike

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: ProjectLauncher readDocumentAtStartup

Alain Plantec-4
Michael Roberts a écrit :
> Hi, this method was changed recently I think probably by the
> preferences work (Alain Plantec). It used to be a pref. Now it is hard
> coded to false.
>  
Hi Michael,
ok, there should a class variable and a setter for that.
> Is there a reason?  
no, its an issue.
It will be fixed as soon as possible.

Cheers
Alain
> Mike
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>  


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: ProjectLauncher readDocumentAtStartup

Alain Plantec-4
Issue 1607 <http://code.google.com/p/pharo/issues/detail?id=1607>:
ProjectLauncher readDocumentAtStartup a setting
argh, it should be ProjectLauncher readDocumentAtStartup needs a setting.
not awake enough
Alain



_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: ProjectLauncher readDocumentAtStartup

Stéphane Ducasse
BTW projectLauncher is a really funny class to support reading in .st files....
So one of these days we will have to kill it but keep the file loading aspect

Stef

On Dec 15, 2009, at 7:43 AM, Alain Plantec wrote:

> Issue 1607 <http://code.google.com/p/pharo/issues/detail?id=1607>:
> ProjectLauncher readDocumentAtStartup a setting
> argh, it should be ProjectLauncher readDocumentAtStartup needs a setting.
> not awake enough
> Alain
>
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: ProjectLauncher readDocumentAtStartup

Alain Plantec-4
Stéphane Ducasse a écrit :
> BTW projectLauncher is a really funny class to support reading in .st files....
>  
yes its cool but
showSplash and splashURL and eToyAuthentificationServer inst vars are  
never used;
SplashMorph class var is never used (so none of its vars are used).
The name, ProjectLauncher isn't that good (no other idea though)
So I would at least clean it up (remove unused variables).
I would also maybe find another name for it and move it to another package.

What do you think ?
Michael ?

Alain

> So one of these days we will have to kill it but keep the file loading aspect
>
> Stef
>
> On Dec 15, 2009, at 7:43 AM, Alain Plantec wrote:
>
>  
>> Issue 1607 <http://code.google.com/p/pharo/issues/detail?id=1607>:
>> ProjectLauncher readDocumentAtStartup a setting
>> argh, it should be ProjectLauncher readDocumentAtStartup needs a setting.
>> not awake enough
>> Alain
>>
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>    
>
>
>  


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: ProjectLauncher readDocumentAtStartup

Michael Rueger-6
Alain Plantec wrote:
> Stéphane Ducasse a écrit :
>> BTW projectLauncher is a really funny class to support reading in .st files....
>>  
> yes its cool but
> showSplash and splashURL and eToyAuthentificationServer inst vars are  
> never used;
> SplashMorph class var is never used (so none of its vars are used).
> The name, ProjectLauncher isn't that good (no other idea though)
> So I would at least clean it up (remove unused variables).

It is (or rather was) the launcher used for loading etoys projects in
the squeakland image. And more as a side effect it was able to load
scripts. Or the script loading was moved into the ProjectLauncher after
I added the launcher framework way back then ;-)

> I would also maybe find another name for it and move it to another package.

We should throw away the current class and build a new
ScriptLoaderLauncher and register it with AutoStart.

Michael

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: ProjectLauncher readDocumentAtStartup

Michael Roberts-2
I agree the current class is pretty poor. It has some odd conventions
when viewed from a unix command line. I had to debug really hard just
to find out how to exactly construct the path.

However lets please fix the current bug, it is just a small
regression.  It will at least behave as it use to.  Then let's write a
new one.

I will check out the fix tonight.  thanks!

Mike

On Tue, Dec 15, 2009 at 11:45 AM, Michael Rueger <[hidden email]> wrote:

> Alain Plantec wrote:
>> Stéphane Ducasse a écrit :
>>> BTW projectLauncher is a really funny class to support reading in .st files....
>>>
>> yes its cool but
>> showSplash and splashURL and eToyAuthentificationServer inst vars are
>> never used;
>> SplashMorph class var is never used (so none of its vars are used).
>> The name, ProjectLauncher isn't that good (no other idea though)
>> So I would at least clean it up (remove unused variables).
>
> It is (or rather was) the launcher used for loading etoys projects in
> the squeakland image. And more as a side effect it was able to load
> scripts. Or the script loading was moved into the ProjectLauncher after
> I added the launcher framework way back then ;-)
>
>> I would also maybe find another name for it and move it to another package.
>
> We should throw away the current class and build a new
> ScriptLoaderLauncher and register it with AutoStart.
>
> Michael
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: ProjectLauncher readDocumentAtStartup

Stéphane Ducasse
In reply to this post by Michael Roberts-2
Mike do you know if this problem was fixed because I need it for coral.

Stef

On Dec 15, 2009, at 12:15 AM, Michael Roberts wrote:

> Hi, this method was changed recently I think probably by the
> preferences work (Alain Plantec). It used to be a pref. Now it is hard
> coded to false.
>
> Is there a reason?  It prevents scripts being executed on the command
> line.  It causes the Mason example to not work in Pharo, and a bunch
> of other stuff to break.
>
> Is there a replacement for loading code on the command line? otherwise
> this needs changing, perhaps you did not know it is registered in the
> auto start mechanism.
>
> thanks,
> Mike
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: ProjectLauncher readDocumentAtStartup

Alain Plantec-4
Stéphane Ducasse a écrit :
> Mike do you know if this problem was fixed because I need it for coral.
>  
Hi Stef,
With my next slice you will be able to do
ProjectLauncher readDocumentAtStartup: true

Alain

> Stef
>
> On Dec 15, 2009, at 12:15 AM, Michael Roberts wrote:
>
>  
>> Hi, this method was changed recently I think probably by the
>> preferences work (Alain Plantec). It used to be a pref. Now it is hard
>> coded to false.
>>
>> Is there a reason?  It prevents scripts being executed on the command
>> line.  It causes the Mason example to not work in Pharo, and a bunch
>> of other stuff to break.
>>
>> Is there a replacement for loading code on the command line? otherwise
>> this needs changing, perhaps you did not know it is registered in the
>> auto start mechanism.
>>
>> thanks,
>> Mike
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>    
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>  


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: ProjectLauncher readDocumentAtStartup

Stéphane Ducasse
:)
Thanks alain. I decided to kill red test on something else... :)
but I will look at that.

Stef

On Dec 17, 2009, at 9:06 PM, Alain Plantec wrote:

> Stéphane Ducasse a écrit :
>> Mike do you know if this problem was fixed because I need it for coral.
>>
> Hi Stef,
> With my next slice you will be able to do
> ProjectLauncher readDocumentAtStartup: true
>
> Alain
>> Stef
>>
>> On Dec 15, 2009, at 12:15 AM, Michael Roberts wrote:
>>
>>
>>> Hi, this method was changed recently I think probably by the
>>> preferences work (Alain Plantec). It used to be a pref. Now it is hard
>>> coded to false.
>>>
>>> Is there a reason?  It prevents scripts being executed on the command
>>> line.  It causes the Mason example to not work in Pharo, and a bunch
>>> of other stuff to break.
>>>
>>> Is there a replacement for loading code on the command line? otherwise
>>> this needs changing, perhaps you did not know it is registered in the
>>> auto start mechanism.
>>>
>>> thanks,
>>> Mike
>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>>
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: ProjectLauncher readDocumentAtStartup

Michael Roberts-2
sorry stef, snowed under. will check and comment soon

I also want to write to stderr and stdout, just like you no doubt for
coral but that is another day ;-)

Mike

On Thu, Dec 17, 2009 at 8:36 PM, Stéphane Ducasse
<[hidden email]> wrote:

> :)
> Thanks alain. I decided to kill red test on something else... :)
> but I will look at that.
>
> Stef
>
> On Dec 17, 2009, at 9:06 PM, Alain Plantec wrote:
>
>> Stéphane Ducasse a écrit :
>>> Mike do you know if this problem was fixed because I need it for coral.
>>>
>> Hi Stef,
>> With my next slice you will be able to do
>> ProjectLauncher readDocumentAtStartup: true
>>
>> Alain
>>> Stef
>>>
>>> On Dec 15, 2009, at 12:15 AM, Michael Roberts wrote:
>>>
>>>
>>>> Hi, this method was changed recently I think probably by the
>>>> preferences work (Alain Plantec). It used to be a pref. Now it is hard
>>>> coded to false.
>>>>
>>>> Is there a reason?  It prevents scripts being executed on the command
>>>> line.  It causes the Mason example to not work in Pharo, and a bunch
>>>> of other stuff to break.
>>>>
>>>> Is there a replacement for loading code on the command line? otherwise
>>>> this needs changing, perhaps you did not know it is registered in the
>>>> auto start mechanism.
>>>>
>>>> thanks,
>>>> Mike
>>>>
>>>> _______________________________________________
>>>> Pharo-project mailing list
>>>> [hidden email]
>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>>
>>>
>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>
>>>
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: ProjectLauncher readDocumentAtStartup

Michael Roberts-2
Hi, there is no comment on the tracker - is the fix ready yet?

Also, I don't think it should be a preference. As far as I can tell it
is the only mechanism to get the image to run a script from the unix
command line and all the VM documentation describes this as a feature

I have uploaded the 1 line change required to Inbox and tested it on
my mac at least
System-Download-MikeRoberts.27


thanks,
Mike

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: ProjectLauncher readDocumentAtStartup

Stéphane Ducasse
In reply to this post by Michael Roberts-2
You read my mind :)
We will arrive there. I hope to send an announce soon for a stupid pre alpha :)

Stef

> sorry stef, snowed under. will check and comment soon
>
> I also want to write to stderr and stdout, just like you no doubt for
> coral but that is another day ;-)
>
> Mike
>
> On Thu, Dec 17, 2009 at 8:36 PM, Stéphane Ducasse
> <[hidden email]> wrote:
>> :)
>> Thanks alain. I decided to kill red test on something else... :)
>> but I will look at that.
>>
>> Stef
>>
>> On Dec 17, 2009, at 9:06 PM, Alain Plantec wrote:
>>
>>> Stéphane Ducasse a écrit :
>>>> Mike do you know if this problem was fixed because I need it for coral.
>>>>
>>> Hi Stef,
>>> With my next slice you will be able to do
>>> ProjectLauncher readDocumentAtStartup: true
>>>
>>> Alain
>>>> Stef
>>>>
>>>> On Dec 15, 2009, at 12:15 AM, Michael Roberts wrote:
>>>>
>>>>
>>>>> Hi, this method was changed recently I think probably by the
>>>>> preferences work (Alain Plantec). It used to be a pref. Now it is hard
>>>>> coded to false.
>>>>>
>>>>> Is there a reason?  It prevents scripts being executed on the command
>>>>> line.  It causes the Mason example to not work in Pharo, and a bunch
>>>>> of other stuff to break.
>>>>>
>>>>> Is there a replacement for loading code on the command line? otherwise
>>>>> this needs changing, perhaps you did not know it is registered in the
>>>>> auto start mechanism.
>>>>>
>>>>> thanks,
>>>>> Mike
>>>>>
>>>>> _______________________________________________
>>>>> Pharo-project mailing list
>>>>> [hidden email]
>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> Pharo-project mailing list
>>>> [hidden email]
>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>>
>>>>
>>>
>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


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