[CentOS-devel] [PATCH] Update API path in centos.git.repolist.py

Fri Jun 23 18:48:47 UTC 2017
Laurentiu Pancescu <lpancescu at gmail.com>

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()
>