[GnuPG Logo]    
· English ·        
Links  
   
 
authorAndre Heinecke <[email protected]>
Wed, 24 Jul 2024 17:35:01 +0000 (19:35 +0200)
committerAndre Heinecke <[email protected]>
Wed, 24 Jul 2024 17:39:21 +0000 (19:39 +0200)
* src/gpg4win.mk.in (VSD_PRODUCT, VSD_PRODUCT_PRETTY): Replace by
MSI_FNAME_PREFIX and MSI_PRODUCT respectively.

--
The complex logic with the $$($(1)_PRODUCT) macro was
required to support different products in the same
build run. This was never really used and we even
use different builds for GPD and VSD nowadays.

This simple replacement of $$($(1)_PRODUCT) by
$(MSI_FNAME_PREFIX) and $$($(1)_PRODUCT_PRETTY) by
$(MSI_PRODUCT_NAME) also fixes the issue that the
wixlib target were redefined for several times
causing a bit of error SPAM.

src/gpg4win.mk.in

index 636d9cb4528ca9d37112de022816e51e5aebf6e5..6a3ab1866d5bf9f2c91a07891ffaaeca21b64b13 100644 (file)
@@ -172,8 +172,10 @@ endif
 
 ifeq ($(IS_GPD_BUILD),yes)
   MSI_FNAME_PREFIX := GnuPG-Desktop
+  MSI_PRODUCT_NAME := GnuPG Desktop
 else
   MSI_FNAME_PREFIX := GnuPG-VS-Desktop
+  MSI_PRODUCT_NAME := GnuPG VS-Desktop
 endif
 
 # We collect the names of all pkg files used.
@@ -1052,41 +1054,25 @@ endef
 # Processed via eval as most other templates.
 define MSI_template
 
-$(1)_PRODUCT := $(msi_target_$(1)_PRODUCT)
-ifeq ($$($(1)_PRODUCT),)
-       $(1)_PRODUCT := $(VSD_PRODUCT)
-endif
-ifeq ($$($(1)_PRODUCT),)
-       $(1)_PRODUCT := Gpg4win
-endif
-
-$(1)_PRODUCT_PRETTY := $(msi_target_$(1)_PRODUCT_PRETTY)
-ifeq ($$($(1)_PRODUCT_PRETTY),)
-       $(1)_PRODUCT_PRETTY := $(VSD_PRODUCT_PRETTY)
-endif
-ifeq ($$($(1)_PRODUCT_PRETTY),)
-       $(1)_PRODUCT_PRETTY := Gpg4win
-endif
-
-$(1)-package: installers/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi
+$(1)-package: installers/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi
 
-$$($(1)_PRODUCT)-$(VERSION).wxs: $(common_nsi) $(stampdir)/stamp-final \
+$(MSI_FNAME_PREFIX)-$(VERSION).wxs: $(common_nsi) $(stampdir)/stamp-final \
                        $(foosum_exe) gpgwrap.exe \
                                make-msi.pl \
        $(addsuffix /VERSION.sig,$(addprefix gnupg-vsd/,$(msi_targets)))
        perl make-msi.pl --guids $(srcdir)/make-msi.guids \
                --manifest gpg4win-$(VERSION).files \
                --version $(VSD_VERSION) \
-               --name "$$($(1)_PRODUCT_PRETTY)" \
+               --name "$(MSI_PRODUCT_NAME)" \
                -DBUILD_DIR=. -DTOP_SRCDIR=$(top_srcdir) \
-               -DSRCDIR=$(srcdir) $(srcdir)/nsis/gpg4win.nsi > $$($(1)_PRODUCT)-$(VERSION).wxs
+               -DSRCDIR=$(srcdir) $(srcdir)/nsis/gpg4win.nsi > $(MSI_FNAME_PREFIX)-$(VERSION).wxs
 
-$(idir)/$$($(1)_PRODUCT)-$(VERSION).wixlib: \
+$(idir)/$(MSI_FNAME_PREFIX)-$(VERSION).wixlib: \
       $(stampdir)/stamp-final \
       $(foosum_exe) \
       license.blurb \
       WixUI_Gpg4win.wxs \
-      $$($(1)_PRODUCT)-$(VERSION).wxs \
+      $(MSI_FNAME_PREFIX)-$(VERSION).wxs \
       $(stampdir)/stamp-additional-signedfiles
        ($(call SETVARS_WINE,); \
        echo "Gpg4win" > $(root)/VERSION; \
@@ -1099,8 +1085,8 @@ $(idir)/$$($(1)_PRODUCT)-$(VERSION).wixlib: \
                -dInstDirEx=j: \
                -dSrcDir=i:\\src \
                -dVersion=$$$$MSI_VERSION \
-               -out k:\\$$($(1)_PRODUCT)-$(VERSION).wixobj \
-               -pedantic -wx i:\\src\\$$($(1)_PRODUCT)-$(VERSION).wxs \
+               -out k:\\$(MSI_FNAME_PREFIX)-$(VERSION).wixobj \
+               -pedantic -wx i:\\src\\$(MSI_FNAME_PREFIX)-$(VERSION).wxs \
                -arch x64 || exit 1;\
                $(WINE) $$$$WIXPREFIX2/candle.exe \
                -dInstDir=k: \
@@ -1112,11 +1098,11 @@ $(idir)/$$($(1)_PRODUCT)-$(VERSION).wixlib: \
                -pedantic -wx i:\\src\\WixUI_Gpg4win.wxs || exit 1;\
                \
                $(WINE) $$$$WIXPREFIX2/lit.exe \
-               -out k:\\$$($(1)_PRODUCT)-$(VERSION).wixlib \
+               -out k:\\$(MSI_FNAME_PREFIX)-$(VERSION).wixlib \
                -bf \
                -wx \
                -pedantic \
-               k:\\$$($(1)_PRODUCT)-$(VERSION).wixobj k:\\gpg4win-ui.wixobj || exit 1; \
+               k:\\$(MSI_FNAME_PREFIX)-$(VERSION).wixobj k:\\gpg4win-ui.wixobj || exit 1; \
                \
        ) || ERR=1; (rm $$$$WINEINST; rm $$$$WINESRC; rm $$$$WINEINSTEX); \
        if [ -n "$$$$ERR" ]; then \
@@ -1152,11 +1138,11 @@ $(idir)/$(1).wixlib: gnupg-vsd/$(1)/$(1).wxs
        )
 
 # This is generated by make-msi.pl
-gnupg-vsd/$(1)/$(1).wxs: gnupg-vsd/$(1)/VERSION.sig make-msi.pl $$($(1)_PRODUCT)-$(VERSION).wxs
+gnupg-vsd/$(1)/$(1).wxs: gnupg-vsd/$(1)/VERSION.sig make-msi.pl $(MSI_FNAME_PREFIX)-$(VERSION).wxs
        perl make-msi.pl --guids $(srcdir)/make-msi.guids \
                --manifest gpg4win-$(VERSION).files \
                --version $(VSD_VERSION) \
-               --name "$$($(1)_PRODUCT_PRETTY)" \
+               --name "$(MSI_PRODUCT_NAME)" \
                -DBUILD_DIR=. -DTOP_SRCDIR=$(top_srcdir) \
                -DSRCDIR=$(srcdir) $(srcdir)/nsis/gpg4win.nsi > /dev/null
 
@@ -1200,47 +1186,47 @@ gnupg-vsd/$(1)/VERSION: gnupg-vsd/custom.mk $(stampdir)/stamp-betanum
 gnupg-vsd/$(1)/announcement.txt: gnupg-vsd/custom.mk gnupg-vsd/announcement.de.in gnupg-vsd/announcement.en.in
        cat gnupg-vsd/$(call GETVAR,msi_target_$(1)_announcement) | \
                sed 's/VSD_VERSION/$(VSD_VERSION)/' | \
-               sed 's@GNUPG_VSD_CUSTOMER_LINK@https://download.gnupg.com/files/$(call GETVAR,msi_target_$(1)_directory)/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi@' | \
+               sed 's@GNUPG_VSD_CUSTOMER_LINK@https://download.gnupg.com/files/$(call GETVAR,msi_target_$(1)_directory)/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi@' | \
                sed 's@GNUPG_VSD_SOURCE_LINK@https://download.gnupg.com/files/src/Gnupg-VS-Desktop-$(VSD_VERSION).tar.xz@' > gnupg-vsd/$(1)/announcement.txt
 
-installers/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi: $(stampdir)/stamp-msi-base $(idir)/$(1).wixlib \
-       gnupg-vsd/$(1)/license.rtf $(idir)/$$($(1)_PRODUCT)-$(VERSION).wixlib
+installers/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi: $(stampdir)/stamp-msi-base $(idir)/$(1).wixlib \
+       gnupg-vsd/$(1)/license.rtf $(idir)/$(MSI_FNAME_PREFIX)-$(VERSION).wixlib
        scp gnupg-vsd/$(1)/license.rtf \
                 $(WINHOST):AppData/Local/Temp/gpg4win-$(VERSION); \
        scp $(idir)/$(1).wixlib \
                 $(WINHOST):AppData/Local/Temp/gpg4win-$(VERSION); \
        scp $(call GETVAR,msi_target_$(1)_branding)/*.bmp \
                 $(WINHOST):AppData/Local/Temp/gpg4win-$(VERSION); \
-       scp $(idir)/$$($(1)_PRODUCT)-$(VERSION).wixlib $(WINHOST):AppData/Local/Temp/gpg4win-$(VERSION); \
+       scp $(idir)/$(MSI_FNAME_PREFIX)-$(VERSION).wixlib $(WINHOST):AppData/Local/Temp/gpg4win-$(VERSION); \
        ssh $(WINHOST) "cd AppData/Local/Temp/gpg4win-$(VERSION) && $(WINLIGHT) \
        -cc . \
        -reusecab \
        -spdb \
        -ext WixUIExtension \
        -ext WixUtilExtension \
-       -out $$($(1)_PRODUCT)-$(VERSION)-$(1).msi \
+       -out $(MSI_FNAME_PREFIX)-$(VERSION)-$(1).msi \
         $(call GETVAR,msi_target_$(1)_l10n) \
        -dcl:high \
        -pedantic \
         \
-        $$($(1)_PRODUCT)-$(VERSION).wixlib \
+        $(MSI_FNAME_PREFIX)-$(VERSION).wixlib \
         gnupg-msi-$(gpg4win_pkg_gnupg_msi_version)-bin.wixlib \
         $(1).wixlib" | grep -v "ICE80" | grep -v "ICE57" ; \
-        scp $(WINHOST):AppData/Local/Temp/gpg4win-$(VERSION)/$$($(1)_PRODUCT)-$(VERSION)-$(1).msi \
-        installers/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi;
+        scp $(WINHOST):AppData/Local/Temp/gpg4win-$(VERSION)/$(MSI_FNAME_PREFIX)-$(VERSION)-$(1).msi \
+        installers/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi;
 
-signed_installers/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi: installers/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi
+signed_installers/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi: installers/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi
        mkdir -p signed_installers
        $$(abs_top_srcdir)/build-aux/authenticode-sign.sh --stamp \
-           --desc="$$($(1)_PRODUCT_PRETTY)" \
-           "installers/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi" \
-           "signed_installers/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi"
+           --desc="$(MSI_PRODUCT_NAME)" \
+           "installers/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi" \
+           "signed_installers/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi"
 
-$(stampdir)/stamp-$(1)-upload: signed_installers/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi \
+$(stampdir)/stamp-$(1)-upload: signed_installers/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi \
        $(stampdir)/stamp-dist-self
        @(set -e; cd signed_installers; \
-          sha256sum $$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi > \
-               $$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi.sha256; \
+          sha256sum $(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi > \
+               $(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi.sha256; \
          if [ -f "../gnupg-vsd/$(1)/customer-enc-key.asc" ]; then \
                echo "Encrypting installer"; \
                k="-f ../gnupg-vsd/$(1)/customer-enc-key.asc" ; \
@@ -1252,45 +1238,45 @@ $(stampdir)/stamp-$(1)-upload: signed_installers/$$($(1)_PRODUCT)-$(VSD_VERSION_
                gpg --no-options --batch --yes -seu $(msi_signkey) \
                        $$$${k} \
                        -f "../gnupg-vsd/general-enc-key.asc" \
-                        -o $$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi.gpg \
-                       $$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi;\
-               sha256sum $$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi.gpg > \
-                  $$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi.gpg.sha256; \
-               scp $$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi.gpg \
-                  $$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi.gpg.sha256 \
+                        -o $(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi.gpg \
+                       $(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi;\
+               sha256sum $(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi.gpg > \
+                  $(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi.gpg.sha256; \
+               scp $(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi.gpg \
+                  $(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi.gpg.sha256 \
                   $(VSD_PUBLISH_HOST)/$(call GETVAR,msi_target_$(1)_directory); \
-               echo "https://download.gnupg.com/files/$(call GETVAR,msi_target_$(1)_directory)/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi.gpg" \
+               echo "https://download.gnupg.com/files/$(call GETVAR,msi_target_$(1)_directory)/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi.gpg" \
                > $(stampdir)/stamp-$(1)-upload; \
          else \
-               gpg --yes -o $$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi.sig -bu $(msi_signkey) \
-                       $$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi; \
-               scp $$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi \
-                       $$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi.sha256 \
-                       $$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi.sig \
+               gpg --yes -o $(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi.sig -bu $(msi_signkey) \
+                       $(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi; \
+               scp $(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi \
+                       $(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi.sha256 \
+                       $(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi.sig \
                        $(VSD_PUBLISH_HOST)/$(call GETVAR,msi_target_$(1)_directory); \
                ssh `echo $(VSD_PUBLISH_HOST) | cut -d ":" -f 1` \
-                       echo "https://download.gnupg.com/files/$(call GETVAR,msi_target_$(1)_directory)/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi" \
+                       echo "https://download.gnupg.com/files/$(call GETVAR,msi_target_$(1)_directory)/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi" \
                                > $(stampdir)/stamp-$(1)-upload; \
-                       echo "https://download.gnupg.com/files/$(call GETVAR,msi_target_$(1)_directory)/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi.sig" \
+                       echo "https://download.gnupg.com/files/$(call GETVAR,msi_target_$(1)_directory)/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi.sig" \
                        >> $(stampdir)/stamp-$(1)-upload; \
          fi)
 
 .phony : $(1)
-$(1): installers/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi
+$(1): installers/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi
 
 .PHONY : $(1)-upload
 $(1)-upload: $(stampdir)/stamp-$(1)-upload $(stampdir)/stamp-dist-self
        gpg --yes -o gnupg-vsd-$(VERSION).tar.xz.sig -bau $(msi_signkey) \
                ../gpg4win-$(VERSION).tar.xz
-       rsync -vP ../gpg4win-$(VERSION).tar.xz $(VSD_PUBLISH_HOST)/src/$$($(1)_PRODUCT)-$(VERSION).tar.xz
-       rsync -vP gnupg-vsd-$(VERSION).tar.xz.sig $(VSD_PUBLISH_HOST)/src/$$($(1)_PRODUCT)-$(VERSION).tar.xz.sig
+       rsync -vP ../gpg4win-$(VERSION).tar.xz $(VSD_PUBLISH_HOST)/src/$(MSI_FNAME_PREFIX)-$(VERSION).tar.xz
+       rsync -vP gnupg-vsd-$(VERSION).tar.xz.sig $(VSD_PUBLISH_HOST)/src/$(MSI_FNAME_PREFIX)-$(VERSION).tar.xz.sig
        echo "Upload URLs:"
        cat $(stampdir)/stamp-$(1)-upload
        echo "Source links:"
-       echo "https://download.gnupg.com/files/src/$$($(1)_PRODUCT)-$(VERSION).tar.xz"
-       echo "https://download.gnupg.com/files/src/$$($(1)_PRODUCT)-$(VERSION).tar.xz.sig"
+       echo "https://download.gnupg.com/files/src/$(MSI_FNAME_PREFIX)-$(VERSION).tar.xz"
+       echo "https://download.gnupg.com/files/src/$(MSI_FNAME_PREFIX)-$(VERSION).tar.xz.sig"
 
-$(1)-signed: signed_installers/$$($(1)_PRODUCT)-$(VSD_VERSION_FILE)-$(1).msi
+$(1)-signed: signed_installers/$(MSI_FNAME_PREFIX)-$(VSD_VERSION_FILE)-$(1).msi
 
 $(1)-announce: gnupg-vsd/$(1)/announcement.txt
        set -x
@@ -1400,15 +1386,6 @@ installers: \
 # Insert the template for msi packages.
 $(foreach msipkg, $(msi_targets), $(eval $(call MSI_template,$(msipkg))))
 
-#define uniq =
-#  $(eval seen :=)
-#  $(foreach _,$1,$(if $(filter $_,${seen}),,$(eval seen += $_)))
-#  ${seen}
-#endef
-
-#$(info $(sort $msi_targets))
-#$(foreach msipkg, $(info $(call uniq,$msi_targets)), $(eval $(call MSI_template,$(msipkg))))
-
 # @emacs_local_vars_begin@
 # @emacs_local_vars_read_only@
 # @emacs_local_vars_end@
 
   
 
 
  Technical resources for this
service are sponsered by
 
  g10 Code GmbH  

Valid XHTML 1.0!     Peace!     Valid CSS!