On 12/7/22 20:09, Jeremie Courreges-Anglas wrote:
On Wed, Dec 07 2022, Stuart Henderson <s...@spacehopper.org> wrote:On 2022/12/07 00:35, Jeremie Courreges-Anglas wrote:You're setting CFLAGS on the make command line because you spotted that its value wasn't in control of the ports framework. But passing CFLAGS on the make command-line means that the CFLAGS assignement and subsequent appends (pkg-config --cflags ...) in upstream's Makefile are ignored, so the build fails. Your do-build target doesn't respect CFLAGS as set in the port Makefile, so the build succeeds.In this kind of situation I think it's fair to patch upstream's Makefile in order to satisfy the needs of both the ports framework and upstream's Makefile, introducing a new variable PORTS_CFLAGS.Your version is OK sthen@ with or without tweaks mentioned here, but it would be slightly neater to do this: -CFLAGS= -O3 -Os +CFLAGS?= -O3 -Os (that could go to the upstream tree easily enough if Thomas wants), and remove CFLAGS from MAKE_FLAGS; it's passed in via the environment anyway and this then does the right thing.Even better, thanks for this!IMHO we could do without the licence sentence in DESCR.agreed.Here's the updated tarball which I'm going to import later tonight.
May I humbly suggest this one instead?It patches the project's Makefile in a way which I would include in the next revision.
It also takes care of two typos in the manpage. Thomas
d11amp.v4.tgz
Description: application/compressed-tar