Boost logo

Boost :

Subject: Re: Recent develop vs master shuffles broke existing pull requests
From: Stefan Seefeld (stefan_at_[hidden])
Date: 2018-08-27 13:45:25

Hi Mateusz,

On 2018-08-26 05:58 AM, Mateusz Loskot wrote:
> Hi,
> I noticed a slightly worrying situation with the pull request
> reported before release of Boost 1.68, that is before we
> shuffled the branches so the latest master become the develop.
> Have a look at
> GitHub now says the PR brings 250+ commits and changes 1164 files!
> If I try to rebase the mloskot:ml/remove-boost-type-traits branch against
> the current origin:develop, I get gazillion conflicts impossible to resolve.
> For comparison, here is unbroken version of the PR #107
> which is against the old develop from before Boost 1.68 release
> Does anyone know how to 'fix' the develop so the existing
> PRs don't become such a nonsense?
> I've run out of my git fu.
> The 'broken' develop affects only two PRs, the #107 and #87
> - the latter is the clang-format brainstorm, we can ignore it.
> The #107 is a non-trivial changeset of ~100 files, so I'd be a little bit
> upset if the broken `develop` sends the effort out of the window.
> I will try to salvage it though.
> How can we avoid such problems in future?

I'm sorry to hear about this trouble. I should try to do this rebase of
the PR. Given that I used the `git rerere` feature during that work, the
machine may be able to assist in bringing over your PR by remembering
conflict resolution.

Of course, we should avoid such problems by never using "history
rewrites" on public branches. (This is common knowledge, and I simply
neglected the fact that there were existing PRs against `develop` that
would have to be migrated if I introduced a "new" `develop` branch.

I'll see what I can do (but unlikely this week, which will be extremely
busy for me).



       ...ich hab' noch einen Koffer in Berlin...

Boost list run by Boost-Gil-Owners