Page MenuHomeFreeBSD
Differential D12592

New USES+=django and Django default version
AbandonedPublic
Actions

Authored by matthew on Oct 4 2017, 10:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 28, 10:06 PM
Unknown Object (File)
Wed, Nov 27, 10:48 PM
Unknown Object (File)
Tue, Nov 26, 4:50 PM
Unknown Object (File)
Tue, Nov 26, 4:34 PM
Unknown Object (File)
Mon, Nov 25, 5:54 PM
Unknown Object (File)
Mon, Nov 25, 4:23 PM
Unknown Object (File)
Mon, Nov 25, 2:45 AM
Unknown Object (File)
Sat, Nov 23, 8:27 PM
View All Files
Subscribers
antoine
dbaio
FreeBSD_ShaneWare.Biz
leres
mat
sunpoet
swills

Details

Reviewers
ultima
kai
sbz
Group Reviewers
O5: Ports Framework(Owns No Changed Paths)
portmgr
Python
Summary

Create a new Mk/Uses/django.mk (largely copied from lua.mk) which adds
a RUN_DEPENDS on the users preferred choice of Django version. Choose
from 1.11, 1.10 or 1.8 currently. Also generate a
DJANGO_PKGNAMEPREFIX which appends the django version to
PYTHON_PKGNAMEPREFIX (so the final result is eg. py27-django118- )

Add a DJANGO_DEFAULT setting to bsd.default-versions.mk

Set the ports default to django-1.11, which is the upstream
recommended default, rather than the current django-1.8

Convert all the ports that have a run-time dependency on Django to
USES+=django

While here convert one port to options helpers and (not really
relevant for this) update another as well as converting to the new
USES+=django

PR: 222724, 224812 (Exp-run)

Diff Detail

Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 29065
Build 27017: arc lint + arc unit

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
matthew added a reviewer: ultima.Oct 5 2017, 6:45 AM
matthew edited the summary of this revision. (Show Details)Oct 5 2017, 8:49 AM
ultima added a comment.Oct 6 2017, 4:05 AM

I'm not positive seahub will run properly on the newest version of django, I know that in the past seahub broke from running a newer version than upstream. databases/py-mysqldb seems to also be broken atm preventing me from doing QA. I'll take a look and see if its a simple fix and do some testing.

sunpoet added a subscriber: sunpoet.Oct 6 2017, 9:27 AM

I would suggest to use USES=django:1.11 instead of USES=django:111 to be consistent with bsd.default-versions.mk and upstream versioning,

Since djano ports cannot coexist, I'm not sure if DJANGO_PKGNAMEPREFIX is necessary.

databases/py-carbon/Makefile
6 ↗(On Diff #33711)

You need to bump PORTREVISION for the dependency change (from django18 to django111). Other ports may have same issue.

matthew added a comment.Oct 6 2017, 10:59 AM
In D12592#261434, @sunpoet wrote:

I would suggest to use USES=django:1.11 instead of USES=django:111 to be consistent with bsd.default-versions.mk and upstream versioning,

Yes -- that's a good idea.

Since djano ports cannot coexist, I'm not sure if DJANGO_PKGNAMEPREFIX is necessary.

Think about installing from package repositories and how things will work in the new flavorful world. This would be better solved by
variable dependencies, but until that lands having multiple packages in the repo that differ only by their dependency tree is the only
available solution.

databases/py-carbon/Makefile
6 ↗(On Diff #33711)

Yes. PORTREVISION bumps all round.

mat added inline comments.Oct 6 2017, 3:56 PM
Mk/Uses/django.mk
11 ↗(On Diff #33711)

I do not think adding python to post is needed, all ports should be using USES=python too.

matthew added inline comments.Oct 6 2017, 4:18 PM
Mk/Uses/django.mk
11 ↗(On Diff #33711)

Ah -- OK. That's me not understanding what _USES_POST actually does. I thought it was all about controlling the order that Uses are processed in.

mat added inline comments.Oct 6 2017, 4:30 PM
Mk/Uses/django.mk
11 ↗(On Diff #33711)

The Uses are processed in the order they are given.

Uses are invoked a first time during bsd.port.pre.mk, and _USES_POST is used to say that this USES has a "post" section that will be invoked later, in bsd.port.post.mk.

matthew updated this revision to Diff 33821.Oct 8 2017, 5:17 PM

Make version numbers include the decimal point like so:
USES+=django:1.11 -- consistent with DEFAULT_VERSIONS. This also
affects DJANGO_PKGNAMEPREFIX, which now looks like py27-django1.11-

Drop use of _USES_POST

matthew marked 5 inline comments as done.Oct 8 2017, 5:19 PM
ultima added inline comments.Oct 9 2017, 9:31 PM
www/seahub/Makefile
35–36

Just finished some testing with Django 1.11, Seahub is not compatible with the newest version and requires 1.8. I tried to change to django:1.8 and the build will fail with
Ignore: Port requires Django version 1.8 which cannot be satisfied by the default version 1.11.

I assume this new USE is being created for the flavor feature and that would probably resolve this issue, though I wonder if there would be conflicts with multiple Django versions installed or if this is even a possible or a desired scenario.

matthew added inline comments.Oct 10 2017, 6:13 AM
www/seahub/Makefile
35–36

Ah, OK. This should certainly have USES= django:1.8 then

You would also need DEFAULT_VERSIONS+= django=1.8

Yes, I was thinking about how this should work in a FLAVOURed ports tree. Given that seahub depends on a number of other Django modules, to support it properly we'd have to build several FLAVOURS of those modules.

The three different possible versions for each Django port conflict with each other, so if one of your applications requires a particular version of Django, then all of the Django apps on that system would have to use that version.

As it stands, this means that there wouldn't be precompiled Seahub packages available from the default ports repo.

matthew updated this revision to Diff 33860.Oct 10 2017, 6:38 AM

www/seahub requires django-1.8

www/py-jsonfield add CONFLICTS_INSTALL, tracking upstream changes

matthew marked 2 inline comments as done.Oct 10 2017, 6:39 AM
sunpoet added a comment.Oct 10 2017, 1:54 PM
In D12592#262055, @matthew wrote:

Make version numbers include the decimal point like so:
USES+=django:1.11 -- consistent with DEFAULT_VERSIONS. This also
affects DJANGO_PKGNAMEPREFIX, which now looks like py27-django1.11-

Please remove the . in DJANGO_PKGNAMEPREFIX.
That means DJANGO_PKGNAMEPREFIX will look like py27-django111-

BTW, DJANGO_PKGNAMEPREFIX is quite a long prefix (14-15 chars).

matthew updated this revision to Diff 34003.Oct 15 2017, 9:58 AM

Remove the '.' from the Django version in DJANGO_PKGNAMEPREFIX, so it
now looks like: py27-django111-

Fix the CONFLICTS_INSTALL settings in www/py-django-happenings +
www/py-django-mptt; www/py-django-registration +
www/py-django-registration-redux

matthew updated this revision to Diff 34204.Oct 21 2017, 6:49 AM

Merge upstream changes

matthew updated this revision to Diff 34206.Oct 21 2017, 7:20 AM
  • Revert accidental inclusion of sysutils/rsyslog8
  • Revert accidental inclusion of devel/libfastjson
  • Whitespace changes only
matthew updated this revision to Diff 35451.Nov 19 2017, 11:06 AM

Track upstream changes

  • Merge branch 'master' into arcpatch-D12592
  • Merge branch 'master' into arcpatch-D12592
  • Merge branch 'master' into arcpatch-D12592
Harbormaster completed remote builds in B12856: Diff 35451.Nov 19 2017, 11:06 AM
FreeBSD_ShaneWare.Biz added a subscriber: FreeBSD_ShaneWare.Biz.Dec 12 2017, 12:22 PM
FreeBSD_ShaneWare.Biz added inline comments.
Mk/Uses/django.mk
13 ↗(On Diff #35451)

py-django20 has landed in ports so 2.0 can be added here.

84 ↗(On Diff #35451)

Removing '.' leads to DJANGO_VER being 111,110,18,20 - meaning 2.0 will compare as less than 1.10. Could DJANGO_VER be made as three digit? (108,110,111,200)

Maybe add DJANGO_REL as three digit similar to PYTHON_REL.

mat added a comment.Dec 12 2017, 12:33 PM

So, this is the time to update this and make it use flavors no ?

matthew added a comment.Dec 12 2017, 1:30 PM
In D12592#281340, @mat wrote:

So, this is the time to update this and make it use flavors no ?

Yes, it is. Although I'm a bit short on time right now, I shall be updating this fairly soon.

matthew updated this revision to Diff 36754.Dec 19 2017, 6:43 AM

Merge upstream changes -- ie. package flavours. www/seahub is
problematic, causing the whole poudriere bulk to fail if included in
the list of package targets. Other than that, builds correctly with
both py27 and py36.

Creating django111 and django20 flavours would be interesting, but
that implies packages with multiple flavors which is a problem.

  • Use the Django pkgname prefix for dependencies
  • Add django-2.0 to the list of possible versions.

ToDo:

fix www/seahub
DJANGO_REL -- release value equivalent to PYTHON_REL
ultima added a comment.Dec 20 2017, 7:53 PM
In D12592#283142, @matthew wrote:

Merge upstream changes -- ie. package flavours. www/seahub is
problematic, causing the whole poudriere bulk to fail if included in
the list of package targets. Other than that, builds correctly with
both py27 and py36.

Creating django111 and django20 flavours would be interesting, but
that implies packages with multiple flavors which is a problem.

  • Use the Django pkgname prefix for dependencies
  • Add django-2.0 to the list of possible versions.

ToDo:

fix www/seahub
DJANGO_REL -- release value equivalent to PYTHON_REL

It maybe time to rename the port to py-seahub. Is this the issue you are running into? Seahub only runs on py27 and django 18. Let me know if there is anything I can assist with.

matthew added a comment.Dec 21 2017, 1:21 PM
In D12592#283656, @ultima wrote:
In D12592#283142, @matthew wrote:

Merge upstream changes -- ie. package flavours. www/seahub is
problematic, causing the whole poudriere bulk to fail if included in
the list of package targets. Other than that, builds correctly with
both py27 and py36.

Creating django111 and django20 flavours would be interesting, but
that implies packages with multiple flavors which is a problem.

  • Use the Django pkgname prefix for dependencies
  • Add django-2.0 to the list of possible versions.

ToDo:

fix www/seahub
DJANGO_REL -- release value equivalent to PYTHON_REL

It maybe time to rename the port to py-seahub. Is this the issue you are running into? Seahub only runs on py27 and django 18. Let me know if there is anything I can assist with.

Well, given the dependencies on both Django and Python, by rights that should be www/py-django-seahub. That however is not precisely the issue. www/seahub builds fine using 'poudriere testbuild' but not when using 'poudriere bulk' -- this is with python=2.7 and django=1.8 in DEFAULT_VERSIONS. It's not that the www/seahub port fails under 'poudriere bulk'. The entire poudriere run aborts without building anything like so:

lucid-nonsense:...work/github/freebsd-ports:# poudriere bulk -j 11_1a -z development -p github -f ~matthew/work/django-port-list
[00:00:00] Creating the reference jail... done
[00:00:00] Mounting system devices for 11_1a-github-development
[00:00:00] Mounting ports/packages/distfiles
[00:00:00] Stashing existing package repository
[00:00:00] Mounting ccache from: /var/cache/ccache
[00:00:00] Mounting packages from: /usr/local/poudriere/data/packages/11_1a-github-development
[00:00:00] Copying /var/db/ports from: /usr/local/etc/poudriere.d/development-options
[00:00:00] Appending to make.conf: /usr/local/etc/poudriere.d/make.conf
[00:00:00] Appending to make.conf: /usr/local/etc/poudriere.d/development-make.conf
/etc/resolv.conf -> /usr/local/poudriere/data/.m/11_1a-github-development/ref/etc/resolv.conf
[00:00:00] Starting jail 11_1a-github-development
[00:00:00] Logs: /usr/local/poudriere/data/logs/bulk/11_1a-github-development/2017-12-21_10h19m34s
[00:00:00] Loading MOVED
[00:00:00] Ports supports: FLAVORS SELECTED_OPTIONS
[00:00:00] Gathering ports metadata
[00:00:11] Calculating ports order and dependencies
[00:00:11] Error: compute_deps_pkg failed to lookup pkgname for www/py-django@ processing package seahub-6.2.4_1 from www/seahub -- Does www/py-django provide the '' FLAVOR?
[00:00:12] Error: Fatal errors encountered calculating dependencies
[00:00:12] Cleaning up
11_1a-github-development: removed
11_1a-github-development-n: removed
[00:00:12] Unmounting file systems
mat added a comment.Dec 21 2017, 1:32 PM
In D12592#283852, @matthew wrote:

[00:00:11] Error: compute_deps_pkg failed to lookup pkgname for www/py-django@ processing package seahub-6.2.4_1 from www/seahub -- Does www/py-django provide the '' FLAVOR?






This means that www/seahub does not have any flavors, so the RUN_DEPENDS in django.mk ends with ...py-django@ which is interpreted as an empty string flavor.  You probably need to replace this with PY_FLAVOR.
matthew added a comment.Dec 21 2017, 3:50 PM


In D12592#283856, @mat wrote:









This means that www/seahub does not have any flavors, so the RUN_DEPENDS in django.mk ends with ...py-django@ which is interpreted as an empty string flavor.  You probably need to replace this with PY_FLAVOR.



Ahah!  Yes, that solves the problem.
matthew updated this revision to Diff 37299.Dec 31 2017, 9:46 AM
Major rework of the code, modelled on python.mk



    

  • FLAVOR support
    • 

  • Now requires python.mk to preceed djano.mk in the USES definition
    • 

  • Add a script 'version-rel.sh' to turn typical multipart version numbers into sortable integers
    • 

  • Add Makefile.version to each of the Django ports
    • 




This mostly works, but is not yet quite ready for primetime -- test

building all of the Django related ports results with all the

different flavors generates about 700 out of 800 ports correctly.
matthew updated this revision to Diff 37330.Dec 31 2017, 9:21 PM
Fix remaining build problems.  Every Django related port now builds

correctly.



Fixes here involve redefining a number of python related variables

using the Python versions determined from the flavoring.  This seems

inelegant -- in a couple of places it's literally cut'n'paste from

python.mk -- but it works!
matthew marked 3 inline comments as done.Dec 31 2017, 9:26 PM
matthew edited the summary of this revision. (Show Details)Jan 1 2018, 8:26 AM
matthew marked an inline comment as done.Jan 1 2018, 8:28 AM
matthew updated this revision to Diff 37373.Jan 1 2018, 9:00 PM
    

  • Merge branch 'master' into arcpatch-D12592
    • 

matthew updated this revision to Diff 37398.Jan 2 2018, 6:51 AM
    

  • Merge branch 'master' into arcpatch-D12592
    • 

mat added inline comments.Jan 2 2018, 2:38 PM
Mk/Uses/django.mk


202 ↗(On Diff #37398)
I do not see this FLAVORS variable defined beforehand, where does it come from exactly ?
matthew added inline comments.Jan 2 2018, 4:14 PM
Mk/Uses/django.mk


202 ↗(On Diff #37398)
That's what is set in python.mk
matthew updated this revision to Diff 37445.Jan 3 2018, 6:35 AM
    

  • Merge branch 'master' into arcpatch-D12592
    • 

mat added inline comments.Jan 3 2018, 1:08 PM
Mk/Uses/django.mk


202 ↗(On Diff #37398)
Mmm, but django < python, you can absolutely assume that the sort police will put django before python and it will break.  You need to add a .include "${USESDIR}/python.mk" before you use any variables coming from python.mk.
matthew added inline comments.Jan 3 2018, 6:59 PM
Mk/Uses/django.mk


202 ↗(On Diff #37398)
Agreed that there will be some people who sort the USES line, but not for long.  That's why I put in the check for .if !defined(_INCLUDE_USES_PYTHON_MK) at line 70 -- all they'll get is an error message which, I trust, will help to educate them that sorting things is not always a good idea.



Notice that the body of this file is in the .else branch of that test, so none of the python variables will be processed unless python.mk has already been included.



If I include ${USESDIR}/python.mk doesn't that break processing any arguments for the python USES?
matthew updated this revision to Diff 37499.Jan 4 2018, 7:30 AM
    

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Add new py-django-netfields port
    • 

mat added inline comments.Jan 4 2018, 12:27 PM
Mk/Uses/django.mk


202 ↗(On Diff #37398)
USES must work the same way whatever their order is.



USES are self contained, and many include other because of dependencies.  It should not break anything.
matthew added inline comments.Jan 5 2018, 7:13 AM
Mk/Uses/django.mk


202 ↗(On Diff #37398)
That's a problem when you've got several different USES each trying to define FLAVOR and FLAVORS -- it's presumably the last-added USES that wins, which breaks the requirement about invariance to ordering.



I'm thinking we need a rule that any variables that a USES .mk file defines should have a distinct namespace eg. PY_FLAVOR, PY_FLAVORS; DJANGO_FLAVOR, DJANGO_FLAVORS.



For flavour handling specifically, the USES could append their namespace prefix to a generic variable, say FLAVOR_NAMESPACES+=PY and then a separate block of code in bsd.ports.mk to combine the flavors from the different namespaces (sorted alphabetically), with a mechanism to filter out incompatible combinations (eg. py27 doesn't mix with django20).  Individual ports could override this mechanism by setting the un-namespaced variables FLAVOR or FLAVORS as now.
antoine added a comment.Mar 8 2018, 7:07 PM
Wouldn't it be easier to have USE_PYTHON+= django:somearg ?
matthew abandoned this revision.Mar 31 2018, 5:49 PM
Yeah.  This approach is a non-starter.  Adding django bits to python.mk is probably a better route as Antoine suggests,

but that'sbasically a start-over.
leres added a subscriber: leres.Mar 9 2019, 5:59 PM
matthew updated this revision to Diff 67553.Jan 31 2020, 4:00 PM
    

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Remove leftovers from incomplete merging
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Remove the py-dj21-django-foo port variants, as these will now be
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • databases/py-carbon no loger has any django dependencies.
    • 

  • Update Makefile.version in each of the main django ports.
    • 

  • Survey the tree to identify all of the Django related ports.
    • 

  • USE_PYTHON=django:1.11+ is identical to the default USES_PYTHON=django
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Django-flavourize newly added www/py-django-cron port
    • 

  • Fix a mis-merge.  Mk/Scripts/version-rel.sh was removed from the
    • 

  • DJango-2.1 now in the tree
    • 

  • Reduce the diff to the master branch
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • py-dj21-djangoql -> py-djangoql, flavoured for django 2.1+
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • The code at line 299 of Mk/Uses/python.mk already parses an extra
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • New upstream Django 2.2 -- py-django22 port added.
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Start by defining the outputs -- exported variables and added dependencies.
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Two more Django ports
    • 

  • Another pass through all the Django ports, switching to the
    • 

  • Generate the django version as a 5 digit integer
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Django 1.8 was dropped from the ports.
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • py-logan needs a TEST_DEPENDS on django
    • 

  • Delete extraneous blank line
    • 

  • Generate a 3 digit number from the version for comparison purposes.
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Check that the specified Django version fits within any version
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Regenerate FLAVORS adding Django version information.
    • 

  • Remove the separate django.mk now that the functionality has been
    • 

  • Add missing '.'
    • 

  • Initialise DJANGO_PORTSDIR before referencing it.
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Add django to the list of valid categories
    • 

  • Revert the approach of using PY_FLAVOR instead of FLAVOR -- FLAVOR is
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Have only one USE_PYTHON line for best results
    • 

  • Multiple debug fixes
    • 

  • Adjust CONFLICTS_INSTALL
    • 

  • PyPi claims that the maximum supported branch of Django for
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Despite the name, django-modelcluster has no dependency on django, so
    • 

  • Where django is only used as a test dependency, the FLAVOR settings
    • 

  • Whitespace
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • With USE_PYTHON=django, the string 'django' will always appear in
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Set FLAVORS immediately.
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 




This is very much a work in progress and not at all useable yet.
Harbormaster completed remote builds in B29065: Diff 67553.Jan 31 2020, 4:00 PM
matthew added a reviewer: kai.Jan 31 2020, 4:01 PM
matthew updated this revision to Diff 67557.Jan 31 2020, 4:13 PM
    

  • Django 2.1 is gone, django 3.0 is here.
    • 

Harbormaster completed remote builds in B29067: Diff 67557.Jan 31 2020, 4:13 PM
matthew updated this revision to Diff 67559.Jan 31 2020, 4:38 PM
    

  • Merge py-django-rq and py-dj22-django-rq
    • 

  • Tidy up cases where django is bought in as a test dependency
    • 

  • Don't support nodjangoflavor as an option any more.
    • 

Harbormaster completed remote builds in B29068: Diff 67559.Jan 31 2020, 4:38 PM
mat added inline comments.Feb 1 2020, 10:25 AM
Mk/Uses/python.mk


279
This says SUFFIX, should be PREFIX.


316–321
Can you try to only change what you need, adding whitespace and rewrapping differently is not great.


705
By definition, the default flavor is the first of the flavors.



.  if empty(FLAVOR)
FLAVOR:=        ${FLAVORS:[1]}
.  endif
matthew marked 2 inline comments as done.Feb 3 2020, 9:25 AM
matthew added inline comments.
Mk/Uses/python.mk


279
This simply parallels the comment on PYTHON_PKGNAMEPREFIX at lines 242-245:



# PYTHON_PKGNAMEPREFIX
#			- Use this as a ${PKGNAMEPREFIX} to distinguish
#			  packages for different Python versions.
#			  default: py${PYTHON_SUFFIX}-



${PYTHON_SUFFIX} is maybe a badly named variable, but changing it would be a lot of churn.
matthew updated this revision to Diff 67684.Feb 3 2020, 9:32 AM
    

  • Don't re-wrap _VALID_PYTHON_FEATURES
    • 

  • The default flavor is the first of the flavours.
    • 

  • Merge branch 'master' into arcpatch-D12592
    • 

Harbormaster completed remote builds in B29121: Diff 67684.Feb 3 2020, 9:32 AM
matthew updated this revision to Diff 67685.Feb 3 2020, 9:35 AM
Diff against master
Harbormaster completed remote builds in B29122: Diff 67685.Feb 3 2020, 9:35 AM
matthew updated this revision to Diff 67993.Feb 9 2020, 11:30 AM
    

  • Merge branch 'master' into arcpatch-D12592
    • 

Harbormaster completed remote builds in B29249: Diff 67993.Feb 9 2020, 11:30 AM
matthew updated this revision to Diff 68869.Feb 27 2020, 6:58 AM
    

  • Merge pull request #1 from knobix/arcpatch-D12592
    • 

Harbormaster completed remote builds in B29635: Diff 68869.Feb 27 2020, 6:58 AM
FreeBSD_ShaneWare.Biz added a comment.Feb 28 2020, 4:02 AM
While there has been a comment on the length of DJANGO_PKGNAMEPREFIX no one seems to have suggested shortening it.



py37-django30-portname



could be shortened to



py37-dj30-portname



port folder names could be similar - py-dj-portname
matthew added a comment.Feb 28 2020, 7:58 AM


In D12592#524776, @FreeBSD_ShaneWare.Biz wrote:


While there has been a comment on the length of DJANGO_PKGNAMEPREFIX no one seems to have suggested shortening it.



py37-django30-portname



could be shortened to



py37-dj30-portname





This is certainly something to consider -- many Django modules already have 'django' in their name, so having it twice in the package name is pretty redundant.



port folder names could be similar - py-dj-portname



However, I don't see much value in this.  As I said, most Django-related ports already self-identify as such.  This patch already makes hundreds of changes to the tree, and I don't think adding quite a lot of additional churn on top of that, to no real benefit, would be a good idea.
kai mentioned this in D24850: www/py-djangorestframework: Update to 3.11.0.May 21 2020, 1:57 PM
dbaio added a subscriber: dbaio.Oct 20 2020, 2:43 AM
swills added a subscriber: swills.Oct 20 2020, 8:05 PM
sbz resigned from this revision.Apr 19 2021, 5:13 PM
matthew abandoned this revision.Mar 16 2023, 6:03 PM
This is well out of date.
Large Diff
This large diff affects 220 files. Files without inline comments have been collapsed. Expand All Files
Revision Contents
Changeset List
PathSize
Mk/
Uses/
248 lines
13 lines
2 lines
databases/
py-django-transaction-hooks/
8 lines
py-south/
2 lines
deskutils/
py-paperless/
15 lines
devel/
py-django-rq-dashboard/
8 lines
py-django-rq/
11 lines
py-django-tastypie-mongoengine/
11 lines
py-eventlib/
11 lines
py-logan/
8 lines
py-pypugjs/
2 lines
py-pytest-django/
2 lines
py-sanetime/
2 lines
py-semantic_version/
4 lines
finance/
bitcoinnodestats/
10 lines
graphics/
py-django-easy-thumbnails/
12 lines
py-sorl-thumbnail/
8 lines
net-mgmt/
netbox/
30 lines
security/
py-first-server/
7 lines
textproc/
py-pyjade/
7 lines
www/
25 lines
py-dj22-django-auth-ldap/
py-dj22-django-cacheops/
py-dj22-django-cors-headers/
py-dj22-django-crispy-forms/
py-dj22-django-debug-toolbar/
py-dj22-django-extensions/
py-dj22-django-filter/
py-dj22-django-formset-js-improved/
py-dj22-django-jquery-js/
py-dj22-django-js-asset/
py-dj22-django-jsonfield2/
py-dj22-django-model-utils/
py-dj22-django-mptt/
py-dj22-django-prometheus/
py-dj22-django-redis/
py-dj22-django-tables2/
py-dj22-django-taggit-serializer/
py-dj22-django-taggit/
py-dj22-django-timezone-field/
py-dj22-django-treebeard/
py-dj22-django_csp/
py-dj22-djangoql/
py-dj22-djangorestframework/
py-dj22-drf-yasg/
py-dj22-swapper/
py-django-allauth/
9 lines
py-django-annotations/
14 lines
py-django-annoying/
8 lines
py-django-appconf/
7 lines
py-django-appmedia/
8 lines
py-django-assets/
9 lines
py-django-auth-ldap/
13 lines
py-django-babel/
9 lines
py-django-bitfield/
9 lines
py-django-bootstrap-form/
8 lines
py-django-bootstrap3/
2 lines
py-django-braces/
9 lines
py-django-bulk-update/
8 lines
py-django-cacheops/
11 lines
py-django-caching-app-plugins/
7 lines
py-django-classy-tags/
8 lines
py-django-cms/
17 lines
py-django-common-helpers/
8 lines
py-django-configurations/
9 lines
py-django-constance/
7 lines
py-django-contact-form/
7 lines
py-django-contrib-comments/
8 lines
py-django-cors-headers/
12 lines
py-django-countries/
6 lines
py-django-crispy-forms/
11 lines
py-django-cron/
9 lines
py-django-datetime-widget/
9 lines
py-django-debug-toolbar/
14 lines
py-django-dpaste/
11 lines
py-django-evolution/
8 lines
py-django-extensions/
12 lines
py-django-filer/
13 lines
py-django-filter/
10 lines
py-django-formtools/
7 lines
py-django-guardian/
7 lines
py-django-happenings/
11 lines
py-django-haystack/
7 lines
py-django-hijack/
8 lines
py-django-htmlmin/
7 lines
py-django-js-asset/
12 lines
py-django-json-rpc/
8 lines
py-django-jsonfield/
7 lines
py-django-jsonfield2/
8 lines
py-django-ldapdb/
9 lines
py-django-markdownx/
9 lines
py-django-markwhat/
6 lines
py-django-mezzanine-filebrowser/
6 lines
py-django-mezzanine-grappelli/
6 lines
py-django-mezzanine/
15 lines
py-django-model-utils/
8 lines
py-django-modelcluster/
6 lines
py-django-modern-rpc/
8 lines
py-django-mptt/
13 lines
py-django-netfields/
9 lines
py-django-openid-auth/
9 lines
py-django-otp-yubikey/
8 lines
py-django-otp/
6 lines
py-django-overextends/
9 lines
py-django-photologue/
9 lines
py-django-picklefield/
7 lines
py-django-pipeline/
6 lines
py-django-post_office/
9 lines
py-django-profiles/
10 lines
py-django-pyscss/
9 lines
py-django-ranged-response/
8 lines
py-django-recaptcha/
8 lines
py-django-redis/
9 lines
py-django-registration-defaults/
10 lines
py-django-registration-redux/
11 lines
py-django-registration/
11 lines
py-django-rest-swagger/
8 lines
py-django-reversion/
8 lines
py-django-sekizai/
9 lines
py-django-simple-captcha/
9 lines
py-django-simple-history/
8 lines
py-django-solo/
8 lines
py-django-sortedm2m/
8 lines
py-django-star-ratings/
13 lines
py-django-statici18n/
9 lines
py-django-storages/
6 lines
py-django-subdomains/
8 lines
py-django-sudo/
6 lines
py-django-tables2/
9 lines
py-django-tagging/
8 lines
py-django-taggit-serializer/
10 lines
py-django-taggit/
11 lines
py-django-tastypie/
7 lines
py-django-templatetag-sugar/
8 lines
py-django-timezone-field/
13 lines
py-django-tinymce/
8 lines
py-django-treebeard/
12 lines
py-django-voting/
7 lines
py-django-webpack-loader/
8 lines
py-django-widget-tweaks/
8 lines
py-django/
2 lines
py-django111/
5 lines
4 lines
py-django22/
4 lines
4 lines
py-django30/
5 lines
4 lines
py-django_compressor/
8 lines
py-django_openstack_auth/
5 lines
py-django_polymorphic/
8 lines
py-djangocms-admin-style/
8 lines
py-djangoql/
py-dj22-djangoql/
9 lines
3 lines
5 lines
py-djangorestframework-csv/
7 lines
py-djangorestframework-filters/
7 lines
py-djangorestframework-xml/
7 lines
py-djangorestframework/
10 lines
py-djangorestframework36/
7 lines
py-djangotoolbox/
8 lines
py-dojango/
8 lines
py-drf-yasg/
11 lines
py-google-api-python-client/
3 lines
py-graphite-web/
9 lines
py-horizon/
15 lines
py-jsonfield/
8 lines
py-swapper/
7 lines
py-wagtail/
15 lines
seahub/
22 lines
Diff 67553
View Options
Mk/Uses/python.mk
Loading...
View Options
Mk/bsd.default-versions.mk
View Options
Mk/bsd.port.mk
View Options
databases/py-django-transaction-hooks/Makefile
View Options
databases/py-south/Makefile
View Options
deskutils/py-paperless/Makefile
View Options
devel/py-django-rq-dashboard/Makefile
View Options
devel/py-django-rq/Makefile
View Options
devel/py-django-tastypie-mongoengine/Makefile
View Options
devel/py-eventlib/Makefile
View Options
devel/py-logan/Makefile
View Options
devel/py-pypugjs/Makefile
View Options
devel/py-pytest-django/Makefile
View Options
devel/py-sanetime/Makefile
View Options
devel/py-semantic_version/Makefile
View Options
finance/bitcoinnodestats/Makefile
View Options
graphics/py-django-easy-thumbnails/Makefile
View Options
graphics/py-sorl-thumbnail/Makefile
View Options
net-mgmt/netbox/Makefile
View Options
security/py-first-server/Makefile
View Options
textproc/py-pyjade/Makefile
View Options
www/py-dj22-django-auth-ldap/Makefile
View Options
www/py-dj22-django-auth-ldap/distinfo
View Options
www/py-dj22-django-auth-ldap/pkg-descr
View Options
www/py-dj22-django-cacheops/Makefile
View Options
www/py-dj22-django-cacheops/distinfo
View Options
www/py-dj22-django-cacheops/pkg-descr
View Options
www/py-dj22-django-cors-headers/Makefile
View Options
www/py-dj22-django-cors-headers/distinfo
View Options
www/py-dj22-django-cors-headers/pkg-descr
View Options
www/py-dj22-django-crispy-forms/Makefile
View Options
www/py-dj22-django-crispy-forms/distinfo
View Options
www/py-dj22-django-crispy-forms/pkg-descr
View Options
www/py-dj22-django-debug-toolbar/Makefile
View Options
www/py-dj22-django-debug-toolbar/distinfo
View Options
www/py-dj22-django-debug-toolbar/pkg-descr
View Options
www/py-dj22-django-extensions/Makefile
View Options
www/py-dj22-django-extensions/distinfo
View Options
www/py-dj22-django-extensions/pkg-descr
View Options
www/py-dj22-django-filter/Makefile
View Options
www/py-dj22-django-filter/distinfo
View Options
www/py-dj22-django-filter/pkg-descr
View Options
www/py-dj22-django-formset-js-improved/Makefile
View Options
www/py-dj22-django-formset-js-improved/distinfo
View Options
www/py-dj22-django-formset-js-improved/pkg-descr
View Options
www/py-dj22-django-jquery-js/Makefile
View Options
www/py-dj22-django-jquery-js/distinfo
View Options
www/py-dj22-django-jquery-js/pkg-descr
View Options
www/py-dj22-django-js-asset/Makefile
View Options
www/py-dj22-django-js-asset/distinfo
View Options
www/py-dj22-django-js-asset/pkg-descr
View Options
www/py-dj22-django-jsonfield2/Makefile
View Options
www/py-dj22-django-jsonfield2/distinfo
View Options
www/py-dj22-django-jsonfield2/pkg-descr
View Options
www/py-dj22-django-model-utils/Makefile
View Options
www/py-dj22-django-model-utils/distinfo
View Options
www/py-dj22-django-model-utils/pkg-descr
View Options
www/py-dj22-django-mptt/Makefile
View Options
www/py-dj22-django-mptt/distinfo
View Options
www/py-dj22-django-mptt/pkg-descr
View Options
www/py-dj22-django-prometheus/Makefile
View Options
www/py-dj22-django-prometheus/distinfo
View Options
www/py-dj22-django-prometheus/pkg-descr
View Options
www/py-dj22-django-redis/Makefile
View Options
www/py-dj22-django-redis/distinfo
View Options
www/py-dj22-django-redis/pkg-descr
View Options
www/py-dj22-django-tables2/Makefile
View Options
www/py-dj22-django-tables2/distinfo
View Options
www/py-dj22-django-tables2/pkg-descr
View Options
www/py-dj22-django-taggit-serializer/Makefile
View Options
www/py-dj22-django-taggit-serializer/distinfo
View Options
www/py-dj22-django-taggit-serializer/pkg-descr
View Options
www/py-dj22-django-taggit/Makefile
View Options
www/py-dj22-django-taggit/distinfo
View Options
www/py-dj22-django-taggit/pkg-descr
View Options
www/py-dj22-django-timezone-field/Makefile
View Options
www/py-dj22-django-timezone-field/distinfo
View Options
www/py-dj22-django-timezone-field/pkg-descr
View Options
www/py-dj22-django-treebeard/Makefile
View Options
www/py-dj22-django-treebeard/distinfo
View Options
www/py-dj22-django-treebeard/pkg-descr
View Options
www/py-dj22-django_csp/Makefile
View Options
www/py-dj22-django_csp/distinfo
View Options
www/py-dj22-django_csp/pkg-descr
View Options
www/py-dj22-djangoql/Makefile
View Options
www/py-dj22-djangoql/distinfo
View Options
www/py-dj22-djangoql/pkg-descr
View Options
www/py-dj22-djangorestframework/Makefile
View Options
www/py-dj22-djangorestframework/distinfo
View Options
www/py-dj22-djangorestframework/pkg-descr
View Options
www/py-dj22-drf-yasg/Makefile
View Options
www/py-dj22-drf-yasg/distinfo
View Options
www/py-dj22-drf-yasg/pkg-descr
View Options
www/py-dj22-swapper/Makefile
View Options
www/py-dj22-swapper/distinfo
View Options
www/py-dj22-swapper/pkg-descr
View Options
www/py-django-allauth/Makefile
View Options
www/py-django-annotations/Makefile
View Options
www/py-django-annoying/Makefile
View Options
www/py-django-appconf/Makefile
View Options
www/py-django-appmedia/Makefile
View Options
www/py-django-assets/Makefile
View Options
www/py-django-auth-ldap/Makefile
View Options
www/py-django-babel/Makefile
View Options
www/py-django-bitfield/Makefile
View Options
www/py-django-bootstrap-form/Makefile
View Options
www/py-django-bootstrap3/Makefile
View Options
www/py-django-braces/Makefile
View Options
www/py-django-bulk-update/Makefile
View Options
www/py-django-cacheops/Makefile
View Options
www/py-django-caching-app-plugins/Makefile
View Options
www/py-django-classy-tags/Makefile
View Options
www/py-django-cms/Makefile
View Options
www/py-django-common-helpers/Makefile
View Options
www/py-django-configurations/Makefile
View Options
www/py-django-constance/Makefile
View Options
www/py-django-contact-form/Makefile
View Options
www/py-django-contrib-comments/Makefile
View Options
www/py-django-cors-headers/Makefile
View Options
www/py-django-countries/Makefile
View Options
www/py-django-crispy-forms/Makefile
View Options
www/py-django-cron/Makefile
View Options
www/py-django-datetime-widget/Makefile
View Options
www/py-django-debug-toolbar/Makefile
View Options
www/py-django-dpaste/Makefile
View Options
www/py-django-evolution/Makefile
View Options
www/py-django-extensions/Makefile
View Options
www/py-django-filer/Makefile
View Options
www/py-django-filter/Makefile
View Options
www/py-django-formtools/Makefile
View Options
www/py-django-guardian/Makefile
View Options
www/py-django-happenings/Makefile
View Options
www/py-django-haystack/Makefile
View Options
www/py-django-hijack/Makefile
View Options
www/py-django-htmlmin/Makefile
View Options
www/py-django-js-asset/Makefile
View Options
www/py-django-json-rpc/Makefile
View Options
www/py-django-jsonfield/Makefile
View Options
www/py-django-jsonfield2/Makefile
View Options
www/py-django-ldapdb/Makefile
View Options
www/py-django-markdownx/Makefile
View Options
www/py-django-markwhat/Makefile
View Options
www/py-django-mezzanine-filebrowser/Makefile
View Options
www/py-django-mezzanine-grappelli/Makefile
View Options
www/py-django-mezzanine/Makefile
View Options
www/py-django-model-utils/Makefile
View Options
www/py-django-modelcluster/Makefile
View Options
www/py-django-modern-rpc/Makefile
View Options
www/py-django-mptt/Makefile
View Options
www/py-django-netfields/Makefile
View Options
www/py-django-openid-auth/Makefile
View Options
www/py-django-otp-yubikey/Makefile
View Options
www/py-django-otp/Makefile
View Options
www/py-django-overextends/Makefile
View Options
www/py-django-photologue/Makefile
View Options
www/py-django-picklefield/Makefile
View Options
www/py-django-pipeline/Makefile
View Options
www/py-django-post_office/Makefile
View Options
www/py-django-profiles/Makefile
View Options
www/py-django-pyscss/Makefile
View Options
www/py-django-ranged-response/Makefile
View Options
www/py-django-recaptcha/Makefile
View Options
www/py-django-redis/Makefile
View Options
www/py-django-registration-defaults/Makefile
View Options
www/py-django-registration-redux/Makefile
View Options
www/py-django-registration/Makefile
View Options
www/py-django-rest-swagger/Makefile
View Options
www/py-django-reversion/Makefile
View Options
www/py-django-sekizai/Makefile
View Options
www/py-django-simple-captcha/Makefile
View Options
www/py-django-simple-history/Makefile
View Options
www/py-django-solo/Makefile
View Options
www/py-django-sortedm2m/Makefile
View Options
www/py-django-star-ratings/Makefile
View Options
www/py-django-statici18n/Makefile
View Options
www/py-django-storages/Makefile
View Options
www/py-django-subdomains/Makefile
View Options
www/py-django-sudo/Makefile
View Options
www/py-django-tables2/Makefile
View Options
www/py-django-tagging/Makefile
View Options
www/py-django-taggit-serializer/Makefile
View Options
www/py-django-taggit/Makefile
View Options
www/py-django-tastypie/Makefile
View Options
www/py-django-templatetag-sugar/Makefile
View Options
www/py-django-timezone-field/Makefile
View Options
www/py-django-tinymce/Makefile
View Options
www/py-django-treebeard/Makefile
View Options
www/py-django-voting/Makefile
View Options
www/py-django-webpack-loader/Makefile
View Options
www/py-django-widget-tweaks/Makefile
View Options
www/py-django/Makefile
View Options
www/py-django111/Makefile
View Options
www/py-django111/Makefile.version
View Options
www/py-django22/Makefile
View Options
www/py-django22/Makefile.version
View Options
www/py-django30/Makefile
View Options
www/py-django30/Makefile.version
View Options
www/py-django_compressor/Makefile
View Options
www/py-django_openstack_auth/Makefile
View Options
www/py-django_polymorphic/Makefile
View Options
www/py-djangocms-admin-style/Makefile
View Options
www/py-djangoql/Makefile
View Options
www/py-djangoql/distinfo
View Options
www/py-djangoql/pkg-descr
View Options
www/py-djangorestframework-csv/Makefile
View Options
www/py-djangorestframework-filters/Makefile
View Options
www/py-djangorestframework-xml/Makefile
View Options
www/py-djangorestframework/Makefile
View Options
www/py-djangorestframework36/Makefile
View Options
www/py-djangotoolbox/Makefile
View Options
www/py-dojango/Makefile
View Options
www/py-drf-yasg/Makefile
View Options
www/py-google-api-python-client/Makefile
View Options
www/py-graphite-web/Makefile
View Options
www/py-horizon/Makefile
View Options
www/py-jsonfield/Makefile
View Options
www/py-swapper/Makefile
View Options
www/py-wagtail/Makefile
View Options
www/seahub/Makefile
Loading...