Page MenuHomeFreeBSD

Update to upstream version 0.10.5, turning FDK_AAC into an OPTION

Authored by riggs on Apr 23 2016, 4:04 PM.
Referenced Files
Unknown Object (File)
Thu, Sep 21, 10:31 PM
Unknown Object (File)
Mon, Sep 18, 10:51 AM
Unknown Object (File)
Sat, Sep 16, 6:29 PM
Unknown Object (File)
Jul 1 2023, 9:11 AM
Unknown Object (File)
Jun 20 2023, 7:18 AM
Unknown Object (File)
May 5 2023, 1:36 PM
Unknown Object (File)
May 5 2023, 1:32 PM
Unknown Object (File)
May 5 2023, 1:29 PM



Update to upstream version 0.10.5

Diff Detail

rP FreeBSD ports repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

riggs retitled this revision from to Update to upstream version 0.10.5, turning FDK_AAC into an OPTION.
riggs updated this object.
riggs edited the test plan for this revision. (Show Details)
riggs added a reviewer: jbeich.
jbeich requested changes to this revision.Apr 23 2016, 6:01 PM
jbeich edited edge metadata.

r- is mainly due to typo in configure option. If you have time try to convert X11 to option helpers as well.

FWIW, according to ffmpeg wiki its own AAC encoder may have better quality than the old version in libav-10.1 if not even fdk-aac.

29 ↗(On Diff #15535)

Defining LICENSE_FILE isn't required if there's a copy under /usr/ports/Templates/Licenses/

68 ↗(On Diff #15535)

Maybe define as USES=localbase

75 ↗(On Diff #15535)

non-free is a poor description, better copy LICENSE* section from audio/fdk-aac and convert to LICENSE_COMB=multi e.g.,

FDK_AAC_DESC=	Enable Fraunhofer FDK AAC Codec

LICENSE_NAME_FDK_AAC=	Software License for The Fraunhofer FDK AAC Codec Library for Android
LICENSE_PERMS_FDK_AAC=	dist-mirror pkg-mirror auto-accept
78 ↗(On Diff #15535)

According to make/ the option is named --enable-fdk-aac. You can also use _CONFIGURE_ENABLE helper here i.e.,

109 ↗(On Diff #15535)

Do you still need it after USES=localbase ?

114 ↗(On Diff #15535)

To move above .include <> this can be converted to

# Enforce linking to the system c++ standard lib
USES+=          compiler:features
LDFLAGS+=       -l${COMPILER_FEATURES:Mlib*c++:S/lib//}
153 ↗(On Diff #15535)

Maybe define as post-extract-FDK_AAC-on: if you're already using option helpers. See 20150701 entry in /usr/ports/CHANGES

197 ↗(On Diff #15535)

Ditto, maybe use post-patch-FDK_AAC-on:

This revision now requires changes to proceed.Apr 23 2016, 6:01 PM
29 ↗(On Diff #15535)

Yes it is.

Most points addressed with upcoming patch.

109 ↗(On Diff #15535)

Seems to work in initial tests. Good catch, thanks!

riggs edited edge metadata.
riggs marked an inline comment as done.

Addressed the review comments from jbeich and mat.
X11 OPTION is still handled via .if{} at some places.

jbeich edited edge metadata.
This revision is now accepted and ready to land.Apr 24 2016, 1:45 PM
This revision was automatically updated to reflect the committed changes.