Bug 245568 - math/py-matplotlib-scalebar: Update to 0.6.2
Summary: math/py-matplotlib-scalebar: Update to 0.6.2
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-04-12 12:27 UTC by Rainer Hurling
Modified: 2020-05-02 09:25 UTC (History)
4 users (show)

See Also:


Attachments
patch to fix math/py-matplotlib-scalebar reciprocal unit prefix (1.08 KB, patch)
2020-04-12 12:27 UTC, Rainer Hurling
rhurlin: maintainer-approval+
Details | Diff
patch to fix math/py-matplotlib-scalebar reciprocal unit prefix, python 3.7 only (1.27 KB, patch)
2020-04-15 18:35 UTC, Rainer Hurling
rhurlin: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Rainer Hurling freebsd_committer 2020-04-12 12:27:42 UTC
Created attachment 213334 [details]
patch to fix math/py-matplotlib-scalebar reciprocal unit prefix

This minor update of math/py-matplotlib-scalebar from 0.6.1 to 0.6.2 fixes the reciprocal unit prefix[1].


The patch is tested on Poudriere (11.3a, 12.0a, and HEADa), i386 not possible ATM because of a build problem with math/suitesparse[2]. 'portlint -AC' seems happy.


[1] https://github.com/ppinard/matplotlib-scalebar/pull/29
[2] http://beefy5.nyi.freebsd.org/data/121i386-default/531468/logs/suitesparse-5.7.2.log
Comment 1 Rainer Hurling freebsd_committer 2020-04-12 17:48:02 UTC
Addendum:

Now, that r531531 for math/suitesparse (Fix on i386, by using GCC), allows to build math/py-matplotlib-scalebar for i386 again.

So now, the patch is tested on Poudriere (11.3i, 12.0i, and HEADi), also.
Comment 2 Tobias C. Berner freebsd_committer 2020-04-12 18:59:49 UTC
Moin moin 

This seems to fail for py27 and py35 -- should it possibly be marked 3.6+?


mfg Tobias
Comment 3 Rainer Hurling freebsd_committer 2020-04-12 20:32:53 UTC
Moin Tobias,

Thanks for bringing this up.

py27 can't build anymore, since py-matplotlib-scalebar uses pathlib[1], which was introduced in Python since 3.4 [2].

[1] https://github.com/ppinard/matplotlib-scalebar/commit/166015e9b4cad5b9d00df31e0d242c335c93ab79
[2] https://docs.python.org/3/library/pathlib.html


py35, py36 and py38 seem to be broken for the following ports, which are ignored with the following error: Unknown flavor 'py3[568]', possible flavors: py37 py27

- devel/py-six@py3[568]
- math/py-kiwisolver@py3[568]
- www/py-tornado@py3[568]
- devel/py-pyparsing@py3[568]
- devel/py-pytz@py3[568]
- textproc/py-toml@py3[568]


So I think, in principle you are right. But for now, we should restrict FLAVORS to only 3.7 for now.

HG, Rainer
Comment 4 Rainer Hurling freebsd_committer 2020-04-15 18:35:05 UTC
Created attachment 213423 [details]
patch to fix math/py-matplotlib-scalebar reciprocal unit prefix, python 3.7 only

This is the patch belonging to comment #3.
Comment 5 Loïc Bartoletti freebsd_committer 2020-05-02 07:19:22 UTC
(In reply to Rainer Hurling from comment #3)
Hi Rainer,

It's not documented (there is a documentation on python/flavors, but personally, I think it's misleading.)

Please, can you try with BUILD_ALL_PYTHON_FLAVORS=yes in /etc/make.conf?

ping koobs@
Comment 6 Kubilay Kocak freebsd_committer freebsd_triage 2020-05-02 09:06:14 UTC
(In reply to Loïc Bartoletti from comment #5)

Let's touch base on IRC to talk about improving the Mk/Uses/python.mk messaging for these things
Comment 7 Rainer Hurling freebsd_committer 2020-05-02 09:25:07 UTC
(In reply to Loïc Bartoletti from comment #5)

With USES=python (w/o :3.7) in the ports Makefile and with BUILD_ALL_PYTHON_FLAVORS=yes in /etc/make.conf I just build on Poudriere: It only builds the 3.7 flavor.

Is this the expected behaviour?