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
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

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?