[CentOS-devel] [PATCH] Update API path in centos.git.repolist.py
Laurentiu Pancescu
lpancescu at gmail.com
Fri Jun 23 18:48:47 UTC 2017
That looks good. Perhaps we could use requests.codes.ok instead of 200,
but maybe that's just splitting hairs about magical constants... :)
On 23/06/17 20:36, Pat Riehecky wrote:
> ---
> centos.git.repolist.py | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/centos.git.repolist.py b/centos.git.repolist.py
> index 75122ae..e135ca7 100755
> --- a/centos.git.repolist.py
> +++ b/centos.git.repolist.py
> @@ -13,7 +13,7 @@ import requests
> import simplejson as json
> import sys
>
> -RPCURL = "https://git.centos.org/rpc?req=LIST_REPOSITORIES"
> +RPCURL = "https://git.centos.org/rpc/?req=LIST_REPOSITORIES"
>
> def read_args():
> '''
> @@ -48,6 +48,11 @@ def get_repo_list(url, branch, projectpath):
> print err_msg
> sys.exit(1)
>
> + if req.status_code != 200:
> + print "Unable to access gitblit api at " + url
> + sys.exit(1)
> +
> +
> payload = req.text
> repos = json.loads(payload)
> branchname = 'refs/heads/' + branch
> @@ -68,6 +73,6 @@ def main():
> repos = get_repo_list(url=options.url, branch=options.branch, projectpath=options.project)
> if repos:
> print '\n'.join(repos)
> -
> +
> if __name__ == "__main__":
> main()
>
More information about the CentOS-devel
mailing list