On 22 November 2013 08:29, Frank Shearar frank.shearar@gmail.com wrote:
It's been over a year, and several rounds of discussion. The last round fizzled out with 'yeah ok we should do it but don't call it Author'. Really, I don't particularly care _what_ we do as long as we remove another part of Utilities. So I'm going to write a minimal thing that pulls the state out of Utilities, where it never belonged. Once the real problem - that state - has been addressed, finessing the API will be much easier.
I sound like a rude, grumpy old man. I don't mean to sound _rude_ :) So I'll go re-re-read the last round of discussion, and make sure I'm not deliberately snubbing anyone's ideas, and deliver some code. Hopefully it won't be a complete disaster, but in order to fix this problem, _someone_ has to write something. So I'll take the hit, and put my code up for tomato-throwing.
frank
frank
On 21 Nov 2013, at 23:05, Chris Muller asqueaker@gmail.com wrote:
Back when Levente approached this, we had a detailed discussion about it. I think we should all go back and read that before introducing a SystemAuthor class.
Personally, I wouldn't mind just having
Smalltalk authorInitials "answers a String"
until we decide we need to support multiple concurrent Author objects with more than just name and initials..
And this makes sense anyway -- Smalltalk's current #author.
I struggle with having a whole class (SystemAuthor) when we cannot take proper advantage of it and only really need one String field at this time, not a heavy-weight, value-holder singleton.
On Thu, Nov 21, 2013 at 4:33 PM, Frank Shearar frank.shearar@gmail.com wrote:
On 21 November 2013 21:20, commits@source.squeak.org wrote:
Frank Shearar uploaded a new version of MonticelloConfigurations to project The Trunk: http://source.squeak.org/trunk/MonticelloConfigurations-fbs.118.mcz
==================== Summary ====================
Name: MonticelloConfigurations-fbs.118 Author: fbs Time: 21 November 2013, 9:20:19.085 pm UUID: aaba44a1-8cfd-4147-8d94-69d5fc5ac571 Ancestors: MonticelloConfigurations-cmm.117
Move the #upgradeIsMerge preference to MCConfiguration.
=============== Diff against MonticelloConfigurations-cmm.117 ===============
Just by the way, MonticelloConfigurations depends on System for one thing only now: Utilities' author stuff. I'm knocking up a completely lame minimal mimic-existing-stuff SystemAuthor that I will hopefully finish tomorrow. I plan to sever this dependency, and then remove all the references to Utilities authorName/Initials with SystemAuthor current, and then we can think about any other stuff we might do with this.
frank