WADevelopment loadPackages: WADevelopment allPackages<br><br>always loads all packages currently defined in GRPackage.<br>Mostly I dont want to update all packages at the same time.<br><br>E.g. I would prefer to update only the seaside core packages,<br>
and not e.g. Magritte or some seaside addOns.<br>So adding an instVar to GRPackage would be an option.<br>Or maybe adding a subclass to GRPackage.<br><br>Gerhard<br><br><br><br><br><div class="gmail_quote">On Sun, Feb 21, 2010 at 11:12 PM, Julian Fitzell <span dir="ltr"><<a href="mailto:jfitzell@gmail.com">jfitzell@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">Yeah, just because it's being stored in the Seaside repo, doesn't mean<br>
it's a "seaside basic package" and, conversely, some seaside packages<br>
are not in that repo.<br>
<br>
If this is really needed, we should add proper meta-data that<br>
specifies a group or a project or something. But I'm still not really<br>
happy with WAPackage and reluctant to keep adding functionality to it.<br>
:)<br>
<font color="#888888"><br>
Julian<br>
</font><div><div></div><div class="h5"><br>
On Sun, Feb 21, 2010 at 2:07 PM, Lukas Renggli <<a href="mailto:renggli@gmail.com">renggli@gmail.com</a>> wrote:<br>
> What about<br>
><br>
> WADevelopment loadPackages: WADevelopment allPackages<br>
><br>
> ?<br>
><br>
> We should not misuse internal state for unrelated things. This<br>
> happened before with the license that was used to find the the<br>
> repository URL, which was really ugly to fix.<br>
><br>
> Lukas<br>
><br>
> On 21 February 2010 20:43, Gerhard Obermann <<a href="mailto:obi068@gmail.com">obi068@gmail.com</a>> wrote:<br>
>> Because I want to do something like this:<br>
>><br>
>> WADevelopment allPackages select: [:each | each urlSelector = #seasideUrl ]<br>
>><br>
>> I need all seaside basic packages to be able to update the packages using<br>
>> Gofer.<br>
>><br>
>> Gerhard<br>
>><br>
>> On Sun, Feb 21, 2010 at 6:56 PM, Lukas Renggli <<a href="mailto:renggli@gmail.com">renggli@gmail.com</a>> wrote:<br>
>>><br>
>>> Is there a particular reason that you added this method?<br>
>>><br>
>>> As you can see in the method comment of #url: the variable can also<br>
>>> contain a string, so your method does not necessarily return what it<br>
>>> says.<br>
>>><br>
>>> Also consider adding a method comment and tests for any public method.<br>
>>><br>
>>> Lukas<br>
>>><br>
>>> On 21 February 2010 18:41,<br>
>>> <<a href="mailto:squeak-dev-noreply@lists.squeakfoundation.org">squeak-dev-noreply@lists.squeakfoundation.org</a>> wrote:<br>
>>> > Gerhard Obermann uploaded a new version of Grease-Core to project<br>
>>> > Seaside 3.0:<br>
>>> > <a href="http://www.squeaksource.com/Seaside30/Grease-Core-obi.29.mcz" target="_blank">http://www.squeaksource.com/Seaside30/Grease-Core-obi.29.mcz</a><br>
>>> ><br>
>>> > ==================== Summary ====================<br>
>>> ><br>
>>> > Name: Grease-Core-obi.29<br>
>>> > Author: obi<br>
>>> > Time: 21 February 2010, 6:43:17 pm<br>
>>> > UUID: 79473ec4-083e-a144-aa40-4d100d2ab1e3<br>
>>> > Ancestors: Grease-Core-lr.28<br>
>>> ><br>
>>> > - add direct accessing method for url<br>
>>> ><br>
>>> > =============== Diff against Grease-Core-lr.28 ===============<br>
>>> ><br>
>>> > Item was added:<br>
>>> > + ----- Method: GRPackage>>urlSelector (in category 'accessing') -----<br>
>>> > + urlSelector<br>
>>> > + ^ url!<br>
>>> ><br>
>>> ><br>
>>> > _______________________________________________<br>
>>> > commits mailing list<br>
>>> > To unsubscribe, email <a href="mailto:commits-leave@lists.seaside.st">commits-leave@lists.seaside.st</a><br>
>>> > <a href="http://lists.seaside.st/listinfo/commits" target="_blank">http://lists.seaside.st/listinfo/commits</a><br>
>>> ><br>
>>><br>
>>><br>
>>><br>
>>> --<br>
>>> Lukas Renggli<br>
>>> <a href="http://www.lukas-renggli.ch" target="_blank">http://www.lukas-renggli.ch</a><br>
>>> _______________________________________________<br>
>>> seaside-dev mailing list<br>
>>> <a href="mailto:seaside-dev@lists.squeakfoundation.org">seaside-dev@lists.squeakfoundation.org</a><br>
>>> <a href="http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev" target="_blank">http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev</a><br>
>><br>
>><br>
>> _______________________________________________<br>
>> seaside-dev mailing list<br>
>> <a href="mailto:seaside-dev@lists.squeakfoundation.org">seaside-dev@lists.squeakfoundation.org</a><br>
>> <a href="http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev" target="_blank">http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev</a><br>
>><br>
>><br>
><br>
><br>
><br>
> --<br>
> Lukas Renggli<br>
> <a href="http://www.lukas-renggli.ch" target="_blank">http://www.lukas-renggli.ch</a><br>
> _______________________________________________<br>
> seaside-dev mailing list<br>
> <a href="mailto:seaside-dev@lists.squeakfoundation.org">seaside-dev@lists.squeakfoundation.org</a><br>
> <a href="http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev" target="_blank">http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev</a><br>
><br>
_______________________________________________<br>
seaside-dev mailing list<br>
<a href="mailto:seaside-dev@lists.squeakfoundation.org">seaside-dev@lists.squeakfoundation.org</a><br>
<a href="http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev" target="_blank">http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev</a><br>
</div></div></blockquote></div><br>