Page MenuHomeFreeBSD

www/py-django-crispy-forms: fix django dependency
ClosedPublic

Authored by xmj on Oct 7 2014, 12:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Feb 13, 8:59 PM
Unknown Object (File)
Tue, Jan 28, 6:36 PM
Unknown Object (File)
Nov 23 2024, 7:20 PM
Unknown Object (File)
Nov 9 2024, 4:21 AM
Unknown Object (File)
Oct 27 2024, 3:26 PM
Unknown Object (File)
Oct 11 2024, 9:27 AM
Unknown Object (File)
Oct 2 2024, 7:05 PM
Unknown Object (File)
Jul 27 2024, 6:32 AM
Subscribers
None

Details

Reviewers
swills
koobs
Summary
  • Fix django dependency, now that www/py-django was bumped to 1.7

    www/py-django-crispy-forms: fix django dependency
    • Fix django dependency, now that www/py-django was bumped to 1.7

      (DiffRev:...) Approved by: swills (mentor)
Test Plan

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

xmj retitled this revision from to www/py-django-crispy-forms: fix django dependency.
xmj updated this object.
xmj edited the test plan for this revision. (Show Details)
xmj added reviewers: swills, koobs.

Are you sure we support this syntax:

RUN_DEPENDS= ${PYTHON_PKGNAMEPREFIX}django<1.7:${PORTSDIR}/www/py-django16

I think it should be

RUN_DEPENDS= ${PYTHON_PKGNAMEPREFIX}django16>=1.6:${PORTSDIR}/www/py-django16

xmj edited edge metadata.

Amend Makefile

the package name for django 1.6 is "django16" not "django", so if it needs to pull that in, it should be:

RUN_DEPENDS= ${PYTHON_PKGNAMEPREFIX}django16>=1.6:${PORTSDIR}/www/py-django16

If 1.7 works, then it should be:

RUN_DEPENDS= ${PYTHON_PKGNAMEPREFIX}django>=1.7:${PORTSDIR}/www/py-django

I don't know if this supports 1.7 or not.

In D909#6, @swills wrote:

the package name for django 1.6 is "django16" not "django", so if it needs to pull that in, it should be:

RUN_DEPENDS= ${PYTHON_PKGNAMEPREFIX}django16>=1.6:${PORTSDIR}/www/py-django16

Yeah, should've seen that.

If 1.7 works, then it should be:

RUN_DEPENDS= ${PYTHON_PKGNAMEPREFIX}django>=1.7:${PORTSDIR}/www/py-django

I don't know if this supports 1.7 or not.

Around the time when I made that port it (explicitly) didn't support 1.7, and hasn't been updated in a while nor touched on GitHub..

swills edited edge metadata.

Looks good!

This revision is now accepted and ready to land.Oct 7 2014, 2:39 PM

Committed revision 370345.