[oe] [meta-multimedia][PATCH] faac: update 1.29 -> 1.29.9.2

[Αθανάσιος Οικονόμου]

Στις Παρ, 28 Δεκ 2018 στις 7:13 μ.μ., ο/η Khem Raj <raj.khem at gmail.com> έγραψε:
>
>
>
> On 12/28/18 5:24 AM, Αθανάσιος Οικονόμου wrote:
> > Στις Παρ, 28 Δεκ 2018 στις 8:51 π.μ., ο/η Khem Raj <raj.khem at gmail.com> έγραψε:
> >>
> >> On Thu, Dec 27, 2018 at 10:32 PM Athanasios Oikonomou <athoik at gmail.com> wrote:
> >>>
> >>> Remove not required patch and extra EXTRA_OECONF "--without-mp4v2".
> >>
> >> have you build tested this patch on a musl platform ?
> >> if not please try one and report back.
> >>
> >
> > bitbake faac was tested using TCLIBC = "musl"
> > (39e933936e11e186f21dacf4bccedddaa0a122df)
> > for machines qemuarm and qemuarm64. Build was successful.
> >
> > Please let me know if above test case is enough or further actions required.
> >
>
> Thanks for confirming.
>
> >>> Previously not required mp4v2 library was dropped on 1.29.9.2.
> >>>
> >>> While being there also fix issue with multilib build on aarch64
> >>> using FILES_lib${BPN} instead of FILES_lib${PN}.
> >>>
> >>> WARNING: lib32-faac-1.29.9.2-r0 do_package: QA Issue: lib32-faac: Files/directories were installed but not shipped in any package:
> >>>    /usr/lib/libfaac.so.0
> >>>    /usr/lib/libfaac.so.0.0.0
> >>> Please set FILES such that these items are packaged. Alternatively if they are unneeded, avoid installing them or delete them within do_install.
> >>> lib32-faac: 2 installed and not shipped files. [installed-vs-shipped]
> >>>
> >>> Signed-off-by: Athanasios Oikonomou <athoik at gmail.com>
> >>> ---
> >>>   .../faac/{faac_1.29.bb => faac_1.29.9.2.bb}        | 14 ++++-------
> >>>   ...Define-__STRING-if-cdefs.h-does-not-exist.patch | 28 ----------------------
> >>>   2 files changed, 4 insertions(+), 38 deletions(-)
> >>>   rename meta-multimedia/recipes-multimedia/faac/{faac_1.29.bb => faac_1.29.9.2.bb} (56%)
> >>>   delete mode 100644 meta-multimedia/recipes-multimedia/faac/files/0001-mp4v2-Define-__STRING-if-cdefs.h-does-not-exist.patch
> >>>
> >>> diff --git a/meta-multimedia/recipes-multimedia/faac/faac_1.29.bb b/meta-multimedia/recipes-multimedia/faac/faac_1.29.9.2.bb
> >>> similarity index 56%
> >>> rename from meta-multimedia/recipes-multimedia/faac/faac_1.29.bb
> >>> rename to meta-multimedia/recipes-multimedia/faac/faac_1.29.9.2.bb
> >>> index d36f2f4..4fc1121 100644
> >>> --- a/meta-multimedia/recipes-multimedia/faac/faac_1.29.bb
> >>> +++ b/meta-multimedia/recipes-multimedia/faac/faac_1.29.9.2.bb
> >>> @@ -1,29 +1,23 @@
> >>>   SUMMARY = "Freeware Advanced Audio Coder (MPEG2-AAC, MPEG4-AAC)"
> >>>   SECTION = "libs"
> >>>   LICENSE = "LGPLv2+"
> >>> -HOMEPAGE = "http://www.audiocoding.com/faac.html"
> >>>   LIC_FILES_CHKSUM = "file://COPYING;md5=3bf50002aefd002f49e7bb854063f7e7 \
> >>>                       file://libfaac/coder.h;beginline=5;endline=17;md5=fa1fd6a5fa8cdc877d63a12530d273e0"
> >>>
> >>> -LICENSE_FLAGS = "commercial"
> >>> -
>
> why is this dropped. May be document it in commit message
>

The faac author didn't provide further information, commit message
contains only "removed unused library"
(https://github.com/knik0/faac/commit/82bfa6eba494c0374aaef91b61a1876d695d2ead).
The bitbake recipe didn't use also that library, it was disabled with
EXTRA_OECONF "--without-mp4v2".

I can open an issue on github page and ask why it was dropped.