- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] test failures in downloaded image 30733

by Marcus Denker

On 02 Feb 2014, at 16:27, btc(a)openinworld.com wrote: > Thanks for your feedback Marcus. A couple of thought inline... > > Marcus Denker wrote: >> >> On 02 Feb 2014, at 14:31, btc(a)openinworld.com wrote: >> >> >>> For diligence and curiosity leading up to the Pharo3 release, I downloaded build image 30733 (with PharoLauncher) >>> >> Another wonderful thing is that tests are fine when run individually, but fail when running all tests. >> > > It seems that when clicking on the failed test, the test is re-run and that result is debugged - but sometimes the error doesn't occur. It would be great the context of the error in the original execution was stored so that actual error could be inspected. Indeed, (to dream) presuming you're working off fresh CI build, it would be cool if using Fuel in one step you send the context of the original error (with CI build meta-info attached) to someone else's PharoLauncher, which automatically downloads the required CI build, and launches with the Fuel file loaded and ready to debug. > On the ci server this is already done: failing tests are serialise with Fuel for debugging. In addition, the image is save and can be downloaded (in the state after the test run). >> We need to think about automatic UI testingâ¦ >> >> Loading an external package changes *everything*. We do not have a a release criterium that all tests are green after loading external packages. >> > > No external packages loaded here. Launcher? Itâs external in the sense that this is not code that is in the image when we run all tests on the build server. > btw, What was your OS environment? > Mac >>> 3. Assuming Pharo3 will go through a Release Candidate phase, >>> >> >> Normally the idea is that we will just release. In the past we used an elaborate process, but the reality is that nobody looks at release candidates. >> You can tell 5 times that the release candidate will be released unchanged and people should check: they will not. They will download >> the release, though, and then complain that the âobviousâ thing X is not fixed that that the people who managed the release process did >> everything wrong. Because âreleaseâ means bug free. By magic. >> >> >> > I've seen that happening previously. In trying to understand the reality of this, perhaps... > * "check if this is okay" is an open ended question that doesn't have a deliverable to drive people to action. How do you know when its time to report success/failure? > * people do download and try it but its only shallow testing > * people only report by exception. > > Perhaps a more defined task such as "Run All Tests" is easier for people to do (single button press) and to report success/failure. PharoLaucher also makes it easier and quicker to do this. You don't need to call it a Release Candidate. Just have a specified version that multiple people run in multiple environments. Of course, that could also open up a can of worms of being hard from your end to dealing every test failure due to other people's environments. There would need a pragmatic approach about which ones to deal with to stay on schedule with the release date. > yesâ¦ its important that we don burn ourselves while doing the release, that is why the idea was to move as much things upfront as possible (âthe build is the release unchangedâ) Marcus

Feb. 3, 2014

[pharo-project/pharo-core]

by GitHub

Branch: refs/tags/30736 Home: https://github.com/pharo-project/pharo-core

Feb. 3, 2014

[pharo-project/pharo-core] 857012: 30736

by GitHub

Branch: refs/heads/3.0 Home: https://github.com/pharo-project/pharo-core Commit: 85701269c487dc49fa2fd4f7c6b1f35593baf903 https://github.com/pharo-project/pharo-core/commit/85701269c487dc49fa2fd4f7… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2014-02-02 (Sun, 02 Feb 2014) Changed paths: A Nautilus.package/PackageTreeNautilusUI.class/instance/package/alertCategoryName_exitstsAlreadyIn_.st A Nautilus.package/PackageTreeNautilusUI.class/instance/package/category_existsIn_.st M Nautilus.package/PackageTreeNautilusUI.class/instance/private dispatch/renameClassCategory_.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - scripts/script389.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - updates/update30736.st M ScriptLoader30.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st A Slot.package/BitsLayout.class/instance/testing/isBytes.st A Slot.package/BitsLayout.class/instance/testing/isWords.st A Slot.package/ByteLayout.class/instance/testing/isBytes.st A Slot.package/WordLayout.class/instance/testing/isBytes.st A SlotTests.package/SlotAnnouncementsTest.class/instance/tests-integration/testFixedClassWhenTraitCompositionChangedShouldAnnounceClassModified.st A SlotTests.package/SlotAnnouncementsTest.class/instance/tests-integration/testFixedClassWithTraitCreatedShouldntAnnounceClassModified.st A SlotTests.package/SlotAnnouncementsTest.class/instance/tests-integration/testVariableByteClassWhenTraitCompositionChangedShouldAnnounceClassModified.st A SlotTests.package/SlotAnnouncementsTest.class/instance/tests-integration/testVariableByteClassWithTraitCreatedShouldntAnnounceClassModified.st A SlotTests.package/SlotAnnouncementsTest.class/instance/tests-integration/testVariableClassWhenTraitCompositionChangedShouldAnnounceClassModified.st A SlotTests.package/SlotAnnouncementsTest.class/instance/tests-integration/testVariableClassWithTraitCreatedShouldntAnnounceClassModified.st A SlotTests.package/SlotAnnouncementsTest.class/instance/tests-integration/testVariableWordClassWhenTraitCompositionChangedShouldAnnounceClassModified.st A SlotTests.package/SlotAnnouncementsTest.class/instance/tests-integration/testVariableWordClassWithTraitCreatedShouldntAnnounceClassModified.st A SlotTests.package/SlotAnnouncementsTest.class/instance/tests-integration/testWeakClassWhenTraitCompositionChangedShouldAnnounceClassModified.st A SlotTests.package/SlotAnnouncementsTest.class/instance/tests-integration/testWeakClassWithTraitCreatedShouldntAnnounceClassModified.st M SlotTests.package/TOne.class/README.md M SlotTests.package/TTwo.class/README.md M SlotTests.package/TestSharedPool.class/README.md M System-Announcements.package/ClassModifiedClassDefinition.class/README.md A Traits.package/TClass.class/instance/subclass creation - private/subclass_uses_with_.st M Traits.package/TClass.class/instance/subclass creation/subclass_uses_instanceVariableNames_classVariableNames_poolDictionaries_category_.st M Traits.package/TClass.class/instance/subclass creation/variableByteSubclass_uses_instanceVariableNames_classVariableNames_poolDictionaries_category_.st M Traits.package/TClass.class/instance/subclass creation/variableSubclass_uses_instanceVariableNames_classVariableNames_poolDictionaries_category_.st M Traits.package/TClass.class/instance/subclass creation/variableWordSubclass_uses_instanceVariableNames_classVariableNames_poolDictionaries_category_.st M Traits.package/TClass.class/instance/subclass creation/weakSubclass_uses_instanceVariableNames_classVariableNames_poolDictionaries_category_.st Log Message: ----------- 30736 12727 Duplicated categories in Nautilus https://pharo.fogbugz.com/f/cases/12727 12157 Wrong announcement of ClassModifiedClassDefinition https://pharo.fogbugz.com/f/cases/12157 11670 strange MC package Generated-code-non-existing-package https://pharo.fogbugz.com/f/cases/11670 http://files.pharo.org/image/30/30736.zip

Feb. 3, 2014

Re: [Pharo-dev] Comparing dates

by Damien Cassou

Please use #assert:equals (that won't change the result though :-)) On Feb 1, 2014 12:04 AM, "Sebastian Sastre" <sebastian(a)flowingconcept.com> wrote: > self assert: Date today = Date today. > > today := Date today translateTo: 2 hours. > self assert: today = Date today > > Is this right? > > > > >

Feb. 3, 2014

WhatsUp from: 2014-02-03 until: 2014-02-16

by seaside＠rmod.lille.inria.fr

Hi! We're sending this automatic email twice a month, to give the community an opportunity to easily know what's happening and to coordinate efforts. Just answer informally, and feel free to spawn discussions thereafter! ### Here's what I've been up to since the last WhatsUp: - $HEROIC_ACHIEVEMENTS_OR_DISMAL_FAILURES_OR_SIMPLE_BORING_NECESSARY_TASKS ### What's next, until 2014-02-16 (*): - $NEXT_STEPS_TOWARDS_WORLD_DOMINATION (*) we'll be expecting results by then ;)

Feb. 3, 2014

leadingChar and friends

by Pharo4Stef

During my journey to the leadingChar realm I took notes and I share them with you. leadingChar: leadChar code: code code >= 16r400000 ifTrue: [ self error: 'code is out of range'. ]. leadChar >= 256 ifTrue: [ self error: 'lead is out of range'. ]. code < 256 ifTrue: [ ^self value: code ]. ^self value: (leadChar bitShift: 22) + code. charCode ^ (value bitAnd: 16r3FFFFF). leadingChar ^ (value bitAnd: (16r3FC00000)) bitShift: -22. characterSet ^ EncodedCharSet charsetAt: self leadingChar => a character encodes the characterSet. ============================ Why are we using Latin1>>leadingChar ^ 0. Unicode>>leadingChar ^ 0 and I do not get why GreekEnvironment>>leadingChar ^0 Latin2Environment>>leadingChar ^0 Latin1Environment>>leadingChar ^0 Latin9Environment>>leadingChar ^0 RussianEnvironment>>leadingChar ^0 SimplifiedChineseEnvironment>>leadingChar ^0 ====================== I do not understand why Unicode is declared as 1 and not 0. Unicode class>>initialize EncodedCharSet declareEncodedCharSet: self atIndex: 0+1. EncodedCharSet declareEncodedCharSet: self atIndex: 256. ================================ I do not understand why Latin1 does not use declareEncodedCharSet Latin1 class>>initialize " self initialize " compoundTextSequence := String streamContents: [ :s | s nextPut: (Character value: 27). s nextPut: $(. s nextPut: $B ]. rightHalfSequence := String streamContents: [ :s | s nextPut: (Character value: 27). s nextPut: $-. s nextPut: $A ] I started to distribute the initialization into subclasses starting from this method: declareEncodedCharSet: anEncodedCharSetOrLanguageEnvironmentClass atIndex: aNumber "this method is used to modularize the old initialize method: EncodedCharSets at: 0+1 put: Unicode. EncodedCharSets at: 1+1 put: JISX0208. EncodedCharSets at: 2+1 put: GB2312. EncodedCharSets at: 3+1 put: KSX1001. EncodedCharSets at: 4+1 put: JISX0208. EncodedCharSets at: 5+1 put: JapaneseEnvironment. EncodedCharSets at: 6+1 put: SimplifiedChineseEnvironment. EncodedCharSets at: 7+1 put: KoreanEnvironment. EncodedCharSets at: 8+1 put: GB2312. EncodedCharSets at: 12+1 put: KSX1001. EncodedCharSets at: 13+1 put: GreekEnvironment. EncodedCharSets at: 14+1 put: Latin2Environment. EncodedCharSets at: 15+1 put: RussianEnvironment. EncodedCharSets at: 17+1 put: Latin9Environment. EncodedCharSets at: 256 put: Unicode. and indeed Latin1Environment was not part of the list. Now apparently we can remove Latin1 because EncodedCharSets of EncodedCharSet do not contain Latin1 ================================== No senders emitSequenceToResetStateIfNeededOn: aStream forState: state rightDirection Funny nextPutRightHalfValue: ascii toStream: aStream withShiftSequenceIfNeededForTextConverterState: state nextPutValue: ascii toStream: aStream withShiftSequenceIfNeededForTextConverterState: state ========================== Ideas of cleaning steps step one: what if Character>>leadingChar ^ 0 We can probably kill the encodedSet. It looks like Latin1Environment and Latin1 are not used. Stef

Feb. 2, 2014

Re: [Pharo-dev] [Seaside] Re: ConfigurationOfGetText broken?

by Johan Brichau

Hi Paul, Gettext for Seaside is now included in the ConfigurationOfSeaside3 #bleedingEdge baseline. I propose to move the Gettext-Slime package to the Seaside30Addons repository, where the Seaside-Gettext is hosted too. Since I'm busy making modifications to Gettext for Pharo3 anyway, I will create a new baseline here too, if that sounds fine for you. cheers, Johan On 18 Jan 2014, at 18:17, Paul DeBruicker <pdebruic(a)gmail.com> wrote: > Sounds fine to me. > > > Thanks for sorting it out. > > > Paul > > > > Johan Brichau-2 wrote >> Fyi I am already adding gettext for seaside to the >> ConfigurationOfSeaside3. >> >> So, it's good to clean ConfigurationOfGettext >> >> Johan (sent from my mobile) >> >>> On 18 Jan 2014, at 13:52, Usman Bhatti < > >> usman.bhatti@ > >> > wrote: >>> >>> We've already discussed it and I am in the favor of removing non-gettext >>> dependancies from its config. >>> http://lists.squeakfoundation.org/pipermail/seaside/2013-October/030221.html >>> >>> Now, I'll wait for Paul's response to cleanup. Because debugging >>> ConfigurationOfGetText or for that matter any configuration with such >>> complexities takes a lot of time. So, as an ad-hoc solution I am loading >>> the only package in Gettext directly. >>> >>> >>> Usman >>> >>> >>>> On Sat, Jan 18, 2014 at 6:24 AM, Johan Brichau < > >> johan@ > >> > wrote: >>>> I agree. >>>> The Seaside packages should be moved to the Seaside configuration and >>>> make that one dependent on ConfigurationOfGettext. >>>> Also, the Seaside30Addons repository was moved to smalltalkhub. >>>> >>>> Johan >>>> >>>> On 17 Jan 2014, at 17:17, Hilaire Fernandes < > >> hilaire.fernandes@ > >> > wrote: >>>> >>>>> Le 17/01/2014 11:42, Usman Bhatti a Ã©crit : >>>>>> >>>>>> It seems that some recent changes in the ConfigurationOfText have >>>> broken >>>>>> the loading in the Pharo 3.0. >>>>>> >>>>> >>>>> >>>>> I only know basic of ConfigurationOf..., but it looks odd to me to see >>>>> Seaside dependencies in the Gettext configuration. Should it not >>>> belong >>>>> to this configuration? Imagine if each package depending on Gettext to >>>>> add itself in the gettext configuration? >>>>> >>>>> Hilaire >>>>> >>>>> -- >>>>> Dr. Geo http://drgeo.eu >>>>> >>>>> >>> >>> _______________________________________________ >>> seaside mailing list >>> > >> seaside@.squeakfoundation > >>> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside > > > > > > -- > View this message in context: http://forum.world.st/ConfigurationOfGetText-broken-tp4737466p4737765.html > Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com. >

Feb. 2, 2014

Re: [Pharo-dev] ZTimestamp>>monthName spelling error

by Sabine KnÃ¶fel

Thank you for the fix, Sven. It works fine now. Sabine On Sat, Feb 1, 2014 at 5:26 PM, Sven Van Caekenberghe <sven(a)stfx.eu> wrote: > Sabine, > > On 01 Feb 2014, at 15:15, Sabine KnÃ¶fel <sabine.knoefel(a)gmail.com> wrote: > > > Hi Sven, > > > > thank you for the quick fix. It works fine now. > > > > I do not call this directly but from > > DateAndTime date: self endDate time: self endTime > > > > Your example: > > I am no english native speaker but I assume, that 'saterday' should be > spelled as 'saturday'. > > If yes, there are several occurences of saterday. > > Terrible spelling of me! > > === > Name: ZTimestamp-SvenVanCaekenberghe.33 > Author: SvenVanCaekenberghe > Time: 1 February 2014, 5:18:50.139922 pm > UUID: 40111894-d387-432a-a3e1-2245af4a269b > Ancestors: ZTimestamp-SvenVanCaekenberghe.32 > > Even worse: Saterday => Saturday (thx Sabine KnÃ¶fel) > === > > ConfigurationOfZTimestamp has been updated everywhere. > > > I saw this when trying your example: > > > > (ZTimestampFormat fromString: 'Saterday, 3 February, 2001') > > german; > > format: Date today. ==>>'Samstag, 1 Februar, 2014' > > > > (ZTimestampFormat fromString: 'Saturday, 3 February, 2001') > > german; > > format: Date today. ==>>'Samurday, 1 Februar, 2014' > > > > Also, 'Samurday' is no german word. 'Saturday' means 'Samstag' in German. > > Now that example should work: > > (ZTimestampFormat fromString: 'Saturday, 3 February, 2001') > german; > format: Date today. > > => 'Samstag, 1 Februar, 2014' > > Thanks for the feedback. > > Sven > > > Regards > > Sabine > > > > > > On Sat, Feb 1, 2014 at 1:31 PM, Sven Van Caekenberghe <sven(a)stfx.eu> > wrote: > > Sabine, > > > > On 01 Feb 2014, at 12:52, Sabine KnÃ¶fel <sabine.knoefel(a)gmail.com> > wrote: > > > > > Hi Sven, > > > > > > could you change Februari into February please? > > > > Ah, terrible typo, it was all over the place. > > > > === > > Name: ZTimestamp-SvenVanCaekenberghe.32 > > Author: SvenVanCaekenberghe > > Time: 1 February 2014, 1:27:02.50151 pm > > UUID: 70fd3b88-e15e-4139-922c-9c602e3fc532 > > Ancestors: ZTimestamp-SvenVanCaekenberghe.31 > > > > The English spelling of the 2nd month name was consistently wrong: > Februari instead of the correct February (thx Sabine KnÃ¶fel); > > Now use #monthIndex instead of the less consistent #month > > === > > > > Btw, did you know you can do: > > > > (ZTimestampFormat fromString: 'Saterday, 3 February, 2001') > > german; > > format: Date today. > > > > => 'Samstag, 1 Februar, 2014' > > > > Regards, > > > > Sven > > > > > (I use http://mc.stfx.eu/Neo, this is the right place?) > > > > > > Regards > > > Sabine > > > > > > ZTimestamp>>monthName > > > "Return the English name of the month of the receiver, according > to the > > > Gregorian calendar, a Symbol" > > > > > > ^ #(January Februari March April May June July August September > October > > > November December) at: self monthIndex > > > > > > > > > > > > -- > > > View this message in context: > http://forum.world.st/ZTimestamp-monthName-spelling-error-tp4740821.html > > > Sent from the Pharo Smalltalk Developers mailing list archive at > Nabble.com. > > > > > > > > > > > >

Feb. 2, 2014

Re: [Pharo-dev] Can someone help me with this issue 12469 (Sometimes an empty protocol is created)

by Sven Van Caekenberghe

Thank you, Nicolai, for the fix ! On 02 Feb 2014, at 19:12, Nicolai Hess <nicolaihess(a)web.de> wrote: > Thank you. > > > 2014-02-01 Sven Van Caekenberghe <sven(a)stfx.eu>: > > On 01 Feb 2014, at 23:03, Sean P. DeNigris <sean(a)clipperadams.com> wrote: > > > Nicolai Hess wrote > >> Every time the slice marks the all classes and methods as changed, > > > > I've experienced that when viewing changes for a whole slice. If you view > > the changes for each package individually instead, you should see it is > > fine. > > Yeah, it seems to be a fluke, very confusing though. > > > I see Sven has created a new slice for the monkey... > > And it seemed to have passed. > >

Feb. 2, 2014

Re: [Pharo-dev] Can someone help me with this issue 12469 (Sometimes an empty protocol is created)

by Nicolai Hess

Thank you. 2014-02-01 Sven Van Caekenberghe <sven(a)stfx.eu>: > > On 01 Feb 2014, at 23:03, Sean P. DeNigris <sean(a)clipperadams.com> wrote: > > > Nicolai Hess wrote > >> Every time the slice marks the all classes and methods as changed, > > > > I've experienced that when viewing changes for a whole slice. If you view > > the changes for each package individually instead, you should see it is > > fine. > > Yeah, it seems to be a fluke, very confusing though. > > > I see Sven has created a new slice for the monkey... > > And it seemed to have passed. > >

Feb. 2, 2014