Page MenuHomeFreeBSD

devel/asfml: New Port, ASFML, Ada bindings to SFML
ClosedPublic

Authored by agh_riseup.net on Oct 11 2023, 2:50 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 2:01 AM
Unknown Object (File)
Jan 18 2024, 8:04 AM
Unknown Object (File)
Dec 30 2023, 6:28 AM
Unknown Object (File)
Dec 13 2023, 8:36 AM
Unknown Object (File)
Nov 28 2023, 1:53 PM
Unknown Object (File)
Nov 21 2023, 6:53 AM
Unknown Object (File)
Oct 25 2023, 6:22 AM
Unknown Object (File)
Oct 21 2023, 8:28 PM
Subscribers

Details

Summary

Ada's Simple and Fast Multimedia Library (ASFML)

ASFML is an Ada semi-thick binding to SFML. The library uses Ada types,
and portable defined types, which eliminates the inclusion of Ada
interface libraries, however most of the functions are directly
imported.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

agh_riseup.net created this revision.
diizzy added inline comments.
devel/asfml/Makefile
23

${PORTNAME:tu} ?

45

Use EXAMPLES_RUN_DEPENDS

50

You can use this trick instead and reduce I/O
https://cgit.freebsd.org/ports/tree/net-mgmt/netdata/Makefile#n29

56

Common practice is to use %% (double) --> %%DATA_AUDIO%% (see Porters Handbook)

Updated diff in response to dizzy's feedback:

  • Use of ${PORTNAME:tu} over hard-coding ASFML
  • Move to ${EXAMPLES_RUN_DEPENDS} and remove if if ${PORT_OPTIONS:MEXAMPLES}

conditional

  • Make use of ${EXTRACT_AFTER_ARGS} over ${RM} in post-extract target
  • Use %% over % for patch file substitution, respecting the practice set out in the Porter's Handbook

All suggestions should be addressed in the updated diff.
Thanks for the review.

devel/asfml/Makefile
23

Great idea, done, thanks.

45

So obvious. Fixed.

50

Oh nice pro tip; fixed.

56

Respected.

This revision is now accepted and ready to land.Oct 21 2023, 8:26 PM

LGTM, thanks for your contributions!