fix...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>

.github/workflows/viewer-build.yml

@@ -29,17 +29,19 @@ jobs:
 
     - uses: actions/setup-node@v6
 
-    - if: ${{ matrix.os == 'windows-latest' }}
+    - name: Windows-specific
+      if: ${{ matrix.os == 'windows-latest' }}
       run: |
         choco install wget zip
 
-    - if: ${{ matrix.os == 'ubuntu-latest' }}
+    - name: Ubuntu-specific
+      if: ${{ matrix.os == 'ubuntu-latest' }}
       run: |
         # fixes an issue when running npx sandboxing...
         sudo sysctl -w kernel.apparmor_restrict_unprivileged_userns=0
 
-    - name: Check build dependencies
-      run: make check
+    #- name: MacOS-specific
+    #  if: ${{ matrix.os == 'macos-latest' }}
 
     - name: Pre-Build
       run: make clean-all dev

Viewer/Makefile

@@ -15,10 +15,12 @@
 # 			also nodejs-legacy seems to be required in some 
 # 			contexts...
 # 		- wget
-# 		- zip / unzip / zipnote
-# 			if zipnote fails this is likely due to a bug in 
-# 			v3.0, to fix this by either upgrading to 3.1 or 
-# 			a patched version of 3.0...
+# 		- zip / unzip
+# 			NOTE: since  zipnote fails in some cases it is 
+# 				removed from dependencies.
+# 				this behavior s is likely due to a bug in 
+# 				v3.0, to fix this by either upgrading to 
+# 				3.1 or a patched version of 3.0...
 # 				for more info and patch see: 
 # 					https://goo.gl/csQmQo
 # 	- Optional
@@ -405,7 +407,7 @@ DEPENDENCIES_WEB = node npm npx
 
 DEPENDENCIES = $(DEPENDENCIES_WEB) \
 	$(DEPENDENCIES_HELP) \
-	wget zip unzip zipnote git
+	wget zip unzip git
 
 
 
@@ -559,6 +561,14 @@ run: electron-run
 install: dev
 
 
+# XXX STUB: move the builds to specific release tags... 
+.PHONY: update-release-tag
+update-release-tag: ## Move release-latest tag to HEAD
+	git push origin :refs/tags/release-latest
+	git tag -fam "TEST" release-latest
+	git push origin --tags
+
+
 ### Cleanup...
 
 .PHONY: clean
@@ -913,18 +923,21 @@ $(BUILD_DIR)/$(APP_NAME)-%.in-place.zip: $(TARGET_DIR)/$(ELECTRON_DIST) \
 	# rename electron stuff...
 #	# XXX rename "Electron Helper" on mac builds -- this needs Info.plist changes...
 #		| sed 's/\(^@ \(.*\)Electron Helper\(.*\)\)$$/\1\n@=\2$(APP_NAME) Helper\3/' 
-	zipnote "$@.tmp" \
-		| sed 's/\(^@ LICENSE\)$$/\1\n@=LICENSE.electron/' \
-		| zipnote -w "$@.tmp"
+	unzip -p "$@.tmp" LICENSE > $(BUILD_DIR)/LICENSE.electron
+	# rename app...
+#	# XXX also rename MacOS/Electron -> $(APP_NAME) -- this needs Info.plist changes...
+	unzip -d $(BUILD_DIR) "$@.tmp" "$(ELECTRON_BIN)$(EXT)"
+	zip -d "$@.tmp" "$(ELECTRON_BIN)$(EXT)"
+	mv "${BUILD_DIR}"/{"$(ELECTRON_BIN)$(EXT)","$(APP_BIN)$(EXT)"}
 	# add app.asar and friends...
 	$(MD) "$(BUILD_DIR)/$(ASAR_PATH)"
 	cp -r $(BUILD_DIR)/app.asar* "$(BUILD_DIR)/$(ASAR_PATH)/"
-	$(call zipFrom,"$(BUILD_DIR)",$@.tmp,"$(ASAR_PATH)" "$(notdir $(BUILD_INFO))" "$(LICENSE)")
-	# rename app...
-#	# XXX also rename MacOS/Electron -> $(APP_NAME) -- this needs Info.plist changes...
-	zipnote "$@.tmp" \
-		| sed 's/\(^@ $(ELECTRON_BIN)$(EXT)\)\(.*$$\)/\1\2\n@=$(APP_BIN)$(EXT)\2/' \
-		| zipnote -w "$@.tmp"
+	$(call zipFrom,"$(BUILD_DIR)",$@.tmp,\
+		"$(ASAR_PATH)" \
+		"$(notdir $(BUILD_INFO))" \
+		"$(LICENSE)" \
+		"$(LICENSE).electron" \
+		"$(APP_BIN)$(EXT)")
 	# cleanup...
 	$(RM) $(BUILD_DIR)/$(firstword $(subst /, ,$(ASAR_PATH)))
 	mv "$@.tmp" "$@"