On Wed, May 10, 2017 at 12:27 PM, Michael Osipov <micha...@apache.org>
wrote:

> Am 2017-05-10 um 21:16 schrieb Oleg Kalnichevski:
>
>> On Wed, 2017-05-10 at 21:07 +0200, Michael Osipov wrote:
>>
>>> Am 2017-05-10 um 20:52 schrieb Oleg Kalnichevski:
>>>
>>>> On Wed, 2017-05-10 at 11:15 -0700, Gary Gregory wrote:
>>>>
>>>>> On Wed, May 10, 2017 at 10:13 AM, Oleg Kalnichevski <olegk@apache
>>>>> .org
>>>>>
>>>>>>
>>>>>>
>>>>> wrote:
>>>>>
>>>>> ...
>>>>>>
>>>>>>
>>>>>> One personal request. Do you think you could try to make
>>>>>>>> your
>>>>>>>> commits
>>>>>>>> less granular and combine logically related changes into
>>>>>>>> larger
>>>>>>>> change
>>>>>>>> sets?
>>>>>>>>
>>>>>>>>
>>>>>>> Old habits die hard. Git might indeed make this better. If
>>>>>>> you
>>>>>>> want
>>>>>>> to add
>>>>>>> this to the style guideline on the site, it will help all
>>>>>>> contributors,
>>>>>>> present and future.
>>>>>>>
>>>>>>>
>>>>>> Hi Gary
>>>>>>
>>>>>> I do not think it would be possible to enforce it through a
>>>>>> style
>>>>>> check
>>>>>> or a commit hook as there are legitimate reasons for one line
>>>>>> commits.
>>>>>>
>>>>>> Please do not get me wrong. I have no intentions of changing
>>>>>> your
>>>>>> way
>>>>>> of working. I fully respect other people's habits. What I am
>>>>>> asking
>>>>>> is
>>>>>> your consent to squash some of your commits (combining small
>>>>>> related
>>>>>> commit into a larger one).
>>>>>>
>>>>>>
>>>>> Oh sure, feel free to do what you want. I did read something a
>>>>> long
>>>>> time
>>>>> ago warning git users about fiddling with repo history, but since
>>>>> we
>>>>> have a
>>>>> master repo and we are not truly using git in a distributed way,
>>>>> that
>>>>> should not hurt us.
>>>>>
>>>>>
>>>> Of course, re-writing history can break forks, but we are not Linux
>>>> kernel. I am not aware of a single fork maintained externally.
>>>> Besides,
>>>> rewriting would be limited to the most recent commits. No one is
>>>> going
>>>> to rewrite history more than a few days back.
>>>>
>>>
>>> INFRA won't allow that. Master is a propertive branch as far as I
>>> know.
>>>
>>>
>> As far as I know this rule could be disabled per request.
>>
>> History rewriting works for ordinary (non-master) branches
>>
>> ---
>> oleg@ok2c:~/src/apache.org/httpcomponents/httpcore$ git push origin
>> --force-with-lease
>> Counting objects: 11, done.
>> Delta compression using up to 4 threads.
>> Compressing objects: 100% (6/6), done.
>> Writing objects: 100% (11/11), 968 bytes | 0 bytes/s, done.
>> Total 11 (delta 3), reused 0 (delta 0)
>> remote: httpcomponents-core git commit: Keep examples self-contained
>> To https://git-wip-us.apache.org/repos/asf/httpcomponents-core.git
>>  + 0ad926e8...5b29a6e4 4.4.x -> 4.4.x (forced update)
>>
>
> To avoid issues like that, we should have the same approach as Tomcat or
> Maven does. One repo per major version, thus
>
> httpcomponents-core-4
> httpcomponents-client-4
> httpcomponents-core-5
> httpcomponents-client-5
>

I like that. I've already checked out HttpCore twice for 4.4.x and 5 just
be able to look at both source trees at the same time.

Also, from Eclipse, switch branches between core4 and core5 is not smooth
because the modules are not the same; which is another reason I ended up
checking out core twice in two parallel directories.

Gary


At the end, it will spare us a lot of issues and we can mark 4.x as legacy
> some day.
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org
> For additional commands, e-mail: dev-h...@hc.apache.org
>
>


-- 
E-Mail: garydgreg...@gmail.com | ggreg...@apache.org
Java Persistence with Hibernate, Second Edition
<https://www.amazon.com/gp/product/1617290459/ref=as_li_tl?ie=UTF8&camp=1789&creative=9325&creativeASIN=1617290459&linkCode=as2&tag=garygregory-20&linkId=cadb800f39946ec62ea2b1af9fe6a2b8>

<http:////ir-na.amazon-adsystem.com/e/ir?t=garygregory-20&l=am2&o=1&a=1617290459>
JUnit in Action, Second Edition
<https://www.amazon.com/gp/product/1935182021/ref=as_li_tl?ie=UTF8&camp=1789&creative=9325&creativeASIN=1935182021&linkCode=as2&tag=garygregory-20&linkId=31ecd1f6b6d1eaf8886ac902a24de418%22>

<http:////ir-na.amazon-adsystem.com/e/ir?t=garygregory-20&l=am2&o=1&a=1935182021>
Spring Batch in Action
<https://www.amazon.com/gp/product/1935182951/ref=as_li_tl?ie=UTF8&camp=1789&creative=9325&creativeASIN=1935182951&linkCode=%7B%7BlinkCode%7D%7D&tag=garygregory-20&linkId=%7B%7Blink_id%7D%7D%22%3ESpring+Batch+in+Action>
<http:////ir-na.amazon-adsystem.com/e/ir?t=garygregory-20&l=am2&o=1&a=1935182951>
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory

Reply via email to