[Bug] Incorrect script/container behaviour that may result in broken installation/incompatible state

[jiaz83]

Whenever JDownloader is updating itself/Core, the main process is gone, while the helper installation process is applying the updates. The script currently doesn't properly wait for this process to finish, which then may result in incomplete installation of updates. In case of no or broken JDownloader.jar file, it is getting replaced by bundeled/old JDownloader.jar

Whenever the JDownloader.jar is replaced, it is important to also delete Core.jar file and the update directory.

Please feel free to get in contact with our support for further solution finding

1.) properly wait for updater process to finish
2.) update init script to also delete Core.jar file and update directory whenever JDownloader.jar file is replaced

[jiaz83]

Also see #132

[pspzockerscene]

I may add that me and @jiaz83 are official JDownloader staff.

[jiaz83]

@kinslayer1982 but still important to notify about this issue as others might clone/continue this project

[jiaz83]

@jaymoulin Thank you for your help and work here! Another docker project for JDownloader solved the waiting by just waiting for all java processes to finish instead of waiting for specific pid. Maybe that'll work here as well?
You need to remove Core.jar and the update folder in case you replace JDownloader.jar, both are important.

[pspzockerscene]

People like you are the main reason I don't want to code anymore

I feel this one! The only thing you can do is to ignore such comments and keep on going.

Back to topic:
The changes which jiaz means that were done in the other container project are these ones:
jlesage/docker-jdownloader-2@6a8a6b4

[jaymoulin]

Back to topic: The changes which jiaz means that were done in the other container project are these ones: jlesage/docker-jdownloader-2@6a8a6b4

This may be a good workaround if java is still a main process. This also may have an impact on performance. I (or anybody willing to PR) should be aware of that to contain this as possible