Page MenuHomeFreeBSD

New port: cad/py-pyfda: GUI tool for designing and analysing discrete time filters
ClosedPublic

Authored by yuri on Feb 2 2018, 8:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 9 2024, 3:02 PM
Unknown Object (File)
Mar 9 2024, 3:02 PM
Unknown Object (File)
Mar 9 2024, 3:02 PM
Unknown Object (File)
Mar 9 2024, 3:02 PM
Unknown Object (File)
Mar 9 2024, 2:35 PM
Unknown Object (File)
Feb 22 2024, 2:16 PM
Unknown Object (File)
Jan 7 2024, 12:46 PM
Unknown Object (File)
Dec 21 2023, 10:45 PM
Subscribers

Details

Summary

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=222442

See the screenshot: https://raw.githubusercontent.com/chipmuenk/pyFDA/master/images/pyFDA_screenshot_3.PNG
The patches are mostly due to the fact that pyFDA doesn't have an installation procedure, and expects users to just run locally.

I kept RUN_DEPENDS in the same order as in setup.py. They also seem to be in the order they are used.
Please let me know if alphanumeric sort is preferable here.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 14748
Build 14869: arc lint + arc unit

Event Timeline

cad/py-pyfda/Makefile
16

The RUN_DEPENDS should be sorted. If there were a ton of them and it was going to be an obstacle for maintaining them against the upstream list, then unsorted would be fine. But for 4 things, it's worth sorting.

22

Can the USE_PYTHON be sorted?

45

Why are all of these REINPLACE_CMDs split after the opening quote? It looks really weird, given that the next line is all one line and contains the closing quote.

Am I missing something? I haven't finished my coffee, so it's entirely possible.

yuri marked 3 inline comments as done.

Sorted depends.

cad/py-pyfda/Makefile
22

mat@ said that these should either be sorted or have the order how they are used.
In this case, they are in their use order.

45

This allows to have many regexp statements in their own lines without the first one having the quote in front of it.
I actually did this a lot before in other ports.

Fair enough. Makes sense to me!

This revision is now accepted and ready to land.Feb 3 2018, 8:35 PM
This revision was automatically updated to reflect the committed changes.
head/cad/py-pyfda/Makefile
22 ↗(On Diff #38831)

Also, setting noflavors (which is weird, no reason to do it) means that the FLAVOR variable is empty, thus, it makes this port broken because the dependencies line end with @ with no flavor.

head/cad/py-pyfda/Makefile
22 ↗(On Diff #38831)

But this is an app, not a library. Shouldn't python apps all be noflavors?

head/cad/py-pyfda/Makefile
22 ↗(On Diff #38831)

this has a PKGNAMEPREFIX, nothing against it having noflavors.