[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#809272: RFS: pygithub/1.26.0-1 ITP



control: tag -1 moreinfo
control: retitle -1 RFS: pygithub/1.26.0-1

On Mon, Dec 28, 2015 at 04:29:54PM -0500, Dmitry Bogatov wrote:
>   * New upstream release
>   * Update watch file to use pypi.debian.net
>   * Disable tests, since they need network
>   * Disable installation of test modules and data
>   * Rename binary packages (python-pygithub -> python-github)
>     (Closes: #808467)

This package is quite cool already, just two bits:

* Vcs-* points to github, which 404 (same as the other, I guess)
* please consider handing a spellchecked to upstream, looks like there
  are some typos
* also duck has something to say about the copyright:
  E: debian/copyright:30: Email: pmuilu@xena: ERROR (Certainty:possible)
     pmuilu@xena: No MX entry found.
     pmuilu@xena: No A entry found.
     pmuilu@xena: No AAAA entry found.
(I must admin check-all-the-things is really noisy over this package...)


wrt the migration:

there is only one r-dep in unstable, which is jeepyb.
I personally find annoying breaking other packages for fun, when there
is really no need for it.
IMHO the better action is to add a
    Provides: python-pygithub
to the python-github stanza (since there are no python3-pygithub r-deps
I'd avoid one there).
After this, I'd open a bug against jeepyb to request the swap of the
dependency, and once that's done, remove the provides in the next
upload of pygithub.

I'd welcome words from other mentors about this, though.
I've never migreted packages myself, so it's kinda new territory for me
too.

-- 
regards,
                        Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540      .''`.
more about me:  http://mapreri.org                              : :'  :
Launchpad user: https://launchpad.net/~mapreri                  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-

Attachment: signature.asc
Description: PGP signature


Reply to: