Page MenuHomeFreeBSD

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

Authored by yuri on Feb 2 2018, 8:04 PM.
Referenced Files
Thu, Feb 22, 2:16 PM
Unknown Object (File)
Jan 7 2024, 12:46 PM
Unknown Object (File)
Dec 21 2023, 10:45 PM
Unknown Object (File)
Dec 21 2023, 9:37 PM
Unknown Object (File)
Dec 20 2023, 3:34 AM
Unknown Object (File)
Nov 8 2023, 5:08 AM
Unknown Object (File)
Nov 6 2023, 1:43 PM
Unknown Object (File)
Nov 5 2023, 12:10 AM



See the screenshot:
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 They also seem to be in the order they are used.
Please let me know if alphanumeric sort is preferable here.

Diff Detail

rP FreeBSD ports repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

16 ↗(On Diff #38792)

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 ↗(On Diff #38792)

Can the USE_PYTHON be sorted?

45 ↗(On Diff #38792)

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.

22 ↗(On Diff #38792)

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 ↗(On Diff #38792)

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.

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.


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


this has a PKGNAMEPREFIX, nothing against it having noflavors.