Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
SolsticeSpectrum opened this issue Dec 30, 2020 · 4 comments

Comments

@SolsticeSpectrum
Copy link

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
Copy link
Author

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

@LeafHacker
Copy link
Contributor

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

@leijurv
Copy link
Member

leijurv commented Jan 2, 2021

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

@SolsticeSpectrum
Copy link
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants