[squeak-dev] MC Merge browser: 'Accept same source' button considered harmful

David T. Lewis lewis at mail.msen.com
Sun Jun 24 00:57:18 UTC 2018


On Sat, Jun 23, 2018 at 06:39:51PM -0500, Chris Muller wrote:
> Does anyone like to go around and change the format of other peoples
> code and nothing else?  I consider this inconsiderate to my peers and
> I'm sure most around here do, as well.
> 
> So does anyone know why in the world we this crazy button tacked onto
> the end of the MC Merge browser?  Every time I have a merge conflict,
> this button rears its ugly head and forces me to reckon with it.  Not
> only is it incongruous with the rest of the buttons, but totallly
> wrongheaded for how we respect each others work, resulting in
> unpleasant thoughts and experience every time I see it.
> 
> Browsing origin just shows:
> ___
>      Name: Monticello-tfel.637
>      Author: tfel
>      Time: 25 July 2016, 3:24:24.996828 pm
>      UUID: cf0d6af1-c703-5044-9c57-e798b0cf3abf
>      Ancestors: Monticello-cmm.636
> 
>      add a button for rejecting all incoming conflicts that only change AST
> ____
> 
> (rant) Explaining WHAT your changed in your version notes is
> redundant, we can all see the code.  Please tell us WHY you are doing
> it.
> 
> So does anyone know?  I think we should remove it and I cringe to
> wonder how many original works we lost to it...   :(
>

I do not understand the rant. The offending button was added almost a year
ago, and its balloon help says this:

  Choose all local conflicting versions that have essentially the same code

Even though I have never used this, the intent seems clear. It says "if
somebody was just messing around with the formatting but did not change
anything, then reject the change because it is not nice to mess with the
format of other peoples' code."

What's not to like about that?

Dave
 


More information about the Squeak-dev mailing list