Remove some unused code
authorAndre Heinecke <aheinecke@intevation.de>
Wed, 3 May 2017 13:07:43 +0000 (15:07 +0200)
committerAndre Heinecke <aheinecke@intevation.de>
Wed, 3 May 2017 13:07:43 +0000 (15:07 +0200)
* g4wihelp.c (path_add, path_remove): Removed.
* installer.nsi (AddToPath, RemoveFromPath, Regkey_for_Env): Removed.
* uninst-gpg4win.nsi (-un.gpg4win): Remove obsolete remove from
path call.

src/g4wihelp.c
src/installer.nsi
src/uninst-gpg4win.nsi

index 3fc742e..12b859a 100644 (file)
@@ -966,181 +966,6 @@ read_w32_registry_string (HKEY root, const char *dir, const char *name)
 }
 
 
-#define ENV_HK HKEY_LOCAL_MACHINE
-#define ENV_REG "SYSTEM\\CurrentControlSet\\Control\\" \
-    "Session Manager\\Environment"
-  /* The following setting can be used for a per-user setting.  */
-#if 0
-#define ENV_HK HKEY_CURRENT_USER
-#define ENV_REG "Environment"
-#endif
-/* Due to a bug in Windows7 (kb 2685893) we better but a lower limit
-   than 8191 on the maximum length of the PATH variable.  Note, that
-   depending on the used toolchain we used to have a 259 byte limit in
-   the past.  */
-#define PATH_LENGTH_LIMIT 2047
-
-void __declspec(dllexport)
-path_add (HWND hwndParent, int string_size, char *variables,
-         stack_t **stacktop, extra_parameters_t *extra)
-{
-  char dir[PATH_LENGTH_LIMIT];
-  char *path;
-  char *path_new;
-  int path_new_size;
-  char *comp;
-  const char delims[] = ";";
-  HKEY key_handle = 0;
-
-  g_hwndParent = hwndParent;
-  EXDLL_INIT();
-
-  setuservariable (INST_R0, "0");
-
-/*   MessageBox (g_hwndParent, "XXX 1", 0, MB_OK); */
-
-  /* The expected stack layout: path component.  */
-  if (popstring (dir, sizeof (dir)))
-    return;
-
-/*   MessageBox (g_hwndParent, "XXX 2", 0, MB_OK); */
-
-  path = read_w32_registry_string (ENV_HK, ENV_REG, "Path");
-  if (! path)
-    {
-      MessageBox (g_hwndParent, "No PATH variable found", 0, MB_OK);
-      return;
-    }
-
-/*   MessageBox (g_hwndParent, "XXX 3", 0, MB_OK); */
-
-  /* Old path plus semicolon plus dir plus terminating nul.  */
-  path_new_size = strlen (path) + 1 + strlen (dir) + 1;
-  if (path_new_size > PATH_LENGTH_LIMIT)
-    {
-      MessageBox (g_hwndParent, "PATH env variable too big", 0, MB_OK);
-      free (path);
-      return;
-    }
-
-/*   MessageBox (g_hwndParent, "XXX 4", 0, MB_OK); */
-
-  path_new = malloc (path_new_size);
-  if (!path_new)
-    {
-      free (path);
-      return;
-    }
-
-/*   MessageBox (g_hwndParent, "XXX 5", 0, MB_OK); */
-
-  strcpy (path_new, path);
-  strcat (path_new, ";");
-  strcat (path_new, dir);
-
-/*   MessageBox (g_hwndParent, "XXX 6", 0, MB_OK); */
-/*   MessageBox (g_hwndParent, dir, 0, MB_OK); */
-/*   MessageBox (g_hwndParent, "XXX 7", 0, MB_OK); */
-
-  /* Check if the directory already exists in the path.  */
-  comp = strtok (path, delims);
-  do
-    {
-/*       MessageBox (g_hwndParent, comp, 0, MB_OK); */
-
-      if (!strcmp (comp, dir))
-       {
-         free (path);
-         free (path_new);
-         return;
-       }
-      comp = strtok (NULL, delims);
-    }
-  while (comp);
-  free (path);
-
-/*   MessageBox (g_hwndParent, "XXX 8", 0, MB_OK); */
-
-  /* Set a key for our CLSID.  */
-  RegCreateKey (ENV_HK, ENV_REG, &key_handle);
-  RegSetValueEx (key_handle, "Path", 0, REG_EXPAND_SZ,
-                path_new, path_new_size);
-  RegCloseKey (key_handle);
-  SetEnvironmentVariable("PATH", path_new);
-  free (path_new);
-
-/*   MessageBox (g_hwndParent, "XXX 9", 0, MB_OK); */
-
-  setuservariable (INST_R0, "1");
-}
-
-
-void __declspec(dllexport)
-path_remove (HWND hwndParent, int string_size, char *variables,
-            stack_t **stacktop, extra_parameters_t *extra)
-{
-  char dir[PATH_LENGTH_LIMIT];
-  char *path;
-  char *path_new;
-  int path_new_size;
-  char *comp;
-  const char delims[] = ";";
-  HKEY key_handle = 0;
-  int changed = 0;
-  int count = 0;
-
-  g_hwndParent = hwndParent;
-  EXDLL_INIT();
-
-  setuservariable (INST_R0, "0");
-
-  /* The expected stack layout: path component.  */
-  if (popstring (dir, sizeof (dir)))
-    return;
-
-  path = read_w32_registry_string (ENV_HK, ENV_REG, "Path");
-  /* Old path plus semicolon plus dir plus terminating nul.  */
-  path_new_size = strlen (path) + 1;
-  path_new = malloc (path_new_size);
-  if (!path_new)
-    {
-      free (path);
-      return;
-    }
-  path_new[0] = '\0';
-
-  /* Compose the new path.  */
-  comp = strtok (path, delims);
-  do
-    {
-      if (strcmp (comp, dir))
-       {
-         if (count != 0)
-           strcat (path_new, ";");
-         strcat (path_new, comp);
-         count++;
-       }
-      else
-       changed = 1;
-
-      comp = strtok (NULL, delims);
-    }
-  while (comp);
-  free (path);
-
-  if (! changed)
-    return;
-
-  /* Set a key for our CLSID.  */
-  RegCreateKey (ENV_HK, ENV_REG, &key_handle);
-  RegSetValueEx (key_handle, "Path", 0, REG_EXPAND_SZ,
-                path_new, path_new_size);
-  RegCloseKey (key_handle);
-  free (path_new);
-
-  setuservariable (INST_R0, "1");
-}
-
 /** @brief Kill processes with the name name.
  *
  * This function tries to kill a process using ExitProcess.
index 20b6e34..59663c4 100644 (file)
 
 !include "WinMessages.nsh"
 
-# Define for the registry key to change the environment.  The
-# commented one may be used if the setting should affect only the
-# current user.
-!define Regkey_for_Env \
-    'HKLM "SYSTEM\CurrentControlSet\Control\Session Manager\Environment"'
-# !define Regkey_for_Env 'HKCU "Environment"'
-
-
 # We use the modern UI 2.
 !ifdef DEBUG
 !include "MUI2.nsh"
 # Set the package name.  Note that this name should not be sufficed
 #  with the version because this would get displayed in the start menu.
 Name "${PRETTY_PACKAGE}"
-
-
-# Set the output filename.
-!ifdef GPG4WIN_VANILLA
-OutFile "${PACKAGE}-vanilla-${VERSION}.exe"
-BrandingText "${PRETTY_PACKAGE}-vanilla-${VERSION}"
-!else
-!ifdef GPG4WIN_LIGHT
-OutFile "${PACKAGE}-light-${VERSION}.exe"
-BrandingText "${PRETTY_PACKAGE}-light-${VERSION}"
-!else
 OutFile "${PACKAGE}-${VERSION}.exe"
 BrandingText "${PRETTY_PACKAGE}-${VERSION}"
-!endif
-!endif
 
 # Details button conflicts with splashscreen
 ShowInstDetails nevershow
@@ -444,9 +423,7 @@ FunctionEnd
 # Called right before installation
 Function BeforeInstallHooks
     Call PrintCloseOtherApps
-!ifndef GPG4WIN_VANILLA
     Call CheckClawsUninstall
-!endif
 FunctionEnd
 
 # Called right before the final page to show more warnings.
@@ -676,28 +653,6 @@ Function TrimNewlines
 FunctionEnd
 
 
-# AddToPath - Adds the given dir to the search path.
-#        Input - head of the stack
-Function AddToPath
-  Exch $0
-  g4wihelp::path_add "$0"
-  StrCmp $R5 "0" add_to_path_done
-  SendMessage ${HWND_BROADCAST} ${WM_WININICHANGE} 0 "STR:Environment" /TIMEOUT=5000
-  add_to_path_done:
-  Pop $0
-FunctionEnd
-
-# RemoveFromPath - Remove a given dir from the path
-#     Input: head of the stack
-Function un.RemoveFromPath
-  Exch $0
-  g4wihelp::path_remove "$0"
-  StrCmp $R5 "0" remove_from_path_done
-  SendMessage ${HWND_BROADCAST} ${WM_WININICHANGE} 0 "STR:Environment" /TIMEOUT=5000
-  remove_from_path_done:
-  Pop $0
-FunctionEnd
-
 Function .onInit
   Call G4wRunOnce
 
index 22820e7..80e77d6 100644 (file)
@@ -26,10 +26,6 @@ Section "-un.gpg4win"
 
   DeleteRegValue HKLM "Software\GNU\GnuPG" "Install Directory"
 
-  # Remove the public directory from the PATH
-  Push "$INSTDIR\pub"
-  Call un.RemoveFromPath
-
   # Delete gpg4win included tools
   Delete "$INSTDIR\bin\sha1sum.exe"
   Delete "$INSTDIR\bin\sha256sum.exe"