packagedb-cli-commits September 2015

packagedb-cli-commits@lists.stg.fedorahosted.org

1 participants
26 discussions

[packagedb-cli] Created tag 2.9
by Pierre-YvesChibon 03 Sep '15

03 Sep '15

The signed tag '2.9' was created. Tagger: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Thu Sep 3 14:41:50 2015 +0200 Release 2.9 Changes since the last tag '2.8.2': Mikolaj Izdebski (1): Fix set_koschei_status() argument name Pierre-Yves Chibon (18): Add a method in pkgdb2client to set the monitoring status of a package Adjust the get_package method to support no returning all the ACLs Add a monitoring action to pkgdb-cli Check if the pkgdb instance we're calling support setting monitoring flag by the API Adjust the error message returned Merge pull request #36 from fedora-infra/monitoring Add a set_koschei_status to the module pkgdb2client Add a koschei action to pkgdb-cli Check if the pkgdb2 instance called support koschei monitoring flag Merge pull request #37 from fedora-infra/koschei_monitoring Merge pull request #38 from tyll/lxml Retrieve the version before doing something with it Merge pull request #40 from fedora-infra/get_version Use the print() method instead of the print statement to be py3 compatible Catch exceptions as to be py3 compatible Merge pull request #42 from mizdebsk/argument-name Merge pull request #39 from fedora-infra/py3 Release 2.9 Ralph Bean (2): Raise an exception when unable to decode JSON. Merge pull request #35 from fedora-infra/fail-on-json Till Maas (1): Set default BeautifulSoup parser

1 0

[packagedb-cli] Created tag 2.8.2
by Pierre-YvesChibon 03 Sep '15

03 Sep '15

The signed tag '2.8.2' was created. Tagger: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Mon May 4 15:49:02 2015 +0200 Release 2.8.2 Changes since the last tag '2.8.1': Pierre-Yves Chibon (7): Adjust name of the parser for the process action Add an `info` action returning some information about the specified action Merge pull request #32 from fedora-infra/do_info Report an error if the fedora-review flag is not set to `+` Merge pull request #33 from fedora-infra/error_fedora_review_flag Adjust copyright year Release 2.8.2

1 0

[packagedb-cli] Release 2.9
by Pierre-YvesChibon 03 Sep '15

03 Sep '15

commit 65c71d7e12966dbd91286b7bb6813978077aedc4 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Thu Sep 3 14:41:01 2015 +0200 Release 2.9 README | 4 ++-- setup.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) --- diff --git a/README b/README index b2d4df8..68ef68a 100644 --- a/README +++ b/README @@ -4,8 +4,8 @@ Fedora PackageDB-cli :Author: Pierre-Yves Chibon :Contact: pingou(a)fedoraproject.org -:Date: Mon May 04, 2015 -:Version: 2.8.2 +:Date: Thu Sep 03, 2015 +:Version: 2.9 The `PackageDB-cli` is a commande line interface to the PackageDB of Fedora. diff --git a/setup.py b/setup.py index a1a919f..823a904 100644 --- a/setup.py +++ b/setup.py @@ -8,7 +8,7 @@ from setuptools import setup setup( name='packagedb-cli', description='A command line tool to access the Fedora Package Database.', - version='2.8.2', + version='2.9', license='GPLv2+', download_url='https://fedorahosted.org/releases/p/a/packagedb-cli/', url='https://fedorahosted.org/packagedb-cli/',

1 0

[packagedb-cli: 7/7] Merge pull request #39 from fedora-infra/py3
by Pierre-YvesChibon 03 Sep '15

03 Sep '15

commit 71203b544dae17cfb9a93547b594d287371b42f7 Merge: 3e3a19e 25d004e Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Thu Sep 3 10:08:57 2015 +0200 Merge pull request #39 from fedora-infra/py3 Start py3 compatibility pkgdb2client/cli.py | 111 ++++++++++++++++++++++++++------------------------- 1 files changed, 57 insertions(+), 54 deletions(-) ---

1 0

[packagedb-cli: 6/7] Merge pull request #42 from mizdebsk/argument-name
by Pierre-YvesChibon 03 Sep '15

03 Sep '15

commit 3e3a19e1a689c46d99176219191c9677ce3cf52c Merge: c27380a cf22a78 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Wed Sep 2 12:04:45 2015 +0200 Merge pull request #42 from mizdebsk/argument-name Fix set_koschei_status() argument name pkgdb2client/__init__.py | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) ---

1 0

[packagedb-cli: 5/7] Catch exceptions as to be py3 compatible
by Pierre-YvesChibon 03 Sep '15

03 Sep '15

commit 25d004ee607999ee4f380c9a0e26dec50074c624 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Sun Aug 9 15:03:14 2015 +0200 Catch exceptions as to be py3 compatible pkgdb2client/cli.py | 36 ++++++++++++++++++------------------ 1 files changed, 18 insertions(+), 18 deletions(-) --- diff --git a/pkgdb2client/cli.py b/pkgdb2client/cli.py index 181d353..baf0992 100644 --- a/pkgdb2client/cli.py +++ b/pkgdb2client/cli.py @@ -160,18 +160,18 @@ def get_last_build(packagename, tag): tag = tag + "-updates" try: _get_last_build(packagename, tag) - except Exception, err: - print err + except Exception as err: + print(err) tag = tag + "-testing" try: _get_last_build(packagename, tag) - except Exception, err: - print err + except Exception as err: + print(err) else: try: _get_last_build(packagename, tag) - except Exception, err: - print err + except Exception as err: + print(err) def setup_parser(): @@ -846,8 +846,8 @@ def main(): # Parse the commandline try: arg = parser.parse_args() - except argparse.ArgumentTypeError, err: - print "\nError: {0}".format(err) + except argparse.ArgumentTypeError as err: + print("\nError: {0}".format(err)) return 2 if arg.nocolor: @@ -896,32 +896,32 @@ def main(): except KeyboardInterrupt: print("\nInterrupted by user.") return_code = 1 - except ServerError, err: + except ServerError as err: LOG.debug('ServerError') print('{0}'.format(err)) return_code = 3 - except ActionError, err: + except ActionError as err: LOG.debug('ActionError') print('{0}'.format(err.message)) return_code = 7 - except argparse.ArgumentError, err: + except argparse.ArgumentError as err: LOG.debug('ArgparseError') print('{0}'.format(err.message)) return_code = 9 - except AppError, err: + except AppError as err: LOG.debug('AppError') print('{0}: {1}'.format(err.name, err.message)) return_code = 4 - except PkgDBException, err: + except PkgDBException as err: LOG.debug('PkgDBException') print('{0}'.format(err)) return_code = 8 - except ValueError, err: - print 'Error: {0}'.format(err) - print 'Did you log in?' + except ValueError as err: + print('Error: {0}'.format(err)) + print('Did you log in?') return_code = 6 - except Exception, err: - print 'Error: {0}'.format(err) + except Exception as err: + print('Error: {0}'.format(err)) logging.exception("Generic error catched:") return_code = 5

1 0

[packagedb-cli: 4/7] Use the print() method instead of the print statement to be py3 compatible
by Pierre-YvesChibon 03 Sep '15

03 Sep '15

commit 74756facbee64e038ed80d64a537ad793833205b Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Sun Aug 9 15:02:57 2015 +0200 Use the print() method instead of the print statement to be py3 compatible pkgdb2client/cli.py | 75 ++++++++++++++++++++++++++------------------------ 1 files changed, 39 insertions(+), 36 deletions(-) --- diff --git a/pkgdb2client/cli.py b/pkgdb2client/cli.py index fb1d234..181d353 100644 --- a/pkgdb2client/cli.py +++ b/pkgdb2client/cli.py @@ -125,13 +125,14 @@ def _get_last_build(packagename, tag): build['version'], build['release']) versions.append(nvr) - print "{0}Last build:{1}{2} by {3} for {4} in {5}".rstrip().format( + print("{0}Last build:{1}{2} by {3} for {4} in {5}".rstrip().format( " " * 8, " " * 5, build['completion_time'].split(" ")[0], build['owner_name'], nvr, tag) + ) def get_last_build(packagename, tag): @@ -425,14 +426,14 @@ def do_acl(args): args.branch = None output = pkgdbclient.get_package(args.package, branches=args.branch) - print 'Fedora Package Database -- {0}'.format(args.package) + print('Fedora Package Database -- {0}'.format(args.package)) if output['packages']: - print output['packages'][0]['package']['summary'] + print(output['packages'][0]['package']['summary']) if args.extra: # print the number of opened bugs LOG.debug("Query bugzilla") bugbz = pkgdb2client.utils.get_bugz(args.package) - print "{0} bugs open (new, assigned, needinfo)".format(len(bugbz)) + print("{0} bugs open (new, assigned, needinfo)".format(len(bugbz))) for pkg in sorted( output['packages'], @@ -445,23 +446,24 @@ def do_acl(args): owner = RED + owner + RESET # Retrieve ACL information - print "\n{0}{1}{2}{3}Point of Contact:{4}{5}".rstrip().format( + print("\n{0}{1}{2}{3}Point of Contact:{4}{5}".rstrip().format( RED + BOLD, pkg['collection']['branchname'], RESET, " " * (8 - len(pkg['collection']['branchname'])), " " * 5, owner) + ) # print header of the table tmp = " " * 24 for acl in ["watchbugzilla", "watchcommits", "commit", "approveacls"]: tmp = tmp + acl + " " * (16 - len(acl)) - print tmp.rstrip() + print(tmp.rstrip()) # print ACL information - print "{0}ACLs:".format(" " * 8) + print("{0}ACLs:".format(" " * 8)) if 'acls' in pkg: acls = _get_acls_info(pkg['acls']) users = sorted(acls, key=lambda user: user.replace('group::', '')) @@ -480,9 +482,9 @@ def do_acl(args): else: tmp = tmp + " " * 16 if tmp is not None and tmp.strip() != "": - print tmp + print(tmp) else: - print ' No ACLs found' + print(' No ACLs found') # print the last build if args.extra: @@ -523,7 +525,7 @@ def do_give(args): output = pkgdbclient.update_package_poc( pkgs, branches, username, former_poc=former_poc) for msg in output.get('messages', []): - print msg + print(msg) def do_list(args): @@ -587,10 +589,10 @@ def do_list(args): if args.name_only: out = " " + pkg['name'] - print out.encode('utf-8') + print(out.encode('utf-8')) cnt = cnt + 1 if not args.name_only: - print 'Total: {0} packages'.format(cnt) + print('Total: {0} packages'.format(cnt)) def do_orphan(args): @@ -629,8 +631,8 @@ def do_orphan(args): 'dead.package?h={1}'.format(pkg_name, pkg_branch) req = requests.get(dead_url) if req.status_code != 200 or not req.text.strip(): - print 'No `dead.package` for %s on %s, please use '\ - '`fedpkg retire`' % (pkg_name, pkg_branch) + print('No `dead.package` for %s on %s, please use '\ + '`fedpkg retire`' % (pkg_name, pkg_branch)) return output = pkgdbclient.retire_packages(pkgs, branches) else: @@ -638,7 +640,7 @@ def do_orphan(args): pkgs, branches, former_poc=former_poc) for msg in output.get('messages', []): - print msg + print(msg) def do_pending(args): @@ -648,8 +650,8 @@ def do_pending(args): LOG.info("user : {0}".format(args.user or args.username)) output = pkgdbclient.get_pending_acls(args.user or args.username) for msg in output.get('pending_acls'): - print '%(user)s requested `%(acl)s` on %(package)s ' \ - '(%(collection)s)' % msg + print('%(user)s requested `%(acl)s` on %(package)s ' \ + '(%(collection)s)' % msg) def do_unorphan(args): @@ -680,7 +682,7 @@ def do_unorphan(args): output = pkgdbclient.unorphan_packages(pkgs, branches, username) for msg in output.get('messages', []): - print msg + print(msg) def do_request(args): @@ -719,7 +721,7 @@ def do_request(args): user=pkgdbclient.username) for msg in output.get('messages', []): - print msg + print(msg) def do_update(args): @@ -762,7 +764,7 @@ def do_update(args): status=status, user=args.user) for msg in output.get('messages', []): - print msg + print(msg) def do_branch(args): @@ -779,11 +781,12 @@ def do_branch(args): cnt = 0 for pkg in branches['collections']: name = '{0} {1}'.format(pkg['name'], pkg['version']) - print " " + pkg['branchname'] + \ - ' ' * (20 - len(pkg['branchname'])) + name + \ + print(" " + pkg['branchname'] + + ' ' * (20 - len(pkg['branchname'])) + name + ' ' * (20 - len(name)) + pkg['status'] + ) cnt = cnt + 1 - print 'Total: {0} collections'.format(cnt) + print('Total: {0} collections'.format(cnt)) def do_monitoring(args): @@ -802,13 +805,13 @@ def do_monitoring(args): pkg = pkgdbclient.get_package( args.package, branches='master', acls=False )['packages'][0]['package'] - print "Current monitoring status of {0} is: {1}".format( - pkg['name'], pkg['monitor']) + print("Current monitoring status of {0} is: {1}".format( + pkg['name'], pkg['monitor'])) else: output = pkgdbclient.set_monitoring_status( args.package, args.monitoring) - print output.get( - 'messages', 'Invalid output returned, please contact an admin') + print(output.get( + 'messages', 'Invalid output returned, please contact an admin')) def do_koschei(args): @@ -862,7 +865,7 @@ def main(): global pkgdbclient if arg.pkgdburl: - print "Querying pkgdb at: %s" % arg.pkgdburl + print("Querying pkgdb at: %s" % arg.pkgdburl) pkgdbclient = PkgDB( arg.pkgdburl, login_callback=pkgdb2client.ask_password) @@ -872,12 +875,12 @@ def main(): if arg.bzurl: if not arg.bzurl.endswith('xmlrpc.cgi'): arg.bzurl = '%s/xmlrpc.cgi' % arg.bzurl - print "Querying bugzilla at: %s" % arg.pkgdburl + print("Querying bugzilla at: %s" % arg.pkgdburl) utils.BZCLIENT.url = arg.bzurl utils.BZCLIENT._sslverify = not arg.insecure if arg.fasurl: - print "Querying FAS at: %s" % arg.pkgdburl + print("Querying FAS at: %s" % arg.pkgdburl) utils.FASCLIENT.base_url = arg.fasurl utils.FASCLIENT.insecure = arg.insecure @@ -891,27 +894,27 @@ def main(): try: arg.func(arg) except KeyboardInterrupt: - print "\nInterrupted by user." + print("\nInterrupted by user.") return_code = 1 except ServerError, err: LOG.debug('ServerError') - print '{0}'.format(err) + print('{0}'.format(err)) return_code = 3 except ActionError, err: LOG.debug('ActionError') - print '{0}'.format(err.message) + print('{0}'.format(err.message)) return_code = 7 except argparse.ArgumentError, err: LOG.debug('ArgparseError') - print '{0}'.format(err.message) + print('{0}'.format(err.message)) return_code = 9 except AppError, err: LOG.debug('AppError') - print '{0}: {1}'.format(err.name, err.message) + print('{0}: {1}'.format(err.name, err.message)) return_code = 4 except PkgDBException, err: LOG.debug('PkgDBException') - print '{0}'.format(err) + print('{0}'.format(err)) return_code = 8 except ValueError, err: print 'Error: {0}'.format(err)

1 0

[packagedb-cli: 3/7] Fix set_koschei_status() argument name
by Pierre-YvesChibon 03 Sep '15

03 Sep '15

commit cf22a789c47e18d35c1c248b4b57b3ab84cae098 Author: Mikolaj Izdebski <mizdebsk(a)redhat.com> Date: Wed Sep 2 11:58:03 2015 +0200 Fix set_koschei_status() argument name pkgdb2client/__init__.py | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) --- diff --git a/pkgdb2client/__init__.py b/pkgdb2client/__init__.py index 6c949ee..f229fec 100644 --- a/pkgdb2client/__init__.py +++ b/pkgdb2client/__init__.py @@ -1118,9 +1118,9 @@ class PkgDB(object): :arg pkgname: The name of the package :type pkgname: str - :arg monitoring: The koschei status to set the package to. Can + :arg koschei The koschei status to set the package to. Can be any of: True, 1, False, 0. - :type monitoring: str + :type koschei str ''' valid = ['true', '1', 'false', '0'] @@ -1131,7 +1131,7 @@ class PkgDB(object): ) args = { 'package': pkgname, - 'status': monitoring, + 'status': koschei, } return self.handle_api_call( - '/package/%s/koschei/%s' % (pkgname, monitoring), data=args) + '/package/%s/koschei/%s' % (pkgname, koschei), data=args)

1 0

[packagedb-cli: 2/7] Merge pull request #40 from fedora-infra/get_version
by Pierre-YvesChibon 03 Sep '15

03 Sep '15

commit c27380adf267fa3cbf2bc78ed8c9a4508d8fd184 Merge: 768962f b71c737 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Wed Sep 2 12:00:57 2015 +0200 Merge pull request #40 from fedora-infra/get_version Retrieve the version before doing something with it pkgdb2client/cli.py | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) ---

1 0

[packagedb-cli: 1/7] Retrieve the version before doing something with it
by Pierre-YvesChibon 03 Sep '15

03 Sep '15

commit b71c737d438ccc1f5a0481e8ae50aba47bfe1f91 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Wed Sep 2 10:05:00 2015 +0200 Retrieve the version before doing something with it pkgdb2client/cli.py | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) --- diff --git a/pkgdb2client/cli.py b/pkgdb2client/cli.py index 74afeef..fb1d234 100644 --- a/pkgdb2client/cli.py +++ b/pkgdb2client/cli.py @@ -793,6 +793,7 @@ def do_monitoring(args): LOG.info("package : {0}".format(args.package)) LOG.info("monitoring : {0}".format(args.monitoring)) + version = pkgdbclient.get_version() if version < (1, 13): raise PkgDBException( 'This version of PkgDB does not support monitoring') @@ -818,6 +819,7 @@ def do_koschei(args): LOG.info("package : {0}".format(args.package)) LOG.info("koschei : {0}".format(args.koschei)) + version = pkgdbclient.get_version() if version < (1, 16): raise PkgDBException( 'This version of PkgDB does not support koschei monitoring')

1 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

packagedb-cli-commits September 2015