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

RFC: Allow .url.txt files only inside src/mods/ #273

Closed
wants to merge 2 commits into from

Conversation

jkroepke
Copy link
Contributor

@jkroepke jkroepke commented Dec 25, 2018

This is an proof of concept for #268 .

Instead ignoring the hashsum completely, this build will follow the link inside the .url.txt. This has two benefits:

  • Hashsum is still calculated.
  • When building the modpack the build checks if the download is working fine.

This is backwards compatible. If the orginal file is still there, the old behavior is used.

TODO:

  • Download files if the creator will test the mod
  • Additional behavior for loaders required.

If a non dev will test this, get creator-tools-2.0.2-SNAPSHOT-all.jar.zip and provide some feedback please.

@Tomoli75
Copy link

Usage?

@jkroepke
Copy link
Contributor Author

See https://github.com/SKCraft/Launcher/wiki/Creating-Modpacks#download-from-a-custom-url

Expect you do not need the original file locally. Exactly what do you described here: #268 (comment)

Do you mean how to use is on travis? Which commands?

@Tomoli75
Copy link

Yes, indeed.

@Tomoli75
Copy link

@jkroepke
Copy link
Contributor Author

If it had worked yet, yes.

@Tomoli75
Copy link

Tomoli75 commented Mar 9, 2019

Active still? PR needs merging 🎉🎉🎉

@sk89q
Copy link
Member

sk89q commented Mar 9, 2019

It just redownloads every time?

Also syntax style is off from Oracle’s

@Tomoli75
Copy link

Tomoli75 commented Mar 9, 2019

@jkroepke

@jkroepke
Copy link
Contributor Author

@sk89q

It just redownloads every time?

Yes, a good developer may add a cache but i'm unable to integrate it.

Also syntax style is off from Oracle’s

No idea. I'm not familiar with java ...

Copy link

@Tomoli75 Tomoli75 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good still.

@hedgehog1029
Copy link
Member

Superseded by 67f3fc1

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

Successfully merging this pull request may close these issues.

Automated Travis-CI building of modpacks Use a url.txt file without the related jar?
5 participants