Error when trying to create a package

When every I try to create a package in the browser, I see an error.
Inside OBPackageOrClassNode>>shouldRefresh: the value of "ann event"
is nil, so the call to itemKind blows up.

I believe I have the latest version of Pharo, with all updates as of a
day or two ago. This has been happening for a long time, though.

Am I doing something wrong, or is this a bug?

Jim
--
Jim Menard, [hidden email], [hidden email]
http://www.io.com/~jimm/

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

> Am I doing something wrong, or is this a bug?

This is a bug. I bumped in it already.
It was introduced when David and I worked on improving the speed of OB

Cheers,
Alexandre
--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






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

Alexandre,

>> Am I doing something wrong, or is this a bug?
>
> This is a bug. I bumped in it already.
> It was introduced when David and I worked on improving the speed of OB

Thanks for the quick response. I'm not familiar enough with the
internals of the browser code to offer a solution. I can see from
travelling down the stack that the OBPackageBrowser object's
changeEvent is nil, but that's all.

Jim
--
Jim Menard, [hidden email], [hidden email]
http://www.io.com/~jimm/

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

I just tried and it works as expected. I fix the bug you mentioned on  
April 20, 2009. Which version of OB-Enhancements do you use?

Cheers,
Alexandre

On 9 May 2009, at 21:17, Jim Menard wrote:

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






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

Alexandre,

On Sat, May 9, 2009 at 4:02 PM, Alexandre Bergel <[hidden email]> wrote:
> I just tried and it works as expected. I fix the bug you mentioned on
> April 20, 2009. Which version of OB-Enhancements do you use?

I don't know. How can I tell?

Jim
--
Jim Menard, [hidden email], [hidden email]
http://www.io.com/~jimm/

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

Open a Monticello browser, and search for OB-Enhancements

Cheers,
Alexandre


On 9 May 2009, at 22:48, Jim Menard wrote:

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






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

> Open a Monticello browser, and search for OB-Enhancements

It's dr.319

Jim



--
Jim Menard, [hidden email], [hidden email]
http://www.io.com/~jimm/

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

Here we are. Just update OB. The following doIt should be enough:
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
        | instClass |
        instClass := self environment at: #Installer ifAbsent: [ScriptLoader  
new installingInstaller].
       
        ScriptLoader new universalInstaller universe
                update;
                addPackage: 'Shout';
                install.
               
        "load latest alpha version of OB directly from its Monticello repo"
        instClass wiresong
                project: 'ob';
                install: 'OmniBrowser';
                install: 'OB-Standard';
                install: 'OB-Morphic'.

        instClass ss
                project: 'AST';
                install: 'AST-lr'.

        instClass ss
                project: 'RefactoringEngine';
                install: 'Refactoring-Core';
                install: 'Refactoring-Spelling'.
               
        instClass ss
                project: 'Regex';
                install: 'VB-Regex'.
       
        instClass wiresong
                project: 'ob';
                install: 'OB-Refactory';
                install: 'OB-Enhancements'.
               
               
        instClass lukas
                project: 'omnibrowser';
                install: 'OB-Regex'
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=

ScriptLoader loadOBAlpha may not work since you probably have a old  
version of ScriptLoader.

Alexandre


On 10 May 2009, at 00:18, Jim Menard wrote:

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






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

Alexandre,

On Sun, May 10, 2009 at 3:30 AM, Alexandre Bergel <[hidden email]> wrote:
> Here we are. Just update OB. The following doIt should be enough:

Thank you.

Jim
--
Jim Menard, [hidden email], [hidden email]
http://www.io.com/~jimm/

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

Alexandre Bergel wrote: Installer was designed so that you could write scripts that you can edit
and execute, wholly or bit by bit in a Workspace.

Great lengths were made to ensure that scripts could be written without
temporary variables.

Showing example scripts that use temporary variables like this is not
showing Installer as it is intended to be used.

Keith




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

Keith

do you if this is my version only or this is just a behavior that is  
not supported:
here is my scenario: I loaded packages and now they are in my package  
cache.
I run to cache the train and there I would like to rerun the script  
but the system
tells me that the network is not present so I end up loading my hand  
the packages that
are in my package cache.
I would like to add that kind of behavior to Installer

Stef

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

On Mon, 11 May 2009 03:05:24 +0100
Keith Hodges <[hidden email]> wrote:

> Showing example scripts that use temporary variables like this is not
> showing Installer as it is intended to be used.

Back when I was trained to be a teacher, my mentor told
me to *always* be constructive when correcting mistakes:
"Don't tell them that they are wrong, tell them how to do
it right."

So how would the script above look like using Installer
in a more idiomatic way?

Thanks,
s.

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

This is what we've been doing for long time...

Alexandre

On 11 May 2009, at 04:05, Keith Hodges wrote:

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






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

The temporary variable is there in case that Installer is not  
installed. It assumes that ScriptLoader is present.

Cheers,
Alexandre


On 11 May 2009, at 08:36, Stefan Schmiedl wrote:

--
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






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

Alexandre Bergel wrote:
> This is what we've been doing for long time...
>
> Alexandre
>
>
>  
Yes I know, that's why it is so annoying.

Keith


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

Alexandre Bergel wrote:
> The temporary variable is there in case that Installer is not  
> installed. It assumes that ScriptLoader is present.
>
> Cheers,
> Alexandre
>  
Installer is intended to be the first and only dependency in a minimal
system.

i.e. you are using ScriptLoader to do Installer's job

Keith

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

On Mon, May 11, 2009 at 07:58:48AM +0200, St?phane Ducasse wrote:
I added that ability to Installer ages ago. Installer cache
install: 'Morphic'.

Or you want it to search the cache without changing the script?
hmm. good idea. I'll make that work

--
Matthew Fulmer -- http://mtfulmer.wordpress.com/

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

>
> I added that ability to Installer ages ago. Installer cache
> install: 'Morphic'.

Ok I was not aware of that.

>
>
> Or you want it to search the cache without changing the script?
> hmm. good idea. I'll make that work

Yeap :)
something like

Installer install: 'Morphic'

and a default to not use the cache to do a real reload

Installer ignoreCache install: ....

Stef

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