Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-3526

Update installer configuration: replace or remove setNextButtonVisible and setCancelButtonVisible deprecated methods

    Details

    • Type: Task
    • Status: To-Do (View Workflow)
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      Building main-JDK21 installers using the Docker image and generated the following warning:

      [INFO] Extracting JDK
      [INFO] Creating media file 'Windows_x86_64':
      [INFO] Created media file output directory /build/main/target/media
      [INFO] 
      [INFO]   Collecting files:
      [INFO]   Compiling launchers:
      [INFO]     Compiling launcher 'IntegratedGenomeBrowser':
      [INFO]       Signing executable
      [INFO]       Generating VM options file IntegratedGenomeBrowser.vmoptions.
      [INFO]     Compiling launcher 'Updater with silent version check':
      [INFO]       Signing executable
      [INFO]   Creating media file: 
      [INFO]     Using custom installer icon: /build/distribution/../distribution/branding/igb_128.ico
      [INFO]     Signing executable
      [INFO]     [WARNING] Warnings during script compilation:
      [INFO]           ----------
      [INFO]           In screen "Update message [Form]", property "Pre-activation expression":
      [INFO]           1. WARNING in /tmp/script2252148973850921069.java.dir/com/install4j/script/I4jScript_Internal_26.java (at line 2)
      [INFO]           	wizardContext.setNextButtonVisible(false);
      [INFO]           	              ^^^^^^^^^^^^^^^^^^^^^^^^^^^
      [INFO]           The method setNextButtonVisible(boolean) from the type WizardContext is deprecated
      [INFO]           ----------
      [INFO]           In screen "Update message [Form]", property "Pre-activation expression":
      [INFO]           2. WARNING in /tmp/script2252148973850921069.java.dir/com/install4j/script/I4jScript_Internal_26.java (at line 3)
      [INFO]           	wizardContext.setCancelButtonVisible(false);
      [INFO]           	              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      [INFO]           The method setCancelButtonVisible(boolean) from the type WizardContext is deprecated
      [INFO]           ----------
      [INFO]           2 problems (2 warnings)
      [INFO]     Zipping custom code & resources JAR file
      [INFO]     Identifying components
      [INFO]     Adding JRE (windows-amd64-21.0.1.tar.gz)
      

        Attachments

          Activity

          Hide
          nfreese Nowlan Freese added a comment -

          This warning is coming from Install4J and is not specific to the main-JDK21-with-javafx-no-theme branch. The same warning appears in the pipeline for main-JDK8 running Install4Jv9 so I do not see this as critical for the JDK-21 release, though it should be addressed as deprecated features may break in the future.

          Show
          nfreese Nowlan Freese added a comment - This warning is coming from Install4J and is not specific to the main-JDK21-with-javafx-no-theme branch. The same warning appears in the pipeline for main-JDK8 running Install4Jv9 so I do not see this as critical for the JDK-21 release, though it should be addressed as deprecated features may break in the future.
          Hide
          ann.loraine Ann Loraine added a comment -

          Thanks for investigating this Nowlan Freese!

          I think now we should put some energy and effort into fixing this problem. This will required doing a deep dive into Install4J, e.g., installing it on a local system and working with it to determine the changes that need to be made to stop using the deprecated method and replace it with something different.

          Show
          ann.loraine Ann Loraine added a comment - Thanks for investigating this Nowlan Freese ! I think now we should put some energy and effort into fixing this problem. This will required doing a deep dive into Install4J, e.g., installing it on a local system and working with it to determine the changes that need to be made to stop using the deprecated method and replace it with something different.

            People

            • Assignee:
              nfreese Nowlan Freese
              Reporter:
              ann.loraine Ann Loraine
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated: