Logisim-Ita / Logisim

Logisim Italian Fork
https://logisim.altervista.org/
GNU General Public License v3.0
145 stars 39 forks source link

Add maven build config #17

Closed gSpera closed 3 years ago

gSpera commented 3 years ago

This pull request is mainly oriented to improve the build system. Currently the build system is oriented around the Eclipse IDE, I propose to add maven support. This will enable build easier and automatically Logisim without the need for an external tutorial nor a graphical display. Instructions are included directly inside the README.md

This pull also implements some other little things, I moved the source from com/... to src/main/java/com/... this is because it is the standard structure for maven, probably if thwe current structure is important in some way a configuration could tell maven where to find the source

Fixed an error which cause me to not compile with the current compiler

arg7 commented 3 years ago

I have tried to compile using maven build system, but I have got an error: [ERROR] Failed to execute goal on project Logisim: Could not resolve dependencies for project logisimit:Logisim:jar:TTL7402: The following artifacts could not be resolved: com.connectina.swing:fontchooser:jar:latest, org.bric:colorpicker:jar:1.3.1: com.connectina.swing:fontchooser:jar:latest was not found in https://repo.clojars.org/ during a previous attempt. This failure was cached in the local repository and resolution is not reattempted until the update interval of clojars has elapsed or updates are forced -> [Help 1]

Looks like maven doesn't search local repository:

libs file://${basedir}/./libs

I am a complete newbie with maven, googling doesn't help very much.

I've tried to build on windows and osx with the same result.

Can you help me configure maven?

AndreaNeti commented 3 years ago

We noticed the problem and we're already trying to fix it asap

gSpera commented 3 years ago

As this pull request is closed I opened #18

You may need to run mvn dependency:purge-local-repository beofre running mvn package again