ssardina-agts / agtcity-sarl-mw

The SARL middleware to for the Agents in City MAC game
GNU General Public License v3.0
1 stars 2 forks source link

Maven Failure to download... warnings on compile #14

Closed ayoung012 closed 4 years ago

ayoung012 commented 4 years ago

There are some packages listed as dependencies for the janusproject kernel that are not supported by maven. These cause many warnings and delays when building, as maven searches every repository known to it for these unsupported packages.

An issue has been created https://github.com/sarl/sarl/issues/1000 and is earmarked for SARL 0.12. In the meantime, excluding these in the pom.xml dependency entry for janusproject.kernel seems to remove the warnings and does not affect compilation/running of the agents.

The full dependency entry including exclusions is listed below:

        <dependency>
            <groupId>io.janusproject</groupId>
            <artifactId>io.janusproject.kernel</artifactId>
            <version>${janus.version}</version>
            <exclusions>
              <exclusion>
                <groupId>org.eclipse.platform</groupId>
                <artifactId>org.eclipse.equinox.common</artifactId>
              </exclusion>
              <exclusion>
                <groupId>org.eclipse.platform</groupId>
                <artifactId>org.eclipse.equinox.preferences</artifactId>
              </exclusion>
              <exclusion>
                <groupId>org.eclipse.platform</groupId>
                <artifactId>org.eclipse.equinox.registry</artifactId>
              </exclusion>
              <exclusion>
                <groupId>org.eclipse.platform</groupId>
                <artifactId>org.eclipse.equinox.registry</artifactId>
              </exclusion>
              <exclusion>
                <groupId>org.eclipse.platform</groupId>
                <artifactId>org.eclipse.core.jobs</artifactId>
              </exclusion>
              <exclusion>
                <groupId>org.eclipse.platform</groupId>
                <artifactId>org.eclipse.core.contenttype</artifactId>
              </exclusion>
              <exclusion>
                <groupId>org.eclipse.platform</groupId>
                <artifactId>org.eclipse.equinox.app</artifactId>
              </exclusion>
              <exclusion>
                <groupId>org.eclipse.platform</groupId>
                <artifactId>org.eclipse.osgi</artifactId>
              </exclusion>
              <exclusion>
                <groupId>org.eclipse.platform</groupId>
                <artifactId>org.eclipse.core.runtime</artifactId>
              </exclusion>
              <exclusion>
                <groupId>org.eclipse.platform</groupId>
                <artifactId>org.eclipse.core.filesystem</artifactId>
              </exclusion>
              <exclusion>
                <groupId>org.eclipse.platform</groupId>
                <artifactId>org.eclipse.core.runtime</artifactId>
              </exclusion>
              <exclusion>
                <groupId>org.eclipse.platform</groupId>
                <artifactId>org.eclipse.core.resources</artifactId>
              </exclusion>
              <exclusion>
                <groupId>org.eclipse.platform</groupId>
                <artifactId>org.eclipse.text</artifactId>
              </exclusion>
              <exclusion>
                <groupId>org.eclipse.jdt</groupId>
                <artifactId>org.eclipse.jdt.core</artifactId>
              </exclusion>
            </exclusions>
        </dependency>

I am fairly confident this can be added to the middleware pom.xml without issue.

ssardina commented 4 years ago

This is great, as I hate those "Failure to transfer" messages, so many....

I have added it to the POM and works well, let's have it.

ssardina commented 4 years ago

I had originally push this to master as it is very safe, but it's good to have it in our working branch so I cherry-picked it from 1558145