Path: csiph.com!news.mixmin.net!news.unit0.net!newsfeed.CARNet.hr!news.spin.it!bofh.it!news.nic.it!robomod From: Neil Williams Newsgroups: linux.debian.maint.python Subject: Re: packaging django-rest-framework-filters Date: Wed, 28 Nov 2018 18:00:01 +0100 Message-ID: References: X-Original-To: Wookey X-Mailbox-Line: From debian-python-request@lists.debian.org Wed Nov 28 16:51:40 2018 Old-Return-Path: X-Amavis-Spam-Status: No, score=-14.2 tagged_above=-10000 required=5.3 tests=[BAYES_00=-2, FOURLA=0.1, LDO_WHITELIST=-5, PGPSIGNATURE=-5, RCVD_IN_DNSWL_MED=-2.3] autolearn=ham autolearn_force=no X-Policyd-Weight: using cached result; rate: -6.1 X-Mailer: Claws Mail 3.17.1 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; boundary="Sig_/Bb17zSsPYv.nhBxdWMaOVlv"; protocol="application/pgp-signature" X-Mailing-List: archive/latest/16583 List-ID: List-URL: List-Archive: https://lists.debian.org/msgid-search/20181128165123.0aefe801@calvin.codehelp Approved: robomod@news.nic.it Lines: 143 Organization: linux.* mail to news gateway Sender: robomod@news.nic.it X-Original-Cc: debian-python@lists.debian.org X-Original-Date: Wed, 28 Nov 2018 16:51:23 +0000 X-Original-Message-ID: <20181128165123.0aefe801@calvin.codehelp> X-Original-References: <20181122050200.tcg7urpyqjd5rx7n@mail.wookware.org> <20181126040505.hzyhkmatjcmrglg4@mail.wookware.org> <20181128085853.65613f39@calvin.codehelp> <20181128143820.xkdawzmo3nvy4wg2@mail.wookware.org> Xref: csiph.com linux.debian.maint.python:11139 --Sig_/Bb17zSsPYv.nhBxdWMaOVlv Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Wed, 28 Nov 2018 14:38:20 +0000 Wookey wrote: > On 2018-11-28 08:58 +0000, Neil Williams wrote: > > On Mon, 26 Nov 2018 04:05:05 +0000 > > Wookey wrote: > > =20 > > > On 2018-11-22 08:59 +0100, Thomas Goirand wrote: =20 > > > > On 11/22/18 6:02 AM, Wookey wrote: =20 > > >=20 > > > I guess djangorestframework-filters is > > > clearer, and closer to upstream so people can find it. I'll go > > > with that unless someone says the drf-* names are a better plan. > > >=20 > > > OK. I've made a new package (djangorestframework-filters), which > > > seems OK. (actually I've made 2 (see below)) > > >=20 > > > I'm not a member of the modules team, so can't follow the > > > instructions to make a salsa project under the python-team banner > > > (https://wiki.debian.org/Python/GitPackaging) > > >=20 > > > I'll put it under wookey for now. > > >=20 > > > Also, the latest release is 0.10.0.post0, which says it's > > > compatible with: > > > * **Python**: 2.7 or 3.3+ > > > * **Django**: 1.8, 1.9, 1.10, 1.11 > > > * **DRF**: 3.5, 3.6 =20 > >=20 > > djangorestframework lists "Breaking changes" in the release notes > > for 3.8, so this looks like an incompatibility with what's already > > in Debian. > >=20 > > https://github.com/encode/django-rest-framework/releases/tag/3.8.0 > >=20 > > It might be worth testing whether 0.10.0.post0 or another release of > > django-rest-framework-filters between that and 1.0.0dev0 is actually > > fine with djangorestframework 3.8 - at least at a unit test level. =20 >=20 > Milosz and I are doing just that. >=20 > It would be great if you could check whether these packages work OK > with lava or not as that's at least one datapoint: > http://wookware.org/software/repo/pool/main/d/drf-extensions/python3-djan= gorestframework-extensions_0.4.0-1_all.deb A bit surprised to see that this binary only depends on: Depends: python3-djangorestframework, python3:any I'd expect at least python3-django to be in that list. Could be an upstream bug in their setup.py - relying only on the indirect dependency via python3-djangorestframework is a bit odd/lazy to me. The package does directly import modules from django, it doesn't exclusively import from django-restframework, not least in it's own settings.py. > http://wookware.org/software/repo/pool/main/d/djangorestframework-filters= /python3-djangorestframework-filters_0.10.2.post0-1_all.deb For comparison: Depends: python3-django, python3-django-filters, python3-djangorestframework, python3:any So I can install both of those packages ok on buster, I can run the existing LAVA unit tests without failures and I can browse the localhost LAVA UI (including REST API) with those installed. However, that only a minimal datapoint - the REST API doesn't yet have code to use the support available from these packages. It's basically only an install test, not execution. I've successfully imported the modules into the django developer shell using lava-server manage.py shell, so that bit works too. If there are particular tests that would be useful, let me know. > > > However the version of DRF in testing is now 3.8, and python 3.6 > > > so perhaps it's better to upload the upcoming v1.0.0.dev0: > > > * **Python**: 3.4, 3.5, 3.6 > > > * **Django**: 1.11, 2.0, 2.1b1 > > > * **DRF**: 3.8 > > > * **django-filter**: 2.0 > > >=20 > > > But the version of django-filter in debian is 1.1.0, so at first > > > glance neither of these versions will work with the components > > > available. > > >=20 > > > I'd normally upload the last released version, i.e. 0.10.2.post0, > > > but I'm not sure how these interactions with versions of > > > djangorestframework and django-filter work. Any advice or shall I > > > work this out with upstreams? =20 > >=20 > > django-filters has a new upstream 2.0 but a migration guide has been > > published for that: > >=20 > > https://django-filter.readthedocs.io/en/master/guide/migration.html#mig= rating-to-2-0 =20 >=20 > Cheers for that link >=20 > > So it's likely that at least some reverse dependencies would be > > broken by django-filters version 2.0. =20 >=20 > As you say - if we have to go to django-filters 2.0, then things > probably get complicated (too complicated for buster). >=20 > Wookey > --=20 > Principal hats: Linaro, Debian, Wookware, ARM > http://wookware.org/ --=20 Neil Williams home@codehelp.co.uk --Sig_/Bb17zSsPYv.nhBxdWMaOVlv Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEf3HB6ceOc10DYMbM8WfkPIFDtoIFAlv+x4sACgkQ8WfkPIFD toJxDBAAkuMuugTYNefcFWaPF7M1D9r0FaVl6EJHR0e5inZGtiRiM+E7LuldeCGN KrRaJzqKRah/hFp9zKOrQU/pZP4E2rzxFoYMbc84YIxynRpaTLV3oBG2sZyfwjr0 +ogKdMvw+otIJzdV2aYd47r8Vx6QGd1NP1BkySmQ++Yhv4jujOo+WnsjQwJsAZ0E UuX1jNCIz7lIba7cbSfa7ojtA/KOwMHZWfj4PtdYGWYhkh4Uv2MPN1p2XpXW+qFa msw2h0wnpleU1vRE7x/hKAnOaq4gtlgwK8kSrO/yj35KMdV/WS+H60QC8Ut6WVXp EeklnYIPGgarNhSOpiBPb0s1UCi8F3xQ2uZMvJ3qcCh9gKjravFS8aTWUSGqtazl NIFiN5SWMBdcjNSrsIi5ynk8AR4Jf97UXxQ3aJjJ8t9zjfPXZJMiC6fOP/XzGVop QLecdnnvLdBXssJvlv+ma78exphQb+veT4Ce+7jVkl1eCbaGBnEll8ZLlcm5jQbr LNJ9mQxiGVmvzFbb/EEVU0nMOca3qNHfW3XCB4T58V0BxzBC44xzfUQ4lMy/UYG9 0sOtbIOgRPJ48mD/531GDhzJyB1NGjak/7MvrQMw1iQ22io0mw17B+olEYLDc7Nn lh0KlKcihZ0kWvLsUWnu2IzNu5G2UJ1GPohnypjMOLphIvXLe30= =EZSj -----END PGP SIGNATURE----- --Sig_/Bb17zSsPYv.nhBxdWMaOVlv--