[PATCH] gettext: Enable generation of libasprintf in gettext 0.19

[PATCH] gettext: Enable generation of libasprintf in gettext 0.19

[Lev Iserovich]

From: Lev Iserovich <iserovil@deshawresearch.com>

Libasprintf was included in gettext 0.16 recipe, but omitted for some reason
in 0.19. This turns it back on.

Signed-off-by: Lev Iserovich <lev.iserovich@deshawresearch.com>
---
 meta/recipes-core/gettext/gettext_0.19.8.1.bb | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/meta/recipes-core/gettext/gettext_0.19.8.1.bb b/meta/recipes-core/gettext/gettext_0.19.8.1.bb
index c2059e6..c7c81ee 100644
--- a/meta/recipes-core/gettext/gettext_0.19.8.1.bb
+++ b/meta/recipes-core/gettext/gettext_0.19.8.1.bb
@@ -28,7 +28,6 @@ inherit autotools texinfo
 
 EXTRA_OECONF += "--without-lispdir \
                  --disable-csharp \
-                 --disable-libasprintf \
                  --disable-java \
                  --disable-native-java \
                  --disable-openmp \
@@ -81,7 +80,7 @@ FILES_gettext-runtime = "${bindir}/gettext \
                          ${bindir}/ngettext \
                          ${bindir}/envsubst \
                          ${bindir}/gettext.sh \
-                         ${libdir}/libasprintf.so* \
+                         ${libdir}/libasprintf${SODEV} \
                          ${libdir}/GNU.Gettext.dll \
                         "
 FILES_gettext-runtime-dev += "${libdir}/libasprintf.a \
-- 
2.7.1

Re: [PATCH] gettext: Enable generation of libasprintf in gettext 0.19

[Leonardo Sandoval]

On Tue, 2017-07-11 at 11:57 -0400, Lev Iserovich wrote:
> From: Lev Iserovich <iserovil@deshawresearch.com>
> 

next type, in case you need to send a second (or third..) revision,
include the 'v2' label after the '[PATCH ' string the subject line.

BTW, this time patchtest did not complain :)

Leo


> Libasprintf was included in gettext 0.16 recipe, but omitted for some reason
> in 0.19. This turns it back on.
> 
> Signed-off-by: Lev Iserovich <lev.iserovich@deshawresearch.com>
> ---
>  meta/recipes-core/gettext/gettext_0.19.8.1.bb | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/meta/recipes-core/gettext/gettext_0.19.8.1.bb b/meta/recipes-core/gettext/gettext_0.19.8.1.bb
> index c2059e6..c7c81ee 100644
> --- a/meta/recipes-core/gettext/gettext_0.19.8.1.bb
> +++ b/meta/recipes-core/gettext/gettext_0.19.8.1.bb
> @@ -28,7 +28,6 @@ inherit autotools texinfo
>  
>  EXTRA_OECONF += "--without-lispdir \
>                   --disable-csharp \
> -                 --disable-libasprintf \
>                   --disable-java \
>                   --disable-native-java \
>                   --disable-openmp \
> @@ -81,7 +80,7 @@ FILES_gettext-runtime = "${bindir}/gettext \
>                           ${bindir}/ngettext \
>                           ${bindir}/envsubst \
>                           ${bindir}/gettext.sh \
> -                         ${libdir}/libasprintf.so* \
> +                         ${libdir}/libasprintf${SODEV} \
>                           ${libdir}/GNU.Gettext.dll \
>                          "
>  FILES_gettext-runtime-dev += "${libdir}/libasprintf.a \
> -- 
> 2.7.1
> 

Re: [PATCH] gettext: Enable generation of libasprintf in gettext 0.19

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 316 bytes --]

On 11 July 2017 at 16:57, Lev Iserovich <lev.iserovich@deshawresearch.com>
wrote:

> Libasprintf was included in gettext 0.16 recipe, but omitted for some
> reason
> in 0.19. This turns it back on.
>

As this pulls C++ libraries into the core package, can it be split out like
the other libraries?

Ross

[-- Attachment #2: Type: text/html, Size: 756 bytes --]

Re: [PATCH] gettext: Enable generation of libasprintf in gettext 0.19

[Lev Iserovich]

[-- Attachment #1: Type: text/html, Size: 1578 bytes --]

Re: [PATCH] gettext: Enable generation of libasprintf in gettext 0.19

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 437 bytes --]

On 12 July 2017 at 15:31, Lev Iserovich <Lev.Iserovich@deshawresearch.com>
wrote:

> As this pulls C++ libraries into the core package, can it be split out
> like the other libraries?
>
> Sure, I can do that - can you point to some prior art for how this is
> generally done?
>

Add the package name to PACKAGES and fiddle FILES to put the bits in the
right place. gettext already does this for some other libraries.

Ross

[-- Attachment #2: Type: text/html, Size: 919 bytes --]

[PATCH] gettext: Enable generation of libasprintf in gettext 0.19

[Lev Iserovich]

Libasprintf was included in gettext 0.16 recipe, but omitted for some reason
in 0.19. This turns it back on.

Signed-off-by: Lev Iserovich <lev.iserovich@deshawresearch.com>
---
  meta/recipes-core/gettext/gettext_0.19.8.1.bb | 3 +--
  1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/meta/recipes-core/gettext/gettext_0.19.8.1.bb 
b/meta/recipes-core/gettext/gettext_0.19.8.1.bb
index c2059e6..c7c81ee 100644
--- a/meta/recipes-core/gettext/gettext_0.19.8.1.bb
+++ b/meta/recipes-core/gettext/gettext_0.19.8.1.bb
@@ -28,7 +28,6 @@ inherit autotools texinfo

  EXTRA_OECONF += "--without-lispdir \
                   --disable-csharp \
-                 --disable-libasprintf \
                   --disable-java \
                   --disable-native-java \
                   --disable-openmp \
@@ -81,7 +80,7 @@ FILES_gettext-runtime = "${bindir}/gettext \
                           ${bindir}/ngettext \
                           ${bindir}/envsubst \
                           ${bindir}/gettext.sh \
-                         ${libdir}/libasprintf.so* \
+                         ${libdir}/libasprintf${SODEV} \
                           ${libdir}/GNU.Gettext.dll \
                          "
  FILES_gettext-runtime-dev += "${libdir}/libasprintf.a \
-- 
2.7.1