JabRef / jabref

Graphical Java application for managing BibTeX and biblatex (.bib) databases
https://devdocs.jabref.org
MIT License
3.63k stars 2.59k forks source link

Decrease size of distribution #5771

Closed koppor closed 4 years ago

koppor commented 4 years ago

(I did not find the issue. Feel free to link the issue and close this one)

Our installer size decreased. I quickly looked into it and foudn following two files:

grafik

Do we really need JabRef.merged.module-....jar?

We also have lib/runtime/lib/modules with 157MB of size. Are the modules doubled here?

tobiasdiez commented 4 years ago

I think the jar files are obsolete (also the ones from the libraries) as they all should be included in the lib/modules file...

koppor commented 4 years ago

The other size info was http://builds.jabref.org/chocoDev/:

grafik

koppor commented 4 years ago

Largest difference - as we discussed. Will remove them at https://github.com/JabRef/jabref/pull/5748

koppor commented 4 years ago

Reading the badass jlink link doc:

The badass-jlink plugin takes a more pragmatic approach by combining all non-modular dependencies into a single jar. This way, only the resulting merged module needs a module descriptor.

I interpret this as that the merged module is required somehow. However, I tried to start the portable zip where this was removed (source: http://builds.jabref.org/change-upload-back-to-script/) and it worked. Strange.

tobiasdiez commented 4 years ago

It's probably related to https://github.com/beryx/badass-jlink-plugin/issues/51. I would recommend to remove these files as part of https://github.com/JabRef/jabref/blob/5ffb1b2924691f132b578c86123248fe3355e3a7/build.gradle#L699

koppor commented 4 years ago

Windows

grafik

Linux

grafik

Mac

grafik

koppor commented 4 years ago

I would assume

Which assumption is wrong?

tobiasdiez commented 4 years ago

The first. jpackageImage creates the so-called application image, which contains everything you need to run JabRef (distribution/JabRef). The installer are created afterwards in jpackageInstaller (or something similar named).

Nonetheless, there appears to be a problem as the mods folder is never deleted, right?

Siedlerchr commented 4 years ago

There is a bug report for jpackage: https://bugs.openjdk.java.net/browse/JDK-8236830

tobiasdiez commented 4 years ago

Guess who reported it :smile:

koppor commented 4 years ago

Try quick shot by deleting mods folder. IF not succeed, take out of v5.1.

koppor commented 4 years ago

Idea:

koppor commented 4 years ago

Should be fixed now.