Opened 5 years ago

Closed 2 years ago

Last modified 2 years ago

#1864 closed defect (fixed)

IzPack 5

Reported by: zzz Owned by: zzz
Priority: minor Milestone: 0.9.40
Component: installer Version: 0.9.27
Keywords: Cc: sadie, slumlord
Parent Tickets: Sensitive: no


Thought dead but finally had a first release a year ago and many more since.
install.xml would have to be migrated.

izpack-dist-5.0.10.jar is 25 MB but must install it first and then take what we need from the 57 jars in lib/ to replace the single 6 MB standalone-compiler.jar. But not clear what. We could also just remove the izpack jar from our source. See bottom of bin/compile for main class.!forum/izpack-announce

Lots of work and not clear for what gain. Install panels look a little nicer but not radically different. Documentation for 5 seems lacking. Would take a while to determine the minimal set of jars required.


Change History (12)

comment:1 Changed 3 years ago by Zlatin Balevsky

It might end up necessary to look into this because IzPack 4 seems to have problems recognizing 64-bit versions of Windows 10

comment:2 Changed 3 years ago by Zlatin Balevsky

Error message that shows in the wrapper console window:

wrapper  | --> Wrapper Started as Console
wrapper  | Java Service Wrapper Community Edition 32-bit 3.5.25
wrapper  |   Copyright (C) 1999-2014 Tanuki Software, Ltd. All Rights Reserved.
wrapper  |
wrapper  |
wrapper  | Launching a JVM...
jvm 1    | WrapperManager: Initializing...
jvm 1    | WrapperManager:
jvm 1    | WrapperManager: WARNING - Unable to load the Wrapper''s native library 'wrapper.dll'.
jvm 1    | WrapperManager:           The file is located on the path at the following location but
jvm 1    | WrapperManager:           could not be loaded:
jvm 1    | WrapperManager:             C:\Users\zlatin.balevsky\i2p\lib\wrapper.dll
jvm 1    | WrapperManager:           Please verify that the file is both readable and executable by the
jvm 1    | WrapperManager:           current user and that the file has not been corrupted in any way.
jvm 1    | WrapperManager:           One common cause of this problem is running a 32-bit version
jvm 1    | WrapperManager:           of the Wrapper with a 64-bit version of Java, or vica versa.
jvm 1    | WrapperManager:           This is a 64-bit JVM.
jvm 1    | WrapperManager:           Reported cause:
jvm 1    | WrapperManager:             C:\Users\zlatin.balevsky\i2p\lib\wrapper.dll: Can't load IA 32-bit .dll on a AMD 64-bit platform
jvm 1    | WrapperManager:           System signals will not be handled correctly.

comment:3 Changed 3 years ago by zzz


So izpack does appear to be guessing wrong. See installer/install.xml.

If there's no easy fix via install.xml, we could look at patching izpack 4.3, or research whether it's fixed in izpack 5 to either backport it, or make the move to izpack 5 sooner than later.

We could also consider doing a 64-bit-only installer build and put on our download page, but that may be even more trouble, as people probably don't know or have an easy way to find out if their OS and java install are 32 or 64 bit.

If we do consider patching, we should compare the izpack code to what we do in net.i2p.util.SystemVersion? lines 53-63, not that our code is necessarily correct either.

comment:4 Changed 3 years ago by Meeh

I got the same message, but in my case it was jbigi.dll it didn't manage to load. - That happened on the first launch after install. Starting a second time solved the issue and warning is gone.

comment:5 Changed 3 years ago by zzz

It's not izpack that is guessing wrong. The 32/64 logic is simply based on a Java System property.

I verified on Windows 10 that the property but it is installing the 32-bit wrapper.

Spent several hours getting IzPack? 5.1.2 working. It has the same problem.

This IzPack? issue may be related?

For some reason it seems the Java system properties aren't being picked up on Win 10. Works fine on linux.

Will check in IzPack? 5 support soon. Will not check in the 64 MB of izpack jars. Will make separate build targets for IzPack? 5, and you'll have to install izpack 5 separately.

comment:6 Changed 3 years ago by zzz

Owner: set to zzz
Status: newaccepted

As above, in fbfd336fe47edc8f7c42a7015ab655329efc9458 to be 0.9.35-8

comment:7 Changed 3 years ago by zzz

Milestone: eventually0.9.36

Figured out the Windows 10 problem. System properties aren't initialized soon enough, so we need to use dynamicvariables. Fixed in both izpack 4 and 5, in dd98084f5e5333c408b6237053fd51621dc156e9 0.9.35-8.

Leaving ticket open for testing, and to decide whether to switch to IzPack? 5 for the release.

comment:8 Changed 3 years ago by zzz

Tickets filed:

comment:9 Changed 3 years ago by zzz

Cc: sadie slumlord added

When moving to IzPack? 5 we may also consider changing/improving any number of things:

  • Changing themes (can be different for Windows and Linux)
  • Changing any text
  • Adding translations for untranslated text, putting on transifex
  • Changing panel order
  • Changing images
  • Changing defaults
  • Fixing and/or disabling the uninstaller
  • Changing the labels on the windows desktop icons ("restartable" and "no window")
  • Adding desktop icons on linux

Guidance for much of this may come out of the usability testing and ongoing twitch tutorials; cc: sadie and slumlord for input.

IZPACK-1631 (console mode helper) may be a blocker to using IzPack? 5 on linux for now.

comment:10 Changed 2 years ago by zzz

Milestone: 0.9.360.9.40
Resolution: fixed
Status: acceptedclosed

Switched releases to use IzPack5 in 7b304521198bd6cf1f6d662d79376c4a5ec59691 to be 0.9.40.
It's too big to bundle, so users can install IzPack? (tested with 5.1.3) and then use the installer5 or installer5-windows targets, as checked in 10 months ago.
Or, use standard build targets to use IzPack? 4 which is still bundled. No plans to remove it at this point.

comment:11 Changed 2 years ago by zzz

Note that we worked around izpack tickets 1629 and 1630 as described above;
1631 not worked around and will prevent any console installs. Opening new ticket #2492 for that.

comment:12 Changed 2 years ago by zzz

Note: Clarification of comment 10 - We only switched the windows release installer to izpack 5 in 0.9.40, because of #2492.
Switching the non-windows installer to izpack 5 for 0.9.41, after fixing #2492.

Note: See TracTickets for help on using tickets.