gama-platform / gama.old

Main repository for developing the 1.x versions of GAMA
GNU General Public License v3.0
304 stars 99 forks source link

The mess of org.apache.commons.compress in Eclipse 2023-09 #3892

Closed AlexisDrogoul closed 1 year ago

AlexisDrogoul commented 1 year ago

Describe the bug It appears that the two same versions of Eclipse (2023-09 - Build 20230907-1323) sport two different versions of org.apache.commons.compress in macOS and Windows/Linux ... with two different names ! 🙄: org.apache.commons.compress on Windows and Linux, org.apache.commons.commons-compress on macOS... See the screenshots below:

image

About Eclipse IDE 2023-09-26 09-20-01

This of course creates acute problems when sharing and building the platform (with the necessity to change back the names every time someone commits a MANIFEST.MF file !) as we tend to declare dependencies using plugins (and not packages).

Before declaring this, I will experiment three solutions:

  1. declare both plugins as optional (but it might create a problem when compiling with maven)
  2. manage the dependencies using packages only (but complicated as compress is a second-hand dependency of some jars in msi.gama.ext
  3. directly incorporate compress in msi.gama.ext and export its packages (and remove it from the features, etc.).
hqnghi88 commented 1 year ago
Screenshot 2023-09-26 at 09 33 20

It is org.apache.commons.compress for both my M1 and Win11

hqnghi88 commented 1 year ago

And it is 1.22.xxx, i cant find the version 1.23 as in your Mac.

AlexisDrogoul commented 1 year ago

@hqnghi88 did you install it with the installer or directly downloaded the DSL package ?

hqnghi88 commented 1 year ago

Yes, both downloaded for DSL package.

AlexisDrogoul commented 1 year ago

OK -- first option does not work with Maven. Second one is more complicated than planned (uses of compress are internal to some libraries). I will try the third one.

AlexisDrogoul commented 1 year ago

Seems that adding manually org.apache.commons.compress in msi.gama.ext finally did the trick. I'm closing the issue, fee free to reopen if weird things happen.