Re: NEWS and UPGRADING files in PHP-5.6
On Thu, Feb 6, 2014 at 2:02 PM, Tjerk Meesters <[email protected]>wrote:
> Hi Ferenc,
>
> > On 6 Feb, 2014, at 7:13 pm, Ferenc Kovacs <[email protected]> wrote:
> >
> >> On Thu, Feb 6, 2014 at 11:31 AM, Ferenc Kovacs <[email protected]>
> wrote:
> >>
> >>
> >>
> >>
> >>> On Thu, Feb 6, 2014 at 10:02 AM, Ferenc Kovacs <[email protected]>
> wrote:
> >>>
> >>>
> >>>
> >>>
> >>>> On Thu, Feb 6, 2014 at 9:53 AM, Bob Weinand <[email protected]>
> wrote:
> >>>>
> >>>> These two files in PHP-5.6 branch actually are the same than master.
> >>>>
> >>>> I just noticed that when I wanted to write about the merge of pow
> >>>> operator RFC…
> >>>>
> >>>> Could someone please fix that one?
> >>>>
> >>>> Bob
> >>>> --
> >>>> PHP Internals - PHP Runtime Development Mailing List
> >>>> To unsubscribe, visit: http://www.php.net/unsub.php
> >>> from a quick look it seems that your last merge caused it, will revert
> >>> it, if nobody beats me to it.
> >>>
> >>> --
> >>> Ferenc Kovács
> >>> @Tyr43l - http://tyrael.hu
> >>
> >> yep, it seems that you've merged a branch/PR which was originally
> branched
> >> from master.
> >> in the past when similar thing happened we always reset the head before
> >> the merge and then force push it.
> >> I don't have karma for that, but I can grant myself if nobody around
> >> (johannes?)
> >>
> >> --
> >> Ferenc Kovács
> >> @Tyr43l - http://tyrael.hu
> >
> >
> > Hi,
> >
> > I've restored the tip of the PHP-5.6 and master branch to where it was
> > before the merge.
> > The branch which holds the wrong merge is available here:
> >
> http://git.php.net/?p=php-src.git;a=shortlog;h=refs/heads/broken-5.6-20140206
> > If you pulled between the wrong merge and the fix (~2 hours of
> timeframe),
> > then you have to put your local 5.6 and master branch aside and reset
> those
> > to the origin and cherrypick your local changes if you have any.
> > Bob: please rebase/cherrypick your changes to the PHP-5.6 branch first,
> and
> > merge that upwards to master, otherwise you will bring along the other
> > commits from the master branch.
>
> Thanks. Was there something I should have done differently with my PR or
> are branches off master okay?
>
> >
> > --
> > Ferenc Kovács
> > @Tyr43l - http://tyrael.hu
>
Yeah, the problem was that the PR was targeted to master (as you can see at
https://github.com/php/php-src/pull/543),
while the RFC targeted 5.6 (
https://wiki.php.net/rfc/pow-operator#proposed_php_version_s)
Ofc. the person doing the merge could have realized this and either rebase
himself, or ask you to send another PR, but next time make sure that you
base the PR on the correct branch.
--
Ferenc Kovács
@Tyr43l - http://tyrael.hu
Thread (7 messages)