Re: cannot push to 5.x branches
| From: | David Soria Parra | Date: | Wed, 06 Nov 2013 16:42:12 +0000 |
| Subject: | Re: cannot push to 5.x branches | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-70035@lists.php.net to get a copy of this message | ||
Hannes Magnusson <hannes.magnusson@gmail.com> schrieb:
> On Tue, Nov 5, 2013 at 1:29 PM, George Wang <gwang@litespeedtech.com> wrote:
>> Hi,
>>
>> I try to update sapi/litespeed code to the latest. Successfully pushed to
>> master branch:
>>
>> remote: Welcome gwang.
>> remote: Changesets accepted. Thank you for your contribution.
>> remote:
>> remote: Attempting to push to mirror git@github.com:php/php-src.git
>> remote: To git@github.com:php/php-src.git
>> remote: c4cfdfa..3720fd7 master -> master
>> To git@git.php.net:php-src.git
>> c4cfdfa..3720fd7 HEAD -> master
>>
>> but cannot push to 5.x branches. Got:
>>
>> remote: Welcome gwang.
>> remote: You have insufficient Karma!
>> remote: I'm sorry, I cannot allow you to write to
>> remote: php-src.git/
>> remote: Have a nice day.
>> To git@git.php.net:php-src.git
>> ! [remote rejected] HEAD -> 5.3 (pre-receive hook declined)
>> error: failed to push some refs to 'git@git.php.net:php-src.git'
>>
>> My .git/config:
>> [core]
>> repositoryformatversion = 0
>> filemode = true
>> bare = false
>> logallrefupdates = true
>> [remote "origin"]
>> url = git@git.php.net:php-src.git
>> fetch = +refs/heads/*:refs/remotes/origin/*
>> pushurl = git@git.php.net:php-src.git
>> [branch "master"]
>> remote = origin
>> merge = refs/heads/master
>> [branch "5.3"]
>> remote = origin
>> merge = refs/heads/PHP-5.3
>> [branch "5.4"]
>> remote = origin
>> merge = refs/heads/PHP-5.4
>> [branch "5.5"]
>> remote = origin
>> merge = refs/heads/PHP-5.5
>>
>> Did I do anything wrong, or I just need more Karma?
>>
>
>
> You have karma for the litespeed sapi in php-src... master.
> We do have a good chunk of people that only have access to specific
> paths/extensions/sapis in php-src, I guess that doesn't work anymore
> since the move to git..
> I am unsure how the avail line should be to include branches, or how
> do modify the pre-receive hook to ignore branches when doing the path
> check.. David?
It works. We are checking the contents of the commit and all affected
paths in order to allow writes or not. However if a person with insufficient
karma merges he will get a reject during push as the merge might touch paths
that he is not allowed to write to.