And before moving classes from one package to another, please consider my small contribution in inbox for cleaning undeclared.<br><br>Nicolas<br><br><div class="gmail_quote">Le 28 mars 2012 13:07, Bert Freudenberg <span dir="ltr"><<a href="mailto:bert@freudenbergs.de">bert@freudenbergs.de</a>></span> a écrit :<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 28.03.2012, at 11:12, Edgar J. De Cleene wrote:<br>
<br>
> On 3/27/12 10:07 PM, "Chris Muller" <<a href="mailto:ma.chris.m@gmail.com">ma.chris.m@gmail.com</a>> wrote:<br>
><br>
>> which now blows up our image build<br>
><br>
> Not doing update from 11925.image.<br>
> Just now I reach 11959 and have yours GetText-cmm.19<br>
><br>
> Still have Undeclared and still none answers to my questions about which of<br>
> Etoys we want in trunk.<br>
<br>
</div>Hi Edgar,<br>
<br>
I didn't know you were waiting on someone. Well, time for a longer message then.<br>
<br>
We want to abandon the Etoys code fork. At some point in the future, we want to build an Etoys release from a Trunk image.<br>
<br>
So "which of Etoys"? All that is needed to load and use current Etoys projects nicely. A trunk-based Etoys release would need to work at least as well as the previous versions (and will hopefully be better).<br>
<br>
It doesn't have to work exactly the same, though. It should be cleaner and faster :)<br>
<br>
As for GetText, maybe the best would be to not have this in a separate package. At the very least we need to move Locale etc. back into the "System" package. In Etoys, GetText had 7 classes, in Trunk you made it 15. The idea was to have NaturalLanguageTranslator as abstract superclass for translation services, and have the old InternalTranslator and the new GetTextTranslator as concrete subclasses.<br>
<br>
We need to decide if keeping the old InternalTranslator around is necessary. I don't know anyone who is using it, so my suggestion would be to discard it.<br>
<br>
Here is my break-down of the classes currently in your GetText package:<br>
<br>
System<br>
Locale - move back to System pkg<br>
LocaleID - move back to System pkg<br>
ISOLanguageDefinition - move back to System pkg<br>
<br>
Translation core (needed at runtime)<br>
NaturalLanguageTranslator - abstract superclass<br>
NaturalLanguageFormTranslator - for translatable bitmaps<br>
GetTextTranslator - looks up translation in MO file<br>
MOFile - parses MO files<br>
TextDomainManager - needed at runtime<br>
<br>
Utilities (not needed at runtime)<br>
GetTextExporter: collects strings from image and writes out POT files<br>
TranslatedReceiverFinder - needed by exporter<br>
GetTextInterchange - superclass for importer<br>
GetTextImporter - loads PO files, only used by exporter and editor<br>
LanguageEditor - interface for editing translations (not currently used, but may be useful)<br>
GetTextExporter2 - delete (I wonder why is this back?)<br>
<br>
Here's the current sizes:<br>
<br>
{NaturalLanguageTranslator. NaturalLanguageFormTranslator. GetTextTranslator. MOFile. TextDomainManager} collect: #linesOfCode<br>
==> #(418 49 135 223 60)<br>
<br>
{GetTextExporter. TranslatedReceiverFinder. GetTextInterchange. GetTextImporter. LanguageEditor} collect: #linesOfCode<br>
==> #(408 190 8 163 1073)<br>
<br>
The Locale classes need to move back to the System package. My suggestion would be to move the "Translation core" classes into the System package as well, and keep only the "Utilities" in the GetText package, which then can easily be unloaded.<br>
<br>
And please clean up the GetTextExporter/GetTextExporter2 mess. I have no idea how that happened. You should have started from the latest Etoys package, not from some old package. I have a hunch you used Hilaire's GetText package for Pharo? This is apparently based on old Etoys code.<br>
<br>
Here is a diff between your GetText-edc.15 and Etoys' GetText-Richo.14 (you can see it in Monticello's history tool). Please remove the unneeded methods in GetTextExporter and TranslatedReceiverFinder, and delete the GetTextExporter2 class.<br>
<br>
GetTextExporter>>createHeaders (changed and recategorized)<br>
GetTextExporter>>export:<br>
GetTextExporter>>exportHeader<br>
GetTextExporter>>exportTranslations<br>
GetTextExporter>>exportUntranslated<br>
GetTextExporter2 (added class)<br>
ISOLanguageDefinition (added class)<br>
InternalTranslator (added class)<br>
LanguageEditor (added class)<br>
Locale (added class)<br>
LocaleID (added class)<br>
NaturalLanguageFormTranslator (added class)<br>
NaturalLanguageTranslator (added class)<br>
String>>translatedInAnyDomain<br>
TextDomainManager class>>initialize (changed)<br>
TranslatedReceiverFinder class>>makeJapaneseTranslationFile<br>
TranslatedReceiverFinder>>searchBlockNode:addTo:<br>
TranslatedReceiverFinder>>searchMessageNode:addTo:<br>
TranslatedReceiverFinder>>searchMethodNode:addTo:<br>
TranslatedReceiverFinder>>searchReturnNode:addTo:<br>
TranslatedReceiverFinder>>senders<br>
<div class="im"><br>
> And the proposal of have a directory on the ftp with all needed files for<br>
> loading if user wish.<br>
<br>
<br>
</div>I previously wrote "put it in the 4.4alpha directory".<br>
<br>
So ... is this enough to unblock you? Thanks for doing it :)<br>
<span class="HOEnZb"><font color="#888888"><br>
- Bert -<br>
<br>
<br>
<br>
</font></span></blockquote></div><br>