[OE-core] [PATCH 1/1] ghostscript: resolve the gconfig.h conflict

[Saul Wold]

On 06/17/2013 07:02 PM, wenzong.fan at windriver.com wrote:
> From: Wenzong Fan <wenzong.fan at windriver.com>
>
> Both base/lib.mak and psi/int.mak will create file gconfig.h:
>      lib.mak: $(CP_) $(gconfig_h) $(GLGEN)gconfig.h
>      int.mak: $(CP_) $(gconfig_h) $(PSGEN)gconfig.h
>
> If $(GLGEN) is same with $(PSGEN), the $(GLGEN)gconfig.h will be
> overridden by $(PSGEN)gconfig.h in a parallel build, and this
> maybe cause build errors.
>
> Upstream has fixed this issue, just portback its fixes from:
>      http://downloads.ghostscript.com/public/ghostscript-9.06.tar.gz
>
If ghostscript upstream has been updated, then why not just upgrade the 
package instead of backporting a patch?

Sau!

> Signed-off-by: Wenzong Fan <wenzong.fan at windriver.com>
> ---
>   .../resolve-the-gconfig.h-conflict.patch           |   84 ++++++++++++++++++++
>   .../ghostscript/ghostscript_9.05.bb                |    1 +
>   2 files changed, 85 insertions(+)
>   create mode 100644 meta/recipes-extended/ghostscript/ghostscript/resolve-the-gconfig.h-conflict.patch
>
> diff --git a/meta/recipes-extended/ghostscript/ghostscript/resolve-the-gconfig.h-conflict.patch b/meta/recipes-extended/ghostscript/ghostscript/resolve-the-gconfig.h-conflict.patch
> new file mode 100644
> index 0000000..9e030e3
> --- /dev/null
> +++ b/meta/recipes-extended/ghostscript/ghostscript/resolve-the-gconfig.h-conflict.patch
> @@ -0,0 +1,84 @@
> +Upstream-Status: Backport
> +
> +Resolve the gconfig.h conflict
> +
> +Both base/lib.mak and psi/int.mak will create file gconfig.h:
> +    lib.mak: $(CP_) $(gconfig_h) $(GLGEN)gconfig.h
> +    int.mak: $(CP_) $(gconfig_h) $(PSGEN)gconfig.h
> +
> +If $(GLGEN) is same with $(PSGEN), the $(GLGEN)gconfig.h will be
> +overridden by $(PSGEN)gconfig.h in a parallel build, and this
> +maybe cause build errors.
> +
> +Upstream has fixed this issue, just portback its fixes from:
> +    http://downloads.ghostscript.com/public/ghostscript-9.06.tar.gz
> +
> +Signed-off-by: Wenzong Fan <wenzong.fan at windriver.com>
> +
> +---
> + base/gs.mak  |   13 +++++++++----
> + base/lib.mak |    1 -
> + psi/int.mak  |    1 -
> + 3 files changed, 9 insertions(+), 6 deletions(-)
> +
> +diff --git a/base/gs.mak b/base/gs.mak
> +index b59e948..e934657 100644
> +--- a/base/gs.mak
> ++++ b/base/gs.mak
> +@@ -261,7 +261,8 @@ MKROMFS_XE=$(AUX)mkromfs$(XEAUX)
> +
> + # Define the names of the generated header files.
> + # gconfig*.h and gconfx*.h are generated dynamically.
> +-gconfig_h=$(GLGENDIR)$(D)gconfxx.h
> ++gconfig_h=$(GLGENDIR)$(D)gconfig.h
> ++gconfxx_h=$(GLGENDIR)$(D)gconfxx.h
> + gconfigf_h=$(GLGENDIR)$(D)gconfxc.h
> + gconfigd_h=$(GLGENDIR)$(D)gconfigd.h
> +
> +@@ -434,12 +435,16 @@ GCONFIG_EXTRAS=
> + ld_tr=$(GLGENDIR)$(D)ld.tr
> + $(ld_tr) : \
> +   $(GS_MAK) $(TOP_MAKEFILES) $(GLSRCDIR)$(D)version.mak $(GENCONF_XE) $(ECHOGS_XE) $(devs_tr) $(DEVS_ALL) $(GLGENDIR)$(D)libcore.dev
> +-	$(EXP)$(GENCONF_XE) $(devs_tr) -h $(gconfig_h) $(CONFILES) $(CONFLDTR) $(ld_tr)
> +-	$(EXP)$(ECHOGS_XE) -a $(gconfig_h) $(GCONFIG_EXTRAS)
> ++	$(EXP)$(GENCONF_XE) $(devs_tr) -h $(gconfxx_h) $(CONFILES) $(CONFLDTR) $(ld_tr)
> ++	$(EXP)$(ECHOGS_XE) -a $(gconfxx_h) $(GCONFIG_EXTRAS)
> +
> +-$(gconfig_h) : $(ld_tr)
> ++$(gconfxx_h) : $(ld_tr)
> + 	$(NO_OP)
> + 	
> ++$(gconfig_h) : $(gconfxx_h)
> ++	$(RM_) $(gconfig_h)
> ++	$(CP_) $(gconfxx_h) $(gconfig_h)
> ++
> + # The line above is an empty command; don't delete.
> +
> + # save our set of makefile variables that are defined in every build (paths, etc.)
> +diff --git a/base/lib.mak b/base/lib.mak
> +index 35328cb..df02e37 100644
> +--- a/base/lib.mak
> ++++ b/base/lib.mak
> +@@ -593,7 +593,6 @@ $(GLOBJ)gconfig.$(OBJ) : $(gconfig_h) $(GLSRC)gconf.c $(AK) $(gx_h)\
> +  $(gscdefs_h) $(gconf_h)\
> +  $(gxdevice_h) $(gxiclass_h) $(gxiodev_h) $(gxiparam_h) $(TOP_MAKEFILES)\
> +  $(MAKEDDIRS)
> +-	$(CP_) $(gconfig_h) $(GLGEN)gconfig.h || true
> + 	$(CP_) $(GLSRC)gconf.c $(GLGEN)gconfig.c || true
> + 	$(GLCC) $(GLO_)gconfig.$(OBJ) $(C_) $(GLGEN)gconfig.c
> +
> +diff --git a/psi/int.mak b/psi/int.mak
> +index 1099c29..fab98e6 100644
> +--- a/psi/int.mak
> ++++ b/psi/int.mak
> +@@ -272,7 +272,6 @@ $(PSOBJ)iconfig.$(OBJ) : $(gconfig_h) $(PSSRC)iconf.c $(stdio__h)\
> +  $(gconf_h) $(gconfigd_h) $(gsmemory_h) $(gstypes_h)\
> +  $(iminst_h) $(iref_h) $(ivmspace_h) $(opdef_h) $(iplugin_h)
> + 	$(RM_) $(PSGEN)iconfig.c
> +-	$(CP_) $(gconfig_h) $(PSGEN)gconfig.h || true
> + 	$(CP_) $(PSSRC)iconf.c $(PSGEN)iconfig.c || true
> + 	$(PSCC) $(PSO_)iconfig.$(OBJ) $(C_) $(PSGEN)iconfig.c
> +
> +--
> +1.7.9.5
> +
> diff --git a/meta/recipes-extended/ghostscript/ghostscript_9.05.bb b/meta/recipes-extended/ghostscript/ghostscript_9.05.bb
> index 9ec5712..c0195c8 100644
> --- a/meta/recipes-extended/ghostscript/ghostscript_9.05.bb
> +++ b/meta/recipes-extended/ghostscript/ghostscript_9.05.bb
> @@ -28,6 +28,7 @@ SRC_URI = "${SRC_URI_BASE} \
>              file://objarch.h \
>              file://ghostscript-9.02-parallel-make.patch \
>              file://ghostscript-9.05-NOT-check-endian.patch \
> +           file://resolve-the-gconfig.h-conflict.patch \
>              "
>
>   SRC_URI_class-native = "${SRC_URI_BASE} \
>