Re: [Pharo-dev] About Git support for windows

2017-09-13 Thread Ben Coman
On Tue, Sep 12, 2017 at 11:23 PM, Peter Uhnák wrote: > Just a random idea, how about each time writing timestamps to >> a different file name "timestamps.$HashOfClassSourceFile" >> Then git would never complain of a conflict(??). >> > > If I understand your proposal correctly,

Re: [Pharo-dev] About Git support for windows

2017-09-12 Thread Peter Uhnák
> > Just a random idea, how about each time writing timestamps to > a different file name "timestamps.$HashOfClassSourceFile" > Then git would never complain of a conflict(??). > If I understand your proposal correctly, that would imo result in the following: accumulating endless list of

Re: [Pharo-dev] About Git support for windows

2017-09-12 Thread Ben Coman
On Sun, Sep 10, 2017 at 5:06 PM, Esteban Lorenzano wrote: > > On 10 Sep 2017, at 10:56, Henrik-Nergaard wrote: > > Everyone who used filetree with metadata can tell it is super annoying and > > destroys the complete experience. > There has been a fix

Re: [Pharo-dev] About Git support for windows

2017-09-11 Thread H. Hirzel
A note: Both variants below do not look nice because of the code starting at the first column. Some of the method body code lines start in the first column: Morph class>>obtainArrowheadFor: aPrompt defaultValue: defaultPoint [ "Allow the user to supply a point to serve as an arrowhead size.

Re: [Pharo-dev] About Git support for windows

2017-09-10 Thread Stephane Ducasse
I read all the morph class and I like the ending ] at the beginning of the line (even if I know the rectangle concern for ifTrue beck formatting) Why because it identifies really well the end of a method. It is especially good for method finishing with conditionals. ] Esteban could you

Re: [Pharo-dev] About Git support for windows

2017-09-10 Thread Stephane Ducasse
So if you follow eliot suggestion methodDeclaration [ methodbody ] put the start in another line. because with methodDeclaration [ methodbody ] we do not identify the body as a rectangle I do not think that [ on the first line is a good idea. Remember that people will send code in

Re: [Pharo-dev] About Git support for windows

2017-09-10 Thread Stephane Ducasse
Yes and this is not really good for method with multiple and long selectors because you have to spot the end of the line Morph class>>obtainArrowheadFor: aPrompt defaultValue: defaultPoint [ "Allow the user to supply a point to serve as an arrowhead size. Answer nil if we fail to get a good

Re: [Pharo-dev] About Git support for windows

2017-09-10 Thread Esteban Lorenzano
heh… the format I propose is like this: methodDeclaration [ methodbody ] I think that keeps good readability and I can parse it easily. > On 10 Sep 2017, at 18:23, Stephane Ducasse wrote: > > Hi esteban > for me having space around [ ] improves readibility. > I

Re: [Pharo-dev] About Git support for windows

2017-09-10 Thread Stephane Ducasse
Hi esteban for me having space around [ ] improves readibility. I preferred methodDeclaration [ methodbody ] over methodDeclaration [ methodbody ] Especially in case of method body is blockish like while true/exception. But this is ok if people prefer methodDeclaration [

Re: [Pharo-dev] About Git support for windows

2017-09-10 Thread stephan
On 09-09-17 19:43, Eliot Miranda wrote: I think this is a serious mistake. Time stamps and author marks are important (even more important if the unit of commit is a single file for the whole class). When converting repos from monticello to git we could add extra commits for groups of

Re: [Pharo-dev] About Git support for windows

2017-09-10 Thread Thierry Goubier
Hi Henrik, Le 10/09/2017 à 10:56, Henrik-Nergaard a écrit : Everyone who used filetree with metadata can tell it is super annoying and destroys the complete experience. There has been a fix for this on the issue tracker for some time:

Re: [Pharo-dev] About Git support for windows

2017-09-10 Thread Esteban Lorenzano
> On 10 Sep 2017, at 10:56, Henrik-Nergaard wrote: > >>> Everyone who used filetree with metadata can tell it is super annoying and > destroys the complete experience. > There has been a fix for this on the issue tracker for some time: >

Re: [Pharo-dev] About Git support for windows

2017-09-10 Thread Henrik-Nergaard
>>Everyone who used filetree with metadata can tell it is super annoying and destroys the complete experience. There has been a fix for this on the issue tracker for some time: https://pharo.fogbugz.com/f/cases/20251/Write-out-filetree-metadata-in-order Best regards, Henrik -- Sent from:

Re: [Pharo-dev] About Git support for windows

2017-09-10 Thread Esteban Lorenzano
> On 9 Sep 2017, at 19:35, Eliot Miranda wrote: > > > Point class >> x: anInt1 y: anInt2 > [ > ^ self new setX: anInt1 Y: anInt2] > > because trailing white space will only accumulate, something I see in Pharo > and Squeak code. > > Alternatively have the

Re: [Pharo-dev] About Git support for windows

2017-09-10 Thread Esteban Lorenzano
at information either… just get it in a different way. Esteban > >> >> Esteban >> >>> >>> >>> Fra: Pharo-dev <pharo-dev-boun...@lists.pharo.org >>> <mailto:pharo-dev-boun...@lists.pharo.org>> på vegne av Esteban Lore

Re: [Pharo-dev] About Git support for windows

2017-09-10 Thread Esteban Lorenzano
> On 10 Sep 2017, at 09:50, Norbert Hartl wrote: > > Hi! > >> Am 10.09.2017 um 05:44 schrieb Alexandre Bergel : >> >> Hi! >> >> I live very very much this idea! >> >> I am wondering why do we need the timestamp? Git knows about who has >>

Re: [Pharo-dev] About Git support for windows

2017-09-10 Thread Norbert Hartl
Hi! > Am 10.09.2017 um 05:44 schrieb Alexandre Bergel : > > Hi! > > I live very very much this idea! > > I am wondering why do we need the timestamp? Git knows about who has > committed each line and by whom. Why should we have some redundant > information? > the

Re: [Pharo-dev] About Git support for windows

2017-09-09 Thread Alexandre Bergel
Hi! I live very very much this idea! I am wondering why do we need the timestamp? Git knows about who has committed each line and by whom. Why should we have some redundant information? Cheers, Alexandre -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel

Re: [Pharo-dev] About Git support for windows

2017-09-09 Thread Peter Uhnák
> > I don't think it's even a problem > I do remember that I hated long classes because of git conflicts, but I suppose if we have good tooling for that it shouldn't be such a pain. Git can sometimes make the diff really weirdly... Peter

Re: [Pharo-dev] About Git support for windows

2017-09-09 Thread Eliot Miranda
eban >> Lorenzano <esteba...@gmail.com> >> Sendt: 9. september 2017 11:39:12 >> Til: Pharo Development List >> Emne: Re: [Pharo-dev] About Git support for windows >> >> >> > On 8 Sep 2017, at 22:02, Eliot Miranda <eliot.mira...@gmail.com> w

Re: [Pharo-dev] About Git support for windows

2017-09-09 Thread Eliot Miranda
Hi Esteban, > On Sep 8, 2017, at 7:08 PM, Esteban Lorenzano wrote: > > > >> On 8 Sep 2017, at 22:35, Peter Uhnák wrote: >> >> for example here >> >> https://github.com/estebanlm/pharo-tonel/blob/master/src/Morphic-Core/Morph.class.st >> > but this

Re: [Pharo-dev] About Git support for windows

2017-09-09 Thread Esteban Lorenzano
ember 2017 11:39:12 > Til: Pharo Development List > Emne: Re: [Pharo-dev] About Git support for windows > > > > On 8 Sep 2017, at 22:02, Eliot Miranda <eliot.mira...@gmail.com> wrote: > > > > Hi All, > > > >> On Sep 8, 2017, at 9:44 AM, Stephan

Re: [Pharo-dev] About Git support for windows

2017-09-09 Thread Henrik Nergaard
endt: 9. september 2017 11:39:12 Til: Pharo Development List Emne: Re: [Pharo-dev] About Git support for windows > On 8 Sep 2017, at 22:02, Eliot Miranda <eliot.mira...@gmail.com> wrote: > > Hi All, > >> On Sep 8, 2017, at 9:44 AM, Stephane Ducasse <stepharo.s...@gm

Re: [Pharo-dev] About Git support for windows

2017-09-09 Thread Esteban Lorenzano
> On 8 Sep 2017, at 22:02, Eliot Miranda wrote: > > Hi All, > >> On Sep 8, 2017, at 9:44 AM, Stephane Ducasse wrote: >> >> Hi all >> >> At ESUG we discussed with Esteban, martin mcClure, Dale and (many many >> others :), esteban designed a

Re: [Pharo-dev] About Git support for windows

2017-09-08 Thread Esteban Lorenzano
> On 8 Sep 2017, at 22:35, Peter Uhnák wrote: > > for example here > > https://github.com/estebanlm/pharo-tonel/blob/master/src/Morphic-Core/Morph.class.st > but this is still WIP: for example timestamp needs to go to avoid conflicts. > and it also shows one downside ---

Re: [Pharo-dev] About Git support for windows

2017-09-08 Thread Peter Uhnák
for example here https://github.com/estebanlm/pharo-tonel/blob/master/src/Morphic-Core/Morph.class.st and it also shows one downside --- up until now we didn't care much about how many methods are in class (because it was just another file), but now we have 1 lines of code class. On Fri,

Re: [Pharo-dev] About Git support for windows

2017-09-08 Thread Stephane Ducasse
>From memory one class of class/trait and one for extension In class/Trait "class / trait comments comment" Special STON order for classes to minimic our logical order (ie package tag add the end). Class { } or Trait { name: #T } + Point class >> x: anInt1 y: anInt2 [ ^ self

Re: [Pharo-dev] About Git support for windows

2017-09-08 Thread Eliot Miranda
Hi All, > On Sep 8, 2017, at 9:44 AM, Stephane Ducasse wrote: > > Hi all > > At ESUG we discussed with Esteban, martin mcClure, Dale and (many many > others :), esteban designed a nice class file format. So that we will > not have 2Gb of space on harddisc, problems

[Pharo-dev] About Git support for windows

2017-09-08 Thread Stephane Ducasse
Hi all At ESUG we discussed with Esteban, martin mcClure, Dale and (many many others :), esteban designed a nice class file format. So that we will not have 2Gb of space on harddisc, problems with long method names and sluggish commits. He is waiting at Wien and is probably checking everything