-
Notifications
You must be signed in to change notification settings - Fork 230
IZPACK-1799 Bug fix: Wrong regex in JarMerge #1057
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
@jmalopoy could you also provide a test for your change? |
@reinhapa , I provided a test case triggering the issue at jmalopoy/izpack5-example-installer. Do you refer to a unit test? |
Woud be nice to integrate that test into your PR. You may look into the |
@jmalopoy any change on this? |
@reinhapa I have been unable to invest time in the integrated test :( |
No worries, want be a release soon... |
@jmalopoy I would like to understand the actual problem here. Maybe the existing Could you elaborate about the cause of the problem of those duplicate entries? Seems that the Regex create logic seems to be implemented twice and could maybe extracted to a separate method. |
@reinhapa Did you try the toy project I provided? If so, it should generate a corrupted uninstaller.jar and you will easily see the issue. |
@jmalopoy just run your installer but got no duplicates within the |
@reinhapa , I apologize for the late response. Anyways, I'll give you more details below:
If you generate the installer and install it, it will generate the uninstaller. Inspecting the contents of the uninstaller, you will find the following entries: As you can see, the entry H has been added as a directory, which is wrong. Besides the entries H/Kscs-B5-H and H/Kscs-B5-V has been added, but they must not exist. Instead, the right ones which should exist are HKscs-B5-H and HKscs-B5-V Finally, if you try to run the uninstaller, you will get the following error: Hopefully the information above will clarify the issue. |
Proposal of fix for IZPACK-1799.