Re: cannot push to 5.x branches

From: 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.

« previous php.internals (#70035) next »