[vlc-devel] [vlc-commits] Revert "Contribs: update fontconfig to 2.13.1"

Hugo Beauzée-Luyssen hugo at beauzee.fr
Wed Apr 1 13:28:04 CEST 2020


On Wed, Apr 1, 2020, at 1:21 PM, Thomas Guillem wrote:
> Why ?
> 
> On Wed, Apr 1, 2020, at 11:12, Jean-Baptiste Kempf wrote:
> > vlc/vlc-3.0 | branch: master | Jean-Baptiste Kempf <jb at videolan.org> | 
> > Wed Apr  1 11:12:01 2020 +0200| 
> > [6ffda1b438ad78ee148d00eb31d79f897c970fee] | committer: Jean-Baptiste 
> > Kempf
> > 
> > Revert "Contribs: update fontconfig to 2.13.1"
> > 
> > This reverts commit c76d3d324a5bd4ee280394f93bba238ad848e103.
> > 
> > > http://git.videolan.org/gitweb.cgi/vlc/vlc-3.0.git/?a=commit;h=6ffda1b438ad78ee148d00eb31d79f897c970fee
> > ---
> > 
> >  .../fontconfig/8208f99-fix-static-linking.patch    | 130 ---------------------
> >  contrib/src/fontconfig/SHA512SUMS                  |   2 +-
> >  contrib/src/fontconfig/rules.mak                   |   4 +-
> >  3 files changed, 3 insertions(+), 133 deletions(-)
> > 
> > diff --git a/contrib/src/fontconfig/8208f99-fix-static-linking.patch 
> > b/contrib/src/fontconfig/8208f99-fix-static-linking.patch
> > deleted file mode 100644
> > index 77359fcf1b..0000000000
> > --- a/contrib/src/fontconfig/8208f99-fix-static-linking.patch
> > +++ /dev/null
> > @@ -1,130 +0,0 @@
> > -From 8208f99fa1676c42bfd8d74de3e9dac5366c150c Mon Sep 17 00:00:00 2001
> > -From: Akira TAGOH <akira at tagoh.org>
> > -Date: Mon, 3 Sep 2018 04:56:16 +0000
> > -Subject: [PATCH] Fix the build issue with --enable-static
> > -
> > -Fixes https://gitlab.freedesktop.org/fontconfig/fontconfig/issues/109
> > ----
> > - doc/fcstring.fncs       | 12 ++++++++++++
> > - fontconfig/fontconfig.h |  4 ++++
> > - src/fcint.h             |  4 ----
> > - test/test-bz106632.c    | 35 ++++++++++++-----------------------
> > - 4 files changed, 28 insertions(+), 27 deletions(-)
> > -
> > -diff --git a/doc/fcstring.fncs b/doc/fcstring.fncs
> > -index 0412bbd..d5ec043 100644
> > ---- a/doc/fcstring.fncs
> > -+++ b/doc/fcstring.fncs
> > -@@ -223,6 +223,18 @@ This is just a wrapper around free(3) which helps 
> > track memory usage of
> > - strings within the fontconfig library.
> > - @@
> > - 
> > -+ at RET@		FcChar8 *
> > -+ at FUNC@		FcStrBuildFilename
> > -+ at TYPE1@		const FcChar8 *			@ARG1@		path
> > -+ at TYPE2@		...
> > -+ at PURPOSE@	Concatenate strings as a file path
> > -+ at DESC@
> > -+Creates a filename from the given elements of strings as file paths
> > -+and concatenate them with the appropriate file separator.
> > -+Arguments must be null-terminated.
> > -+This returns a newly-allocated memory which should be freed when no 
> > longer needed.
> > -+@@
> > -+
> > - @RET@		FcChar8 *
> > - @FUNC@		FcStrDirname
> > - @TYPE1@		const FcChar8 *			@ARG1@		file
> > -diff --git a/fontconfig/fontconfig.h b/fontconfig/fontconfig.h
> > -index bac1dda..af870d0 100644
> > ---- a/fontconfig/fontconfig.h
> > -+++ b/fontconfig/fontconfig.h
> > -@@ -1076,6 +1076,10 @@ FcUtf16Len (const FcChar8   *string,
> > - 	    int		    *nchar,
> > - 	    int		    *wchar);
> > - 
> > -+FcPublic FcChar8 *
> > -+FcStrBuildFilename (const FcChar8 *path,
> > -+		    ...);
> > -+
> > - FcPublic FcChar8 *
> > - FcStrDirname (const FcChar8 *file);
> > - 
> > -diff --git a/src/fcint.h b/src/fcint.h
> > -index de78cd8..a9d075a 100644
> > ---- a/src/fcint.h
> > -+++ b/src/fcint.h
> > -@@ -1282,10 +1282,6 @@ FcStrUsesHome (const FcChar8 *s);
> > - FcPrivate FcBool
> > - FcStrIsAbsoluteFilename (const FcChar8 *s);
> > - 
> > --FcPrivate FcChar8 *
> > --FcStrBuildFilename (const FcChar8 *path,
> > --		    ...);
> > --
> > - FcPrivate FcChar8 *
> > - FcStrLastSlash (const FcChar8  *path);
> > - 
> > -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)
> > - {
> > --- 
> > -2.24.1
> > -
> > diff --git a/contrib/src/fontconfig/SHA512SUMS 
> > b/contrib/src/fontconfig/SHA512SUMS
> > index 1f59c3e808..5d0a1437d4 100644
> > --- a/contrib/src/fontconfig/SHA512SUMS
> > +++ b/contrib/src/fontconfig/SHA512SUMS
> > @@ -1 +1 @@
> > -830df32e944ee21ad02a9df04787b9902af36ffc13913524acef6e38799a38c5df7a6e407cc0ff9c24455520549d53b3d85d22642a229ac654dc9269926f130b  fontconfig-2.13.1.tar.gz
> > +b17725c028be1c5e6f76c136b0ed7db1be7694cbbf217310083512708e05cdc1a824427f89082e6ef259c10297900f26cbe899f7c5762e7662855739f3eff5ea  fontconfig-2.12.3.tar.gz
> > diff --git a/contrib/src/fontconfig/rules.mak 
> > b/contrib/src/fontconfig/rules.mak
> > index a4925445eb..4d44c09597 100644
> > --- a/contrib/src/fontconfig/rules.mak
> > +++ b/contrib/src/fontconfig/rules.mak
> > @@ -1,6 +1,6 @@
> >  # fontconfig
> >  
> > -FONTCONFIG_VERSION := 2.13.1
> > +FONTCONFIG_VERSION := 2.12.3
> >  FONTCONFIG_URL := 
> > https://www.freedesktop.org/software/fontconfig/release/fontconfig-$(FONTCONFIG_VERSION).tar.gz
> >  
> >  PKGS += fontconfig
> > @@ -19,7 +19,7 @@ ifdef HAVE_WIN32
> >  	$(APPLY) $(SRC)/fontconfig/fontconfig-win32.patch
> >  	$(APPLY) $(SRC)/fontconfig/fontconfig-noxml2.patch
> >  endif
> > -	$(APPLY) $(SRC)/fontconfig/8208f99-fix-static-linking.patch
> > +	$(RM) $(UNPACK_DIR)/src/fcobjshash.gperf
> >  	$(call pkg_static, "fontconfig.pc.in")
> >  	$(MOVE)
> >  
> > 
> > _______________________________________________
> > vlc-commits mailing list
> > vlc-commits at videolan.org
> > https://mailman.videolan.org/listinfo/vlc-commits
> >
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel

It breaks the windows build

-- 
  Hugo Beauzée-Luyssen
  hugo at beauzee.fr


More information about the vlc-devel mailing list