Charles, Marc's advice reminded me that I left out an important step- creating a branch off of the karaf branch. His advice to squash commits into blocks before rebasing master can save you quite a bit of time, so I'd go with that approach over all others we've discussed. Best of luck!

Rachel

On Mon, Mar 21, 2016 at 4:36 AM, Marc Savy <marc.savy@redhat.com> wrote:
It's right there in the archive -

http://lists.jboss.org/pipermail/apiman-dev/2016-March/000090.html

`git rebase --interactive` is the same as `git rebase -i`

On 16/03/2016 08:52, Charles Moulliard wrote:
> No advices from @Rachel or @Marc ?
>
>
> On 08/03/16 14:30, Charles Moulliard wrote:
>> Perhaps the option "git rebase --interactive" is better ?
>>
>> I'm also thinking that the rebase should be done from the commits of
>> karaf branch to master. If this is the case, then the cmds to be used are
>>
>> git checkout master
>> git rebase --interactive karaf // squash of the commits can be done
>> within the editor
>>
>>
>> On 08/03/16 12:44, Eric Wittmann wrote:
>>> @Marc or @Rachel - any advice to charles on the best way to merge and
>>> squash?
>>>
>>> -Eric
>>>
>>> On 3/8/2016 3:21 AM, Charles Moulliard wrote:
>>>> On 01/03/16 13:36, Eric Wittmann wrote:
>>>>> Hi Charles.  Here are some things we would need before we can merge
>>>>> the karaf branch:
>>>>>
>>>>> 1) rebase the karaf branch to master
>>>>> 2) squash the commits to a single commit
>>>>
>>>>   >> Can you confirm my scenario to rebase and squash ?
>>>> git checkout karaf
>>>> git rebase master
>>>> --> Fix merge issues and commit them ?
>>>> git checkout master
>>>> git merge --squash
>>>>
>>>>> 3) thorough code review
>>>>> 4) testing
>>>>>
>>>>> #1 - I think this is pretty straightforward.
>>>>>
>>>>> #2 - Normally I wouldn't mind a small number of commits for a
>>>>> contribution like the karaf support, but you've got 221 commits in the
>>>>> karaf branch, representing 117 files changed or added. That's a very
>>>>> large impact on the project's commit history and will make future
>>>>> diffs more difficult than is warranted.  So that's the reason to
>>>>> squash the commits down to one (or at most a handful).
>>>>>
>>>>> #3 - I'll need to run through the code with the rest of the apiman dev
>>>>> team so we're all familiar with the changes.  We need some extra
>>>>> eyeballs on the code to make sure we don't (for example) duplicate a
>>>>> lot of functionality.
>>>>>
>>>>> #4 - Obviously we'll need to do some testing to make sure everything
>>>>> is working as expected.
>>>>>
>>>>> Unfortunately we've got a bunch of high priority tasks at the moment,
>>>>> so it may be a little while before we can get to #3 and #4. I'm happy
>>>>> to accomplish #1 and #2 at some point in the future, although if you'd
>>>>> like to give it a try that would be fine too.
>>>>>
>>>>> -Eric
>>>>>
>>>>>
>>>>> On 3/1/2016 4:13 AM, Charles Moulliard wrote:
>>>>>> Hi,
>>>>>>
>>>>>> Can we consider to merge Karaf branch with Master - trunk ? What
>>>>>> are the
>>>>>> steps that we have to take care in order to prepare the merge ?
>>>>>>
>>>>>> Regards,
>>>>>>
>>>>>> Charles
>>>>>> _______________________________________________
>>>>>> Apiman-dev mailing list
>>>>>> Apiman-dev@lists.jboss.org
>>>>>> https://lists.jboss.org/mailman/listinfo/apiman-dev
>>>>>>
>>>>
>>
>
> _______________________________________________
> Apiman-dev mailing list
> Apiman-dev@lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/apiman-dev
>

_______________________________________________
Apiman-dev mailing list
Apiman-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/apiman-dev