NEWS and UPGRADING files in PHP-5.6

11 years ago by Bob Weinand — view source — reply

unread

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

11 years ago by Ferenc Kovacs — view source — reply

unread

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

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

11 years ago by Ferenc Kovacs — view source — reply

unread

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

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

11 years ago by Ferenc Kovacs — view source — reply

unread

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

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.

--
Ferenc Kovács
@Tyr43l - http://tyrael.hu

11 years ago by Tjerk Meesters — view source — reply

unread

Hi Ferenc,

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

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

11 years ago by Ferenc Kovacs — view source — reply

unread

On Thu, Feb 6, 2014 at 2:02 PM, Tjerk Meesters tjerk.meesters@gmail.comwrote:

Hi Ferenc,

On Thu, Feb 6, 2014 at 11:31 AM, Ferenc Kovacs tyra3l@gmail.com
wrote:

On Thu, Feb 6, 2014 at 10:02 AM, Ferenc Kovacs tyra3l@gmail.com
wrote:

On Thu, Feb 6, 2014 at 9:53 AM, Bob Weinand bobwei9@hotmail.com
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

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

11 years ago by Tjerk Meesters — view source — reply

unread

On Thu, Feb 6, 2014 at 2:02 PM, Tjerk Meesters tjerk.meesters@gmail.comwrote:

Hi Ferenc,

On Thu, Feb 6, 2014 at 11:31 AM, Ferenc Kovacs tyra3l@gmail.com
wrote:

On Thu, Feb 6, 2014 at 10:02 AM, Ferenc Kovacs tyra3l@gmail.com
wrote:

On Thu, Feb 6, 2014 at 9:53 AM, Bob Weinand bobwei9@hotmail.com
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

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.

Roger that. In case it helps, I've created a branch off PHP-5.6 and cherry
picked my commit into it; it can be found here:

https://github.com/datibbaw/php-src/compare/php:PHP-5.6...pow-5.6

--
Ferenc Kovács
@Tyr43l - http://tyrael.hu

--

Tjerk