pbr isn't required at runtime

Bug #1506679 reported by graingert
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
pylockfile
Fix Released
Undecided
graingert

Bug Description

delete requirements.txt and remove it from tox.ini

tests still passes, package is still installable and runnable.

wheel users never install pbr, sdist users temporarily install pbr as a setup requirement

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to pylockfile (master)

Fix proposed to branch: master
Review: https://review.openstack.org/235633

Changed in pylockfile:
assignee: nobody → graingert (tagrain)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to pylockfile (master)

Reviewed: https://review.openstack.org/235633
Committed: https://git.openstack.org/cgit/openstack/pylockfile/commit/?id=d5afa7891925eb790dfc7d670287018e1e5e9e4c
Submitter: Jenkins
Branch: master

commit d5afa7891925eb790dfc7d670287018e1e5e9e4c
Author: Thomas Grainger <email address hidden>
Date: Fri Oct 16 00:51:33 2015 +0100

    PBR setup requirement only

    Change-Id: Ifcf7227bc14e8d74e0cd8423c127464b8fd8064a
    Closes-Bug: #1506679

Changed in pylockfile:
status: In Progress → Fix Committed
Revision history for this message
Stefan Praszalowicz (stfp) wrote :

It would be great if someone could push this to pypi.

Right now the version that's on there still has "Requires-Dist: pbr (>=0.6,!=0.7,<1.0)", which conflicts with, for example, mock (which has Requires-Dist: pbr (>=0.11)).

Revision history for this message
graingert (tagrain) wrote : Re: [Bug 1506679] Re: pbr isn't required at runtime

Out of interest does mock need pbr either?
On 4 Nov 2015 6:01 p.m., "Stefan Praszalowicz" <email address hidden> wrote:

> It would be great if someone could push this to pypi.
>
> Right now the version that's on there still has "Requires-Dist: pbr
> (>=0.6,!=0.7,<1.0)", which conflicts with, for example, mock (which has
> Requires-Dist: pbr (>=0.11)).
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1506679
>
> Title:
> pbr isn't required at runtime
>
> Status in pylockfile:
> Fix Committed
>
> Bug description:
> delete requirements.txt and remove it from tox.ini
>
> tests still passes, package is still installable and runnable.
>
> wheel users never install pbr, sdist users temporarily install pbr as
> a setup requirement
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/pylockfile/+bug/1506679/+subscriptions
>

Revision history for this message
Seth Rosenblum (ricklerre) wrote :

Ping on this, this is causing lots of version conflicts, with python-keystoneclient and oslo for instance.

Revision history for this message
graingert (tagrain) wrote :

Probably those packages don't need pbr either
On 12 Nov 2015 23:11, "Seth Rosenblum" <email address hidden> wrote:

> Ping on this, this is causing lots of version conflicts, with python-
> keystoneclient and oslo for instance.
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1506679
>
> Title:
> pbr isn't required at runtime
>
> Status in pylockfile:
> Fix Committed
>
> Bug description:
> delete requirements.txt and remove it from tox.ini
>
> tests still passes, package is still installable and runnable.
>
> wheel users never install pbr, sdist users temporarily install pbr as
> a setup requirement
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/pylockfile/+bug/1506679/+subscriptions
>

Revision history for this message
graingert (tagrain) wrote :

Yup keystone doesn't need pbr at runtime either
On 12 Nov 2015 23:42, "Thomas Grainger" <email address hidden> wrote:

> Probably those packages don't need pbr either
> On 12 Nov 2015 23:11, "Seth Rosenblum" <email address hidden> wrote:
>
>> Ping on this, this is causing lots of version conflicts, with python-
>> keystoneclient and oslo for instance.
>>
>> --
>> You received this bug notification because you are subscribed to the bug
>> report.
>> https://bugs.launchpad.net/bugs/1506679
>>
>> Title:
>> pbr isn't required at runtime
>>
>> Status in pylockfile:
>> Fix Committed
>>
>> Bug description:
>> delete requirements.txt and remove it from tox.ini
>>
>> tests still passes, package is still installable and runnable.
>>
>> wheel users never install pbr, sdist users temporarily install pbr as
>> a setup requirement
>>
>> To manage notifications about this bug go to:
>> https://bugs.launchpad.net/pylockfile/+bug/1506679/+subscriptions
>>
>

Changed in pylockfile:
milestone: none → 0.12.0
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.