[Bug 1105050] New: Review Request: python-distlib - Low-level components of distutils2/packaging, augmented with higher-level APIs

[Red Hat Bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1105050

Bug ID: 1105050
           Summary: Review Request: python-distlib - Low-level components
                    of distutils2/packaging, augmented with higher-level
                    APIs
           Product: Fedora
           Version: rawhide
         Component: Package Review
          Severity: medium
          Priority: medium
          Assignee: [e-mail]
          Reporter: [e-mail]
        QA Contact: [e-mail]
                CC: [e-mail]

Spec URL: http://mstuchli.fedorapeople.org/python-distlib.spec
SRPM URL: http://mstuchli.fedorapeople.org/python-distlib-0.1.9-1.fc20.src.rpm
Description: Distlib contains the implementations of the packaging PEPs and
other low-level
features which relate to packaging, distribution and deployment of Python
software. If Distlib can be made genuinely useful, then it is possible for
third-party packaging tools to transition to using it. Their developers and
users then benefit from standardised implementation of low-level functions,
time saved by not having to reinvent wheels, and improved interoperability
between tools.

Fedora Account System Username: mstuchli

koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6926117

[Bug 1105050] Review Request: python-distlib - Low-level components of distutils2/packaging, augmented with higher-level APIs

[Red Hat Bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1105050

Robert Kuska  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |[e-mail]
           Assignee|[e-mail]    |[e-mail]
              Flags|                            |fedora-review?

[Bug 1105050] Review Request: python-distlib - Low-level components of distutils2/packaging, augmented with higher-level APIs

[Red Hat Bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1105050

--- Comment #1 from Robert Kuska  ---
BuildRequires:  python-setuptools
and
BuildRequires:  python3-setuptools

are actualy not needed.

Also distlib vendorize some python libs in distlib/_backport (tarfile.py,
sysconfig.py, shutil.py).

cat distlib/_backport/__init__.py
"""Modules copied from Python 3 standard libraries, for internal use only.

Individual classes and functions are found in d2._backport.misc.  Intended
usage is to always import things missing from 3.1 from that module: the
built-in/stdlib objects will be used if found.
"""

There is SKIP_SLOW[1] env variable to skip some of those tests which fail, but
it's only small bonus. :-)

[1] http://pythonhosted.org//distlib/tutorial.html#pypi-availability

[Bug 1105050] Review Request: python-distlib - Low-level components of distutils2/packaging, augmented with higher-level APIs

[Red Hat Bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1105050

--- Comment #2 from Matej Stuchlik  ---
(In reply to Robert Kuska from comment #1)
> BuildRequires:  python-setuptools
> and
> BuildRequires:  python3-setuptools
>
> are actualy not needed.

Good point.
> Also distlib vendorize some python libs in distlib/_backport (tarfile.py,
> sysconfig.py, shutil.py).
>
> cat distlib/_backport/__init__.py
> """Modules copied from Python 3 standard libraries, for internal use only.
>
> Individual classes and functions are found in d2._backport.misc.  Intended
> usage is to always import things missing from 3.1 from that module: the
> built-in/stdlib objects will be used if found.
> """

Well done noticing this, it did not occur to me to check, will fix.
> There is SKIP_SLOW[1] env variable to skip some of those tests which fail,
> but it's only small bonus. :-)
>
> [1] http://pythonhosted.org//distlib/tutorial.html#pypi-availability

I did notice that, however since, as you note, it only skips some of the
failing tests I chose not to use it.

[Bug 1105050] Review Request: python-distlib - Low-level components of distutils2/packaging, augmented with higher-level APIs

[Red Hat Bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1105050

--- Comment #3 from Matej Stuchlik  ---
Updated the spec.

[Bug 1105050] Review Request: python-distlib - Low-level components of distutils2/packaging, augmented with higher-level APIs

[Red Hat Bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1105050

Robert Kuska  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|fedora-review?              |fedora-review+

--- Comment #4 from Robert Kuska  ---
Looks nice, approved.

[Bug 1105050] Review Request: python-distlib - Low-level components of distutils2/packaging, augmented with higher-level APIs

[Red Hat Bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1105050

Matej Stuchlik  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |fedora-cvs?

--- Comment #5 from Matej Stuchlik  ---
New Package SCM Request
=======================
Package Name: python-distlib
Short Description: Low-level components of distutils2/packaging, augmented with
higher-level APIs
Upstream URL: https://bitbucket.org/vinay.sajip/distlib
Owners: mstuchli
Branches: 
InitialCC:

[Bug 1105050] Review Request: python-distlib - Low-level components of distutils2/packaging, augmented with higher-level APIs

[Red Hat Bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1105050

Jon Ciesla  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|fedora-cvs?                 |fedora-cvs+

[Bug 1105050] Review Request: python-distlib - Low-level components of distutils2/packaging, augmented with higher-level APIs

[Red Hat Bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1105050

--- Comment #6 from Jon Ciesla  ---
Git done (by process-git-requests).

[Bug 1105050] Review Request: python-distlib - Low-level components of distutils2/packaging, augmented with higher-level APIs

[Red Hat Bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1105050

Matej Stuchlik  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|---                         |NEXTRELEASE
        Last Closed|                            |2014-06-06 08:50:15