gst-package

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

gst-package

MrGwen
Hi,

when you launch gst-package without a parameter it raises an exception.
Here is a very small patch to change this behavior and display the help
message :)

Cheers,
Gwen

_______________________________________________
help-smalltalk mailing list
[hidden email]
http://lists.gnu.org/mailman/listinfo/help-smalltalk

package-help.patch (9K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: gst-package

Paolo Bonzini-2
On 01/01/2010 04:47 PM, Gwenael Casaccio wrote:
> Hi,
>
> when you launch gst-package without a parameter it raises an exception.
> Here is a very small patch to change this behavior and display the help
> message :)

Applied, will push soon.

Paolo


_______________________________________________
help-smalltalk mailing list
[hidden email]
http://lists.gnu.org/mailman/listinfo/help-smalltalk
Reply | Threaded
Open this post in threaded view
|

Re: gst-package

MrGwen
Hi,

I'm working an a new command for displaying packages inside a
repository. Basicly it will download a file (repository.xml) and parse
it so we have the list of packages inside a repository. The file will be
simple (<repository><package>name</package>...</repository>)

I would like to make some small changes to the pckloader.st create a new
namespace Package instead of Kernel ?? and split the one file in
multiple file (or directory). Do you agree or not :D ?

Gwen
 
On Fri, 2010-01-01 at 18:33 +0100, Paolo Bonzini wrote:

> On 01/01/2010 04:47 PM, Gwenael Casaccio wrote:
> > Hi,
> >
> > when you launch gst-package without a parameter it raises an exception.
> > Here is a very small patch to change this behavior and display the help
> > message :)
>
> Applied, will push soon.
>
> Paolo
>
>
> _______________________________________________
> help-smalltalk mailing list
> [hidden email]
> http://lists.gnu.org/mailman/listinfo/help-smalltalk




_______________________________________________
help-smalltalk mailing list
[hidden email]
http://lists.gnu.org/mailman/listinfo/help-smalltalk
Reply | Threaded
Open this post in threaded view
|

Re: gst-package

Paolo Bonzini-2
On 01/02/2010 01:43 PM, Gwenael Casaccio wrote:
> Hi,
>
> I'm working an a new command for displaying packages inside a
> repository. Basicly it will download a file (repository.xml) and parse
> it so we have the list of packages inside a repository.

Uhm, you want me to do that in PHP too for smalltalk.gnu.org... ;-)

> I would like to make some small changes to the pckloader.st create a new
> namespace Package instead of Kernel ?? and split the one file in
> multiple file (or directory). Do you agree or not :D ?

Naming the namespace Package is a problem because you'd have a Package
class within a same-name namespace.  I agree that putting them in Kernel
is suboptimal.  For 3.2 I'd rather keep things as is, we have to start
the stabilization somewhere.

Paolo


_______________________________________________
help-smalltalk mailing list
[hidden email]
http://lists.gnu.org/mailman/listinfo/help-smalltalk
Reply | Threaded
Open this post in threaded view
|

Re: gst-package

MrGwen
On Sat, 2010-01-02 at 13:46 +0100, Paolo Bonzini wrote:
> On 01/02/2010 01:43 PM, Gwenael Casaccio wrote:
> > Hi,
> >
> > I'm working an a new command for displaying packages inside a
> > repository. Basicly it will download a file (repository.xml) and parse
> > it so we have the list of packages inside a repository.
>
> Uhm, you want me to do that in PHP too for smalltalk.gnu.org... ;-)

;)

>
> > I would like to make some small changes to the pckloader.st create a new
> > namespace Package instead of Kernel ?? and split the one file in
> > multiple file (or directory). Do you agree or not :D ?
>
> Naming the namespace Package is a problem because you'd have a Package
> class within a same-name namespace.  I agree that putting them in Kernel
> is suboptimal.  For 3.2 I'd rather keep things as is, we have to start
> the stabilization somewhere.
>
> Paolo
So here is the first version of the patch :
  I've made a tiny XML parser (very very simple ^^), a new
PckRepositoryList command and a new class PackageRepository which stores
packages from a repository. Any comments, remarks  are welcomes

 I've found also a minor "bug" when I've bootstraped the image if there
are errors the compiler display nothing, may be an improvement for the
3.3 ;).

Gwen

_______________________________________________
help-smalltalk mailing list
[hidden email]
http://lists.gnu.org/mailman/listinfo/help-smalltalk

package-repository.patch (14K) Download Attachment