@giuspen commented on this pull request.


> +- sign binary files with certificate
+- create installers
+- sign installers
+"""
+
+VERSION = '2.1'
+# adjust paths to your needs ($HOME is used because expanduser() returns the 
Windows home directory)
+BASE_DIR = join(os.environ['HOME'], 'geany_build')
+SOURCE_DIR = join(os.environ['HOME'], 'git', 'geany-plugins')
+RELEASE_DIR_ORIG = join(BASE_DIR, 'release', 'geany-plugins-orig')
+RELEASE_DIR = join(BASE_DIR, 'release', 'geany-plugins')
+BUNDLE_BASE_DIR = join(BASE_DIR, 'bundle')
+BUNDLE_GEANY_PLUGINS = join(BASE_DIR, 'bundle', 'geany-plugins-dependencies')
+INSTALLER_NAME = join(BASE_DIR, f'geany-plugins-{VERSION}_setup.exe')
+
+# signing params

Similarly to what discussed in https://github.com/geany/geany/pull/4223 as @b4n 
questioned the signing part of the script, @eht16 then commented that he 
explained parts of the signing topic in 
https://github.com/geany/geany/issues/4189#issuecomment-2629001461 and we will 
probably not sign the binaries any longer and so we could also remove the 
related code.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/1408#pullrequestreview-2837759655
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany-plugins/pull/1408/review/2837759...@github.com>

Reply via email to