ImpactDevelopment / maven

Maven repo
https://impactdevelopment.github.io/maven
9 stars 5 forks source link

Pls add some docs for including with maven so client developers don't have to violate license #4

Open SolsticeSpectrum opened 3 years ago

SolsticeSpectrum commented 3 years ago

Just add some doc for how to include Baritone. It took me a while until I realized that Impact has own maven and that Baritone can be included even into Fabric mod without using broken Fabritone.

At least like cover this

// This adds the Impact maven and spongepowered maven which is required by Baritone
repositories {
    maven {
        name = 'impactdevelopment-repo'
        url = 'https://impactdevelopment.github.io/maven/'
    }
    maven {
        name = 'spongepowered-repo'
        url = 'http://repo.spongepowered.org/maven/'
    }
    mavenCentral()
    jcenter()
}
dependencies {
    // This adds Baritone 
    modImplementation 'cabaletta:baritone-standalone:1.6'
    include 'cabaletta:baritone-standalone:1.6'

    // This adds Baritone api
    modImplementation 'cabaletta:baritone-api:1.6'
    include 'cabaletta:baritone-api:1.6'
}

With this you can include Baritone into Fabric mod. Might be helpful if you also cover how to do it with Forge. I don't work with Forge so I don't know.

SolsticeSpectrum commented 3 years ago

Okay it builds but does not work. I really don't know how to do this.

LeafHacker commented 3 years ago

Don't think upstream baritone actually supports fabric atm, but I agree @leijurv should add some maven stuff to baritone's documentation.

leijurv commented 3 years ago

I don't understand what you want, baritone is already available under LGPL? How is any license involved with this question?

SolsticeSpectrum commented 3 years ago

Well some people include baritone to fabric clients by literally taking the whole source and merging it into it.