#1886 cli: make list-signed accepting integer params
Merged 4 years ago by tkopecek. Opened 4 years ago by julian8628.
julian8628/koji issue/1884  into  master

file modified
+18 -7
@@ -2040,18 +2040,24 @@ 

      if options.key:

          qopts['sigkey'] = options.key

      if options.rpm:

-         rinfo = session.getRPM(options.rpm)

+         rpm_info = options.rpm

+         try:

+             rpm_info = int(rpm_info)

+         except ValueError:

+             pass

+         rinfo = session.getRPM(rpm_info, strict=True)

          rpm_idx[rinfo['id']] = rinfo

-         if rinfo is None:

-             parser.error(_("No such RPM: %s") % options.rpm)

          if rinfo.get('external_repo_id'):

              parser.error(_("External rpm: %(name)s-%(version)s-%(release)s.%(arch)s@%(external_repo_name)s") % rinfo)

          qopts['rpm_id'] = rinfo['id']

      if options.build:

-         binfo = session.getBuild(options.build)

+         build = options.build

+         try:

+             build = int(build)

+         except ValueError:

+             pass

+         binfo = session.getBuild(build, strict=True)

          build_idx[binfo['id']] = binfo

-         if binfo is None:

-             parser.error(_("No such build: %s") % options.rpm)

          sigs = []

          rpms = session.listRPMs(buildID=binfo['id'])

          for rinfo in rpms:
@@ -2060,7 +2066,12 @@ 

      else:

          sigs = session.queryRPMSigs(**qopts)

      if options.tag:

-         rpms, builds = session.listTaggedRPMS(options.tag, inherit=False, latest=False)

+         tag = options.tag

+         try:

+             tag = int(tag)

+         except ValueError:

+             pass

+         rpms, builds = session.listTaggedRPMS(tag, inherit=False, latest=False)

          tagged = {}

          for binfo in builds:

              build_idx.setdefault(binfo['id'], binfo)

pretty please pagure-ci rebuild

4 years ago

Metadata Update from @tkopecek:
- Pull-request tagged with: testing-ready

4 years ago

rebased onto 3375162

4 years ago

Metadata Update from @jcupova:
- Pull-request tagged with: testing-done

4 years ago

Commit b593001 fixes this pull-request

Pull-Request has been merged by tkopecek

4 years ago