The Simple and Fast Multimedia Library (SFML) exists in devel/sfml. SFML is a C++ library, and as such, may introduce complexity for other languages to use the library. CSFML is an official SFML project to provide C bindings to SFML, with the aim of reducing language binding complexity, to enable other languages to use C over C++.
Details
Details
Diff Detail
Diff Detail
- Repository
- R11 FreeBSD ports repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| devel/csfml/Makefile | ||
|---|---|---|
| 3 | PORTVERSION --> DISTVERSION | |
| 21 | Please use upstream release archive if possible, see also https://docs.freebsd.org/en/books/porters-handbook/book/#makefile-master_sites-github | |
| 26 | Is this needed? | |
Comment Actions
devel/csfml: Address items in review D41519
Address dizzy's comments in review D41519:
- Use ${DISTVERSION} instead of ${PORTVERSION}
- Set ${WWW} to the CSFML subproject site in attempt to clarify that the project itself is separate from the main project SFML releases
- Remove option DOCS and related support, the Readme is very minimal, and provides little advice or help
Differential Revision: https://reviews.freebsd.org/D41519
Comment Actions
Thanks for the feedback.
| devel/csfml/Makefile | ||
|---|---|---|
| 3 | This has been corrected. | |
| 21 | I am not sure what I am to do here. CSFML does not produce releases from the main SFML project. CSFML could be considered as sub-project of SFML, where other bindings are not formally part of the project like this. To make things challenging, CSFML releases are not turly synchronised with SFML's releases. So I changed ${WWW} to point to the CSFML download page, in an attempt to make the relationship, and separation of the two projects clearer. | |
| 26 | You are correct, this Readme is not needed, and has been removed with the updated diff. | |
Comment Actions
Thanks, I got a bit confused about their "sub-projects". Patch looks good and queued for commit.