May 2015 - Pharo-dev - lists.pharo.org

Selected context specific debugger action
by Yuriy Tymchuk May 8, 2015

May 8, 2015

Hi, is it possible to have a debugger action specific for selected context? I.e. define label, icon, whether the action is visible depending on the selected context? Uko

5 9

[pharo-project/pharo-core]
by GitHub May 8, 2015

May 8, 2015

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

1 0

[pharo-project/pharo-core] 1e0add: 50040
by GitHub May 8, 2015

May 8, 2015

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 1e0add797028c9fcac04d1a4a2bffcc4feeeb75f https://github.com/pharo-project/pharo-core/commit/1e0add797028c9fcac04d1a4… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-05-08 (Fri, 08 May 2015) Changed paths: M Collections-Sequenceable.package/SharedQueue.class/instance/printing/printOn_.st A IssueTracking-Tests.package/ManifestIssueTrackingTests.class/README.md A IssueTracking-Tests.package/ManifestIssueTrackingTests.class/definition.st A IssueTracking-Tests.package/ManifestIssueTrackingTests.class/instance/code-critics/ruleBadMessageRuleV1FalsePositive.st A IssueTracking-Tests.package/ManifestIssueTrackingTests.class/instance/code-critics/ruleVariableAssignedLiteralRuleV1FalsePositive.st A IssueTracking-Tests.package/PharoIssueTest.class/README.md A IssueTracking-Tests.package/PharoIssueTest.class/definition.st A IssueTracking-Tests.package/PharoIssueTest.class/instance/running/setUp.st A IssueTracking-Tests.package/PharoIssueTest.class/instance/tests-implementation/testHeaderString.st A IssueTracking-Tests.package/PharoIssueTest.class/instance/tests/testAutoTitle.st A IssueTracking-Tests.package/PharoIssueTest.class/instance/tests/testCreating.st A IssueTracking-Tests.package/PharoIssueTest.class/instance/tests/testNumber.st A IssueTracking-Tests.package/PharoIssueTest.class/instance/tests/testSettingTitle.st A IssueTracking-Tests.package/PharoIssueTest.class/instance/tests/testSlice.st A IssueTracking-Tests.package/PharoIssueTest.class/instance/tests/testTrackerUrl.st A IssueTracking.package/ManifestIssueTracking.class/README.md A IssueTracking.package/ManifestIssueTracking.class/definition.st A IssueTracking.package/ManifestIssueTracking.class/instance/code-critics/ruleMissingYourselfRuleV1FalsePositive.st A IssueTracking.package/NullPharoIssue.class/README.md A IssueTracking.package/NullPharoIssue.class/definition.st A IssueTracking.package/NullPharoIssue.class/instance/acccessing/title.st A IssueTracking.package/NullPharoIssue.class/instance/accessing/number.st A IssueTracking.package/PharoIssue.class/README.md A IssueTracking.package/PharoIssue.class/class/instance creation/number_.st A IssueTracking.package/PharoIssue.class/definition.st A IssueTracking.package/PharoIssue.class/instance/accessing/number.st A IssueTracking.package/PharoIssue.class/instance/accessing/number_.st A IssueTracking.package/PharoIssue.class/instance/accessing/slice.st A IssueTracking.package/PharoIssue.class/instance/accessing/title.st A IssueTracking.package/PharoIssue.class/instance/accessing/title_.st A IssueTracking.package/PharoIssue.class/instance/accessing/trackerUrl.st A IssueTracking.package/PharoIssue.class/instance/communicating/creationNotification.st A IssueTracking.package/PharoIssue.class/instance/communicating/sliceNotification.st A IssueTracking.package/PharoIssue.class/instance/private/changesFromSliceCommitMessage.st A IssueTracking.package/PharoIssue.class/instance/private/downloadTitle.st A IssueTracking.package/PharoIssue.class/instance/private/headerString.st A IssueTracking.package/PharoIssue.class/instance/private/sliceCommitMessage.st M Komitter.package/MCKomitSubmitter.class/instance/protocol/commitSlice_.st A Komitter.package/ManifestKomitter.class/README.md A Komitter.package/ManifestKomitter.class/definition.st A Komitter.package/ManifestKomitter.class/instance/code-critics/ruleMissingYourselfRuleV1FalsePositive.st A Komitter.package/extension/MCPackage/instance/package.st M Komitter.package/extension/MCRepositoryGroup/instance/remotes.st R Komitter.package/extension/MCSliceInfo/instance/makeKomitSlice.st A MonticelloGUI.package/MCSliceInfo.class/class/accessing/namePrefix.st M MonticelloGUI.package/MCSliceInfo.class/definition.st A MonticelloGUI.package/MCSliceInfo.class/instance/accessing-MC/mcPackageName.st A MonticelloGUI.package/MCSliceInfo.class/instance/accessing-MC/workingCopy.st M MonticelloGUI.package/MCSliceInfo.class/instance/accessing/includedPackages_.st A MonticelloGUI.package/MCSliceInfo.class/instance/accessing/issue.st M MonticelloGUI.package/MCSliceInfo.class/instance/accessing/issueNumber.st M MonticelloGUI.package/MCSliceInfo.class/instance/accessing/issueNumber_.st R MonticelloGUI.package/MCSliceInfo.class/instance/accessing/issueSummary.st R MonticelloGUI.package/MCSliceInfo.class/instance/accessing/issueSummary_.st A MonticelloGUI.package/MCSliceInfo.class/instance/accessing/title.st A MonticelloGUI.package/MCSliceInfo.class/instance/accessing/title_.st R MonticelloGUI.package/MCSliceInfo.class/instance/accessing/usableIssueNumber.st R MonticelloGUI.package/MCSliceInfo.class/instance/accessing/usableIssueSummary.st A MonticelloGUI.package/MCSliceInfo.class/instance/initialization/initialize.st A MonticelloGUI.package/MCSliceInfo.class/instance/input sanitizing/sanitizeIssueNumberFromString_.st A MonticelloGUI.package/MCSliceInfo.class/instance/input sanitizing/sanitizeIssueTitle_.st A MonticelloGUI.package/MCSliceInfo.class/instance/issue tracker/downloadIssueTitle.st A MonticelloGUI.package/MCSliceInfo.class/instance/issue tracker/setTitleFromIssue.st M MonticelloGUI.package/MCSliceInfo.class/instance/slice-making/makeSlice.st A MonticelloGUI.package/MCSliceInfo.class/instance/testing/canMakePackage.st R MonticelloGUI.package/MCSliceMaker.class/instance/accessing/issueIdString.st R MonticelloGUI.package/MCSliceMaker.class/instance/accessing/issueNumber.st R MonticelloGUI.package/MCSliceMaker.class/instance/actions/downloadIssueSummary.st R MonticelloGUI.package/MCSliceMaker.class/instance/actions/downloadIssueSummaryFailed.st A MonticelloGUI.package/MCSliceMaker.class/instance/actions/downloadIssueTitle.st M MonticelloGUI.package/MCSliceMaker.class/instance/actions/informFailedWith_.st A MonticelloGUI.package/MCSliceMaker.class/instance/private/cleanUp.st M MonticelloGUI.package/MCSliceMaker.class/instance/selecting/selectionChanged_.st R MonticelloGUI.package/MCSliceMaker.class/instance/selecting/update_.st A MonticelloGUI.package/MCSliceMaker.class/instance/updating/update_.st M MonticelloGUI.package/MCSliceMaker.class/instance/user interface/cancel.st M MonticelloGUI.package/MCSliceMaker.class/instance/user interface/issueNumberEntryOn_.st R MonticelloGUI.package/MCSliceMaker.class/instance/user interface/issueSummaryDownloaderButtonOn_.st R MonticelloGUI.package/MCSliceMaker.class/instance/user interface/issueSummaryOn_.st A MonticelloGUI.package/MCSliceMaker.class/instance/user interface/issueTitleOn_.st M MonticelloGUI.package/MCSliceMaker.class/instance/user interface/ok.st A MonticelloGUI.package/MCSliceMaker.class/instance/user interface/okEnabled.st R MonticelloGUI.package/MCSliceMaker.class/instance/user interface/okUnabled.st M MonticelloGUI.package/MCSliceMaker.class/instance/user interface/rootItems.st M MonticelloGUI.package/MCSliceMaker.class/instance/user interface/window.st M MonticelloGUI.package/MCWorkingCopyBrowser.class/instance/morphic ui/repositoryGroup.st A MonticelloGUI.package/ManifestMonticelloGUI.class/README.md A MonticelloGUI.package/ManifestMonticelloGUI.class/definition.st A Reflectivity.package/Breakpoint.class/README.md A Reflectivity.package/Breakpoint.class/class/cleanup/removeAll.st A Reflectivity.package/Breakpoint.class/class/instance creation/inNode_.st A Reflectivity.package/Breakpoint.class/definition.st A Reflectivity.package/Breakpoint.class/instance/accessing/node.st A Reflectivity.package/Breakpoint.class/instance/accessing/node_.st A Reflectivity.package/Breakpoint.class/instance/api/always.st A Reflectivity.package/Breakpoint.class/instance/api/now.st A Reflectivity.package/Breakpoint.class/instance/api/once.st A Reflectivity.package/Breakpoint.class/instance/api/when_.st A Reflectivity.package/Breakpoint.class/instance/as yet unclassified/break_withArguments_.st A Reflectivity.package/Breakpoint.class/instance/controll/disable.st A Reflectivity.package/Breakpoint.class/instance/controll/enable.st A Reflectivity.package/Breakpoint.class/instance/halt/haltNow.st A Reflectivity.package/Breakpoint.class/instance/install/installLink_.st A Reflectivity.package/Breakpoint.class/instance/install/remove.st A Reflectivity.package/Breakpoint.class/instance/links/conditionalHaltLink.st A Reflectivity.package/Breakpoint.class/instance/links/haltLink.st A Reflectivity.package/Breakpoint.class/instance/links/haltLinkOneShot.st M Reflectivity.package/ReflectiveMethod.class/instance/invalidate/destroyTwin.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script648.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script649.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50039.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50040.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st R SmartSuggestions.package/SugsAbstractContext.class/instance/accesing/isMethodSelected.st A SmartSuggestions.package/SugsAbstractContext.class/instance/accessing/sourceTextArea.st A SmartSuggestions.package/SugsAbstractContext.class/instance/accessing/window.st R SmartSuggestions.package/SugsAbstractContext.class/instance/refactoring/sourceTextArea.st R SmartSuggestions.package/SugsAbstractContext.class/instance/refactoring/window.st A SmartSuggestions.package/SugsAbstractContext.class/instance/testing/hasUnacceptedEdits.st A SmartSuggestions.package/SugsAbstractContext.class/instance/testing/isMethodSelected.st M SmartSuggestions.package/SugsMenuBuilder.class/class/private/findBestNodeFor_.st Log Message: ----------- 50040 15513 SmartSuggestions: use cached AST when editor is not dirty https://pharo.fogbugz.com/f/cases/15513 15491 Debugger hangs vm on step into https://pharo.fogbugz.com/f/cases/15491 15514 Add BreakPoint class and fix recompilation when removing the last link https://pharo.fogbugz.com/f/cases/15514 15507 SliceMaker Overhaul https://pharo.fogbugz.com/f/cases/15507 http://files.pharo.org/image/50/50040.zip

1 0

Re: [Pharo-dev] Penelope Doesn't Skip "Wrong" Lint Rules
by Ben Coman May 8, 2015

May 8, 2015

----- Sean P. DeNigris <sean(a)clipperadams.com> a Ã©crit : > I marked them wrong in the Critic Browser, which now skips them, but the > validation report still shows them as Warnings. Anyone know why? I see the > versions in the inbox references by the slice have their manifest classes... On Fri, May 8, 2015 at 4:00 AM, Christophe Demarey < christophe.demarey(a)inria.fr> wrote: > The monkey does a first run of tests/critics with the current pharo image > ans store failed tests/rules. Then, it loads the slice and does a second > run. At the end, issues reported are only the new failed tests/ rules. > That is how I understand it should work, but I've found that it doesn't always work in practice. Though at the time I thought it might just be me and I didn't stop to extract a reproducible case. And then once an issue is integrated, its not there for troubleshooting. How about we open an issue for this, and temporarily introduce a new package CaseNNNN for testing this, where Sean could reproduce his case. cheers -ben

2 1

Rendering morph list items in NewList
by Yuriy Tymchuk May 7, 2015

May 7, 2015

Hi, when I use strings for list items of NewList - everything is fine, but if I use morphs - I get a strange artifacts if another window showers over them. Here is an example video: https://dl.dropboxusercontent.com/u/83145561/morphItems.mov <https://dl.dropboxusercontent.com/u/83145561/morphItems.mov> Maybe some morph gurus have an idea whatâs wrong? For now Iâve discovered that all items render themselves with #listRenderOn:atRow:bounds:color:backgroundColor:from: Cheers! Uko

3 4

Penelope Doesn't Skip "Wrong" Lint Rules
by Sean P. DeNigris May 7, 2015

May 7, 2015

I marked them wrong in the Critic Browser, which now skips them, but the validation report still shows them as Warnings. Anyone know why? I see the versions in the inbox references by the slice have their manifest classes... ----- Cheers, Sean -- View this message in context: http://forum.world.st/Penelope-Doesn-t-Skip-Wrong-Lint-Rules-tp4825134.html Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.

2 3

[Review]: Issue 15507: SliceMaker Overhaul
by Sean P. DeNigris May 7, 2015

May 7, 2015

Issue 15507: SliceMaker Overhaul https://pharo.fogbugz.com/default.asp?15507 It all started simply enough: I wanted to extract the issue title downloading - which is generally useful - from slice maker. One full day later, I ended up with a pretty extensive rewrite. Now, I feel like it would've been better to start from scratch, but IMHO it's better, and good enough to move on to more pressing things. Fix in inbox: SLICE-Issue-15507-SliceMaker-Overhaul-SeanDeNigris.1 ###[FEATURE]: PharoIssue class. Here is it's class comment: I represent a Pharo development issue. Responsibilities: - create text templates (e.g. for the mailing list or issue tracker comments) for common events like issue creation and slice submission - communicate with the issue tracker Collaborators: - ZnClient - my link to the issue tracker - MCSliceInfo - I can get you one if you need it Public API and Key Messages: Most of my behavior is illustrated in my tests. The noticable exception are my text template creation messages, the testing of which seemed to hurt as much in duplication as it added in QA. ###[Cleanup]: Komitter - Remove Duplication Caused By MCSliceInfo - For some reason, Komitter seems to deal with MCPackages, while MCSliceInfo deals with MCWorkingCopy instances. This causes a tiny difference when making a slice (one sends #package to the wc, the other already has the package). The previous solution was to copy/paste a long method, changing one statement. This one trades that method in for "MCPackage>>#package ^ self" with a careful explanation in the comment of why we are doing this ugly thing! ###[ENH]: MC Slice Making - This is the most significant change - Extract issue-related behavior to PharoIssue - Extract slice-related behavior to MCSliceInfo. This class is still pretty confusing. It's a cross between a model of an actual slice, and an Application Model for MCSliceMaker. But since slices are supposed to go away soon... - Many small cleanups - fix misspellings, improve comments, simplify code - Remove the "Grab" button. The issue title is automatically downloaded when the issue number field loses focus ----- Cheers, Sean -- View this message in context: http://forum.world.st/Review-Issue-15507-SliceMaker-Overhaul-tp4825070.html Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.

1 1

Debugger hangs vm on step into
by Alain Rastoul May 7, 2015

May 7, 2015

Hi I wanted to step into SharedQueue code, but got a hanged vm on simple test code below. It works fine in Pharo 3 but hangs in Pharo4-40611 and Pharo5-50029, it does not hang in squeak 4.5 but the Monitor code is not the same. In a playgound, execute the first two lines of code then select the third line and 'Debug it' (right click menu option) to step into it. q := SharedQueue new . q nextPut: 1 . q next. Everything goes fine until stepping into 'aBlock value' in Monitor>>critical: critical: aBlock ^[ self enter. Hangs here >>>>aBlock value<<<<<< ] ensure: [self exit]. The vm get hanged and can be interrupted only once, then it is completely hanged. I once got an error message Error: Monitor access violation message and a stack trace in debug log before hanging but was unable to reproduce it (something related to effectiveProcess?) --- The full stack --- Monitor(Object)>>error: Monitor>>checkOwnerProcess Monitor>>waitWhile:for:maxMilliseconds: Monitor>>waitWhile:for: Monitor>>waitWhile: [ monitor waitWhile: [ items isEmpty ]. items removeFirst ] in SharedQueue>>next [ self enter. aBlock value ] in Monitor>>critical: BlockClosure>>ensure: Monitor>>critical: SharedQueue>>next UndefinedObject>>DoIt CompiledMethod>>valueWithReceiver:arguments: [ aCompiledMethod valueWithReceiver: anObject arguments: (evalContext ifNil: [ #() ] ifNotNil: [ {evalContext} ]) ] in RubSmalltalkEditor>>debug:receiver:in: [ self value. Processor terminateActive ] in BlockClosure>>newProcess ------------------------------------------------------------------------------- Can someone have a look and confirm it is a bug before I open a bug entry ? And/or give me hints on how to debug this ? I tried to modify Monitor>>waitWhile: and put a delay, put traces but still hanging. DebugSession someInstance stepInto does not hang, but I'm not sure that the execution flow is the same ... TIA -- Regards, Alain

2 9

RewriteTool
by Sean P. DeNigris May 7, 2015

May 7, 2015

There are two repos: http://smalltalkhub.com/#!/~PharoExtras/RewriteTool , which seems to have been forked some time ago and http://smalltalkhub.com/#!/~MarkRizun/RewriteTool/ , which is now about 70 commits ahead of the first! What should be done? ----- Cheers, Sean -- View this message in context: http://forum.world.st/RewriteTool-tp4824877.html Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.

4 11

[pharo-project/pharo-core] f22176: 50039
by GitHub May 7, 2015

May 7, 2015

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: f22176c65834be28372208e6add9f57a39f1b644 https://github.com/pharo-project/pharo-core/commit/f22176c65834be28372208e6… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-05-07 (Thu, 07 May 2015) Changed paths: M Kernel.package/Character.class/instance/printing/hex.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script647.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script648.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50038.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50039.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Slot.package/extension/Behavior/instance/layout.st Log Message: ----------- 50039 15286 `Class new` doesnt result in a properly initialized class object. https://pharo.fogbugz.com/f/cases/15286 15506 Hexadecimal is misspelled in Character>>#hex https://pharo.fogbugz.com/f/cases/15506 http://files.pharo.org/image/50/50039.zip

1 0