IzPack

SelfModifier not respecting whitespaces in paths

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 4.3.0
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    0

Description

The Selfmodifier is not respection whitespaces in paths, e.g. the classpath or the path to the java executable.

Activity

Hide
Christian d'Heureuse added a comment -

The current SVN version of SelfModifier does not work on Windows.

The Java command in SelfModifier.spawn() does not run, because quotes are used for the value of -Dself.mod.jar.

Show
Christian d'Heureuse added a comment - The current SVN version of SelfModifier does not work on Windows. The Java command in SelfModifier.spawn() does not run, because quotes are used for the value of -Dself.mod.jar.
Hide
Christian d'Heureuse added a comment -

Dennis, I have seen that you have been improving SelfModifier. Would it be possible to display a GUI error message when the Java command fails in SelfModifier.spawn()? The current situation is that on Windows the uninstaller just terminates without a visible error message.

Additionally it would be good if there was a fallback mechanism when SelfModifier fails. After a warning message, the uninstaller should continue without SelfModifier. Locked files and directories could not be removed in this case, but most parts of the application would be uninstalled, including start menu entries and desktop links.

Show
Christian d'Heureuse added a comment - Dennis, I have seen that you have been improving SelfModifier. Would it be possible to display a GUI error message when the Java command fails in SelfModifier.spawn()? The current situation is that on Windows the uninstaller just terminates without a visible error message. Additionally it would be good if there was a fallback mechanism when SelfModifier fails. After a warning message, the uninstaller should continue without SelfModifier. Locked files and directories could not be removed in this case, but most parts of the application would be uninstalled, including start menu entries and desktop links.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: