New versions of IBus require this.
Details
Diff Detail
- Repository
- rP FreeBSD ports repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
misc/unicode-emoji/Makefile | ||
---|---|---|
7 ↗ | (On Diff #34965) | I would add a DIST_SUBDIR=${PORTNAME}/${PORTVERSION} as the files themselves do not carry a version. https://www.freebsd.org/doc/en/books/porters-handbook/makefile-distfiles.html#makefile-dist_subdir |
The two for-loops could probably be done more concise using variable modifiers ala
PLIST_FILES=${DISTFILES:S/^/share\/unicode\/emoji\/g}
but I think that would loose some readability :D
misc/unicode-emoji/Makefile | ||
---|---|---|
18 ↗ | (On Diff #34974) | maybe EXTRACT_ONLY= # nothing todo ? |
It says If only some of the DISTFILES must be extracted ..., and here we have no DISTFILES must be extracted, so it technically doesn't fall under this section. :-)
I am ok either way, but I feel do-extract version is marginally more readable. It explicitly says do nothing under the action.
When none of the DISTFILES need to be uncompressed, set EXTRACT_ONLY to the empty string. EXTRACT_ONLY=
misc/unicode-emoji/Makefile | ||
---|---|---|
22 ↗ | (On Diff #34975) | ^-${DISTDIR}/${DISTSUBDIR} |
head/misc/unicode-emoji/Makefile | ||
---|---|---|
14 | Didn't this line generate a warning ? You must use BSD[234]CLAUSE and not BSD. |
head/misc/unicode-emoji/Makefile | ||
---|---|---|
14 | I will remove the license since nowhere on there site it says what the license is. |