Bug 245568

Summary: math/py-matplotlib-scalebar: Update to 0.6.2
Product: Ports & Packages Reporter: Rainer Hurling <rhurlin>
Component: Individual Port(s)Assignee: Dmitry Marakasov <amdmi3>
Status: Closed Overcome By Events    
Severity: Affects Only Me CC: koobs, lbartoletti, rhurlin, tcberner
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch to fix math/py-matplotlib-scalebar reciprocal unit prefix
rhurlin: maintainer-approval+
patch to fix math/py-matplotlib-scalebar reciprocal unit prefix, python 3.7 only rhurlin: maintainer-approval+

Description Rainer Hurling freebsd_committer freebsd_triage 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 freebsd_triage 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 freebsd_triage 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 freebsd_triage 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 freebsd_triage 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 freebsd_triage 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 freebsd_triage 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?
Comment 8 Dmitry Marakasov freebsd_committer freebsd_triage 2021-03-17 19:44:06 UTC 
The port is now at 0.7.2