Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] fontconfig: fix static build
@ 2018-12-02  9:37 Fabrice Fontaine
  2018-12-02 12:40 ` Yann E. MORIN
  0 siblings, 1 reply; 4+ messages in thread
From: Fabrice Fontaine @ 2018-12-02  9:37 UTC (permalink / raw)
  To: buildroot

Retrieved patch from upstream to fix static build

Fixes:
 - http://autobuild.buildroot.org/results/17e5f9ce5e7566f5a88abfd27b7db5614c1a3086

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 ...x-the-build-issue-with-enable-static.patch | 80 +++++++++++++++++++
 1 file changed, 80 insertions(+)
 create mode 100644 package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch

diff --git a/package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch b/package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch
new file mode 100644
index 0000000000..6104f1ed10
--- /dev/null
+++ b/package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch
@@ -0,0 +1,80 @@
+From 8208f99fa1676c42bfd8d74de3e9dac5366c150c Mon Sep 17 00:00:00 2001
+From: Akira TAGOH <akira@tagoh.org>
+Date: Mon, 3 Sep 2018 04:56:16 +0000
+Subject: Fix the build issue with --enable-static
+
+Fixes https://gitlab.freedesktop.org/fontconfig/fontconfig/issues/109
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+[Retrieved from:
+https://gitlab.freedesktop.org/fontconfig/fontconfig/issues/109]
+---
+ test/test-bz106632.c | 35 ++++++++++++-----------------------
+ 1 file changed, 12 insertions(+), 23 deletions(-)
+
+(limited to 'test')
+
+diff --git a/test/test-bz106632.c b/test/test-bz106632.c
+index daa0c1e..2d67c2e 100644
+--- a/test/test-bz106632.c
++++ b/test/test-bz106632.c
+@@ -25,25 +25,26 @@
+ #ifdef HAVE_CONFIG_H
+ #include "config.h"
+ #endif
++#include <stdio.h>
+ #include <stdlib.h>
++#include <string.h>
+ #include <dirent.h>
++#include <unistd.h>
++#include <errno.h>
+ #ifndef HAVE_STRUCT_DIRENT_D_TYPE
+ #include <sys/types.h>
+ #include <sys/stat.h>
+-#include <unistd.h>
+ #endif
+-#include "fcstr.c"
+-#undef FcConfigBuildFonts
+-#undef FcConfigCreate
+-#undef FcConfigGetCurrent
+-#undef FcConfigParseAndLoadFromMemory
+-#undef FcConfigUptoDate
+-#undef FcFontList
+-#undef FcInitReinitialize
+-#undef FcPatternCreate
+-#undef FcPatternDestroy
+ #include <fontconfig/fontconfig.h>
+ 
++#ifdef _WIN32
++#  define FC_DIR_SEPARATOR         '\\'
++#  define FC_DIR_SEPARATOR_S       "\\"
++#else
++#  define FC_DIR_SEPARATOR         '/'
++#  define FC_DIR_SEPARATOR_S       "/"
++#endif
++
+ #ifdef HAVE_MKDTEMP
+ #define fc_mkdtemp	mkdtemp
+ #else
+@@ -154,18 +155,6 @@ unlink_dirs (const char *dir)
+     return ret;
+ }
+ 
+-FcChar8 *
+-FcLangNormalize (const FcChar8 *lang)
+-{
+-    return NULL;
+-}
+-
+-FcChar8 *
+-FcConfigHome (void)
+-{
+-    return NULL;
+-}
+-
+ int
+ main (void)
+ {
+-- 
+cgit v1.1
+
-- 
2.17.1

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH 1/1] fontconfig: fix static build
  2018-12-02  9:37 [Buildroot] [PATCH 1/1] fontconfig: fix static build Fabrice Fontaine
@ 2018-12-02 12:40 ` Yann E. MORIN
  2018-12-02 12:50   ` Fabrice Fontaine
  0 siblings, 1 reply; 4+ messages in thread
From: Yann E. MORIN @ 2018-12-02 12:40 UTC (permalink / raw)
  To: buildroot

Fabrice, All,

On 2018-12-02 10:37 +0100, Fabrice Fontaine spake thusly:
> Retrieved patch from upstream to fix static build
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/17e5f9ce5e7566f5a88abfd27b7db5614c1a3086
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
>  ...x-the-build-issue-with-enable-static.patch | 80 +++++++++++++++++++
>  1 file changed, 80 insertions(+)
>  create mode 100644 package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch
> 
> diff --git a/package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch b/package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch
> new file mode 100644
> index 0000000000..6104f1ed10
> --- /dev/null
> +++ b/package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch
> @@ -0,0 +1,80 @@
> +From 8208f99fa1676c42bfd8d74de3e9dac5366c150c Mon Sep 17 00:00:00 2001
> +From: Akira TAGOH <akira@tagoh.org>
> +Date: Mon, 3 Sep 2018 04:56:16 +0000
> +Subject: Fix the build issue with --enable-static
> +
> +Fixes https://gitlab.freedesktop.org/fontconfig/fontconfig/issues/109
> +
> +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> +[Retrieved from:
> +https://gitlab.freedesktop.org/fontconfig/fontconfig/issues/109]

Is it not better to backport and reference the actual commit, instead:
    https://gitlab.freedesktop.org/fontconfig/fontconfig/commit/8208f99fa1676c42bfd8d74de3e9dac5366c150c

Regards,
Yann E. MORIN.

> +---
> + test/test-bz106632.c | 35 ++++++++++++-----------------------
> + 1 file changed, 12 insertions(+), 23 deletions(-)
> +
> +(limited to 'test')
> +
> +diff --git a/test/test-bz106632.c b/test/test-bz106632.c
> +index daa0c1e..2d67c2e 100644
> +--- a/test/test-bz106632.c
> ++++ b/test/test-bz106632.c
> +@@ -25,25 +25,26 @@
> + #ifdef HAVE_CONFIG_H
> + #include "config.h"
> + #endif
> ++#include <stdio.h>
> + #include <stdlib.h>
> ++#include <string.h>
> + #include <dirent.h>
> ++#include <unistd.h>
> ++#include <errno.h>
> + #ifndef HAVE_STRUCT_DIRENT_D_TYPE
> + #include <sys/types.h>
> + #include <sys/stat.h>
> +-#include <unistd.h>
> + #endif
> +-#include "fcstr.c"
> +-#undef FcConfigBuildFonts
> +-#undef FcConfigCreate
> +-#undef FcConfigGetCurrent
> +-#undef FcConfigParseAndLoadFromMemory
> +-#undef FcConfigUptoDate
> +-#undef FcFontList
> +-#undef FcInitReinitialize
> +-#undef FcPatternCreate
> +-#undef FcPatternDestroy
> + #include <fontconfig/fontconfig.h>
> + 
> ++#ifdef _WIN32
> ++#  define FC_DIR_SEPARATOR         '\\'
> ++#  define FC_DIR_SEPARATOR_S       "\\"
> ++#else
> ++#  define FC_DIR_SEPARATOR         '/'
> ++#  define FC_DIR_SEPARATOR_S       "/"
> ++#endif
> ++
> + #ifdef HAVE_MKDTEMP
> + #define fc_mkdtemp	mkdtemp
> + #else
> +@@ -154,18 +155,6 @@ unlink_dirs (const char *dir)
> +     return ret;
> + }
> + 
> +-FcChar8 *
> +-FcLangNormalize (const FcChar8 *lang)
> +-{
> +-    return NULL;
> +-}
> +-
> +-FcChar8 *
> +-FcConfigHome (void)
> +-{
> +-    return NULL;
> +-}
> +-
> + int
> + main (void)
> + {
> +-- 
> +cgit v1.1
> +
> -- 
> 2.17.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH 1/1] fontconfig: fix static build
  2018-12-02 12:40 ` Yann E. MORIN
@ 2018-12-02 12:50   ` Fabrice Fontaine
  2018-12-02 13:00     ` Yann E. MORIN
  0 siblings, 1 reply; 4+ messages in thread
From: Fabrice Fontaine @ 2018-12-02 12:50 UTC (permalink / raw)
  To: buildroot

Dear Yann,
Le dim. 2 d?c. 2018 ? 13:40, Yann E. MORIN <yann.morin.1998@free.fr> a ?crit :
>
> Fabrice, All,
>
> On 2018-12-02 10:37 +0100, Fabrice Fontaine spake thusly:
> > Retrieved patch from upstream to fix static build
> >
> > Fixes:
> >  - http://autobuild.buildroot.org/results/17e5f9ce5e7566f5a88abfd27b7db5614c1a3086
> >
> > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> > ---
> >  ...x-the-build-issue-with-enable-static.patch | 80 +++++++++++++++++++
> >  1 file changed, 80 insertions(+)
> >  create mode 100644 package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch
> >
> > diff --git a/package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch b/package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch
> > new file mode 100644
> > index 0000000000..6104f1ed10
> > --- /dev/null
> > +++ b/package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch
> > @@ -0,0 +1,80 @@
> > +From 8208f99fa1676c42bfd8d74de3e9dac5366c150c Mon Sep 17 00:00:00 2001
> > +From: Akira TAGOH <akira@tagoh.org>
> > +Date: Mon, 3 Sep 2018 04:56:16 +0000
> > +Subject: Fix the build issue with --enable-static
> > +
> > +Fixes https://gitlab.freedesktop.org/fontconfig/fontconfig/issues/109
> > +
> > +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> > +[Retrieved from:
> > +https://gitlab.freedesktop.org/fontconfig/fontconfig/issues/109]
>
> Is it not better to backport and reference the actual commit, instead:
>     https://gitlab.freedesktop.org/fontconfig/fontconfig/commit/8208f99fa1676c42bfd8d74de3e9dac5366c150c
OK, I will send a v2 with a link on the actual commit.
>
> Regards,
> Yann E. MORIN.
>
> > +---
> > + test/test-bz106632.c | 35 ++++++++++++-----------------------
> > + 1 file changed, 12 insertions(+), 23 deletions(-)
> > +
> > +(limited to 'test')
> > +
> > +diff --git a/test/test-bz106632.c b/test/test-bz106632.c
> > +index daa0c1e..2d67c2e 100644
> > +--- a/test/test-bz106632.c
> > ++++ b/test/test-bz106632.c
> > +@@ -25,25 +25,26 @@
> > + #ifdef HAVE_CONFIG_H
> > + #include "config.h"
> > + #endif
> > ++#include <stdio.h>
> > + #include <stdlib.h>
> > ++#include <string.h>
> > + #include <dirent.h>
> > ++#include <unistd.h>
> > ++#include <errno.h>
> > + #ifndef HAVE_STRUCT_DIRENT_D_TYPE
> > + #include <sys/types.h>
> > + #include <sys/stat.h>
> > +-#include <unistd.h>
> > + #endif
> > +-#include "fcstr.c"
> > +-#undef FcConfigBuildFonts
> > +-#undef FcConfigCreate
> > +-#undef FcConfigGetCurrent
> > +-#undef FcConfigParseAndLoadFromMemory
> > +-#undef FcConfigUptoDate
> > +-#undef FcFontList
> > +-#undef FcInitReinitialize
> > +-#undef FcPatternCreate
> > +-#undef FcPatternDestroy
> > + #include <fontconfig/fontconfig.h>
> > +
> > ++#ifdef _WIN32
> > ++#  define FC_DIR_SEPARATOR         '\\'
> > ++#  define FC_DIR_SEPARATOR_S       "\\"
> > ++#else
> > ++#  define FC_DIR_SEPARATOR         '/'
> > ++#  define FC_DIR_SEPARATOR_S       "/"
> > ++#endif
> > ++
> > + #ifdef HAVE_MKDTEMP
> > + #define fc_mkdtemp  mkdtemp
> > + #else
> > +@@ -154,18 +155,6 @@ unlink_dirs (const char *dir)
> > +     return ret;
> > + }
> > +
> > +-FcChar8 *
> > +-FcLangNormalize (const FcChar8 *lang)
> > +-{
> > +-    return NULL;
> > +-}
> > +-
> > +-FcChar8 *
> > +-FcConfigHome (void)
> > +-{
> > +-    return NULL;
> > +-}
> > +-
> > + int
> > + main (void)
> > + {
> > +--
> > +cgit v1.1
> > +
> > --
> > 2.17.1
> >
> > _______________________________________________
> > buildroot mailing list
> > buildroot at busybox.net
> > http://lists.busybox.net/mailman/listinfo/buildroot
>
> --
> .-----------------.--------------------.------------------.--------------------.
> |  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
> | +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
> | +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
> | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
> '------------------------------^-------^------------------^--------------------'
Best Regards,

Fabrice

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH 1/1] fontconfig: fix static build
  2018-12-02 12:50   ` Fabrice Fontaine
@ 2018-12-02 13:00     ` Yann E. MORIN
  0 siblings, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2018-12-02 13:00 UTC (permalink / raw)
  To: buildroot

Fabrice, All,

On 2018-12-02 13:50 +0100, Fabrice Fontaine spake thusly:
> Le dim. 2 d?c. 2018 ? 13:40, Yann E. MORIN <yann.morin.1998@free.fr> a ?crit :
> > On 2018-12-02 10:37 +0100, Fabrice Fontaine spake thusly:
> > > Retrieved patch from upstream to fix static build
> > >
> > > Fixes:
> > >  - http://autobuild.buildroot.org/results/17e5f9ce5e7566f5a88abfd27b7db5614c1a3086
> > >
> > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> > > ---
> > >  ...x-the-build-issue-with-enable-static.patch | 80 +++++++++++++++++++
> > >  1 file changed, 80 insertions(+)
> > >  create mode 100644 package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch
> > >
> > > diff --git a/package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch b/package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch
> > > new file mode 100644
> > > index 0000000000..6104f1ed10
> > > --- /dev/null
> > > +++ b/package/fontconfig/0001-Fix-the-build-issue-with-enable-static.patch
> > > @@ -0,0 +1,80 @@
> > > +From 8208f99fa1676c42bfd8d74de3e9dac5366c150c Mon Sep 17 00:00:00 2001
> > > +From: Akira TAGOH <akira@tagoh.org>
> > > +Date: Mon, 3 Sep 2018 04:56:16 +0000
> > > +Subject: Fix the build issue with --enable-static
> > > +
> > > +Fixes https://gitlab.freedesktop.org/fontconfig/fontconfig/issues/109
> > > +
> > > +Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> > > +[Retrieved from:
> > > +https://gitlab.freedesktop.org/fontconfig/fontconfig/issues/109]
> >
> > Is it not better to backport and reference the actual commit, instead:
> >     https://gitlab.freedesktop.org/fontconfig/fontconfig/commit/8208f99fa1676c42bfd8d74de3e9dac5366c150c
> OK, I will send a v2 with a link on the actual commit.

Please, note that the patch you used is not exactly the same as the one
in that commit.

So, it's not just about pointing to the commit, but also using it.

Regards,
Yann E. MORIN.
> > > +---
> > > + test/test-bz106632.c | 35 ++++++++++++-----------------------
> > > + 1 file changed, 12 insertions(+), 23 deletions(-)
> > > +
> > > +(limited to 'test')
> > > +
> > > +diff --git a/test/test-bz106632.c b/test/test-bz106632.c
> > > +index daa0c1e..2d67c2e 100644
> > > +--- a/test/test-bz106632.c
> > > ++++ b/test/test-bz106632.c
> > > +@@ -25,25 +25,26 @@
> > > + #ifdef HAVE_CONFIG_H
> > > + #include "config.h"
> > > + #endif
> > > ++#include <stdio.h>
> > > + #include <stdlib.h>
> > > ++#include <string.h>
> > > + #include <dirent.h>
> > > ++#include <unistd.h>
> > > ++#include <errno.h>
> > > + #ifndef HAVE_STRUCT_DIRENT_D_TYPE
> > > + #include <sys/types.h>
> > > + #include <sys/stat.h>
> > > +-#include <unistd.h>
> > > + #endif
> > > +-#include "fcstr.c"
> > > +-#undef FcConfigBuildFonts
> > > +-#undef FcConfigCreate
> > > +-#undef FcConfigGetCurrent
> > > +-#undef FcConfigParseAndLoadFromMemory
> > > +-#undef FcConfigUptoDate
> > > +-#undef FcFontList
> > > +-#undef FcInitReinitialize
> > > +-#undef FcPatternCreate
> > > +-#undef FcPatternDestroy
> > > + #include <fontconfig/fontconfig.h>
> > > +
> > > ++#ifdef _WIN32
> > > ++#  define FC_DIR_SEPARATOR         '\\'
> > > ++#  define FC_DIR_SEPARATOR_S       "\\"
> > > ++#else
> > > ++#  define FC_DIR_SEPARATOR         '/'
> > > ++#  define FC_DIR_SEPARATOR_S       "/"
> > > ++#endif
> > > ++
> > > + #ifdef HAVE_MKDTEMP
> > > + #define fc_mkdtemp  mkdtemp
> > > + #else
> > > +@@ -154,18 +155,6 @@ unlink_dirs (const char *dir)
> > > +     return ret;
> > > + }
> > > +
> > > +-FcChar8 *
> > > +-FcLangNormalize (const FcChar8 *lang)
> > > +-{
> > > +-    return NULL;
> > > +-}
> > > +-
> > > +-FcChar8 *
> > > +-FcConfigHome (void)
> > > +-{
> > > +-    return NULL;
> > > +-}
> > > +-
> > > + int
> > > + main (void)
> > > + {
> > > +--
> > > +cgit v1.1
> > > +
> > > --
> > > 2.17.1
> > >
> > > _______________________________________________
> > > buildroot mailing list
> > > buildroot at busybox.net
> > > http://lists.busybox.net/mailman/listinfo/buildroot
> >
> > --
> > .-----------------.--------------------.------------------.--------------------.
> > |  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
> > | +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
> > | +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
> > | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
> > '------------------------------^-------^------------------^--------------------'
> Best Regards,
> 
> Fabrice

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2018-12-02 13:00 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-12-02  9:37 [Buildroot] [PATCH 1/1] fontconfig: fix static build Fabrice Fontaine
2018-12-02 12:40 ` Yann E. MORIN
2018-12-02 12:50   ` Fabrice Fontaine
2018-12-02 13:00     ` Yann E. MORIN

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox