diff mbox

[v2,01/12] libass: new package

Message ID 1389831355-19983-2-git-send-email-maxime.hadjinlian@gmail.com
State Superseded
Headers show

Commit Message

Maxime Hadjinlian Jan. 16, 2014, 12:15 a.m. UTC 
libass is a portable subtitle renderer for the
ASS/SSA (Advanced Substation Alpha/Substation Alpha) subtitle format.
This package was originally found at : https://github.com/huceke/buildroot-rbp
By gimli <ebsi4711@gmail.com>

Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
Cc: gimli <ebsi4711@gmail.com>
Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
Changes v2 -> v3
  - Fix header
Changes v1 -> v2
  - Bump version to 0.10.2 (Bernd Kuhls)
---
 package/Config.in        |  1 +
 package/libass/Config.in | 10 ++++++++++
 package/libass/libass.mk | 14 ++++++++++++++
 3 files changed, 25 insertions(+)
 create mode 100644 package/libass/Config.in
 create mode 100644 package/libass/libass.mk

Comments

Peter Korsgaard Jan. 17, 2014, 8:08 a.m. UTC | #1 
>>>>> "Maxime" == Maxime Hadjinlian <maxime.hadjinlian@gmail.com> writes:

 > libass is a portable subtitle renderer for the
 > ASS/SSA (Advanced Substation Alpha/Substation Alpha) subtitle format.
 > This package was originally found at : https://github.com/huceke/buildroot-rbp
 > By gimli <ebsi4711@gmail.com>

 > Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
 > Cc: gimli <ebsi4711@gmail.com>
 > Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
 > ---
 > Changes v2 -> v3
 >   - Fix header
 > Changes v1 -> v2
 >   - Bump version to 0.10.2 (Bernd Kuhls)

 > +config BR2_PACKAGE_LIBASS
 > +	bool "libass"
 > +	select BR2_PACKAGE_FREETYPE
 > +	select BR2_PACKAGE_FONTCONFIG
 > +	select BR2_PACKAGE_LIBENCA

The patch ordering is wrong. You only introduce libenca in patch
11. Otherwise it looks OK.
Bernd Kuhls Jan. 18, 2014, 11:41 a.m. UTC | #2 
Maxime Hadjinlian
<maxime.hadjinlian@gmail.com> wrote in
news:1389831355-19983-2-git-send-email-maxime.hadjinlian@gmail.com: 

> +LIBASS_DEPENDENCIES = freetype fontconfig libenca

Hi,

please add libfribidi as dependency as its needed for xbmc gotham.

Regards, Bernd
diff mbox

Patch

diff --git a/package/Config.in b/package/Config.in
index 2f62d6c..1d6c9c6 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -627,6 +627,7 @@  source "package/yajl/Config.in"
 endmenu
 
 menu "Multimedia"
+source "package/libass/Config.in"
 source "package/libdvbsi/Config.in"
 source "package/libdvdnav/Config.in"
 source "package/libdvdread/Config.in"
diff --git a/package/libass/Config.in b/package/libass/Config.in
new file mode 100644
index 0000000..ff34b64
--- /dev/null
+++ b/package/libass/Config.in
@@ -0,0 +1,10 @@ 
+config BR2_PACKAGE_LIBASS
+	bool "libass"
+	select BR2_PACKAGE_FREETYPE
+	select BR2_PACKAGE_FONTCONFIG
+	select BR2_PACKAGE_LIBENCA
+	help
+	  libass is a portable subtitle renderer for the ASS/SSA
+	  (Advanced Substation Alpha/Substation Alpha) subtitle format
+
+	  http://code.google.com/p/libass/
diff --git a/package/libass/libass.mk b/package/libass/libass.mk
new file mode 100644
index 0000000..854a7d5
--- /dev/null
+++ b/package/libass/libass.mk
@@ -0,0 +1,14 @@ 
+################################################################################
+#
+# libass
+#
+################################################################################
+
+LIBASS_VERSION = 0.10.2
+LIBASS_SITE = http://libass.googlecode.com/files
+LIBASS_INSTALL_STAGING = YES
+LIBASS_DEPENDENCIES = freetype fontconfig libenca
+LIBASS_LICENSE = BSD-3c
+LIBASS_LICENSE_FILES = COPYING
+
+$(eval $(autotools-package))