Updated For Use With Loom 0.5-SNAPSHOT #67
Reference in New Issue
Block a user
Delete Branch "master"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Currently due to this bug, https://github.com/FabricMC/fabric-loom/issues/280, I've added mavenCentral to the repository list. I've also added .DS_Store files from Mac OSX to the .gitignore list as there's no reason anyone would want folder setting files from Mac to be in their repo.
PR name may be a bit misleading as it doesn't actually move to loom 0.5
Oops. I forgot to actually change the version. I'll fix that.
Fixed. I updated loom and also the yarn mappings. I've somehow managed to forget to do the one thing I put in the title. 😄
@@ -5,3 +6,4 @@url = 'https://maven.fabricmc.net/'}gradlePluginPortal()mavenCentral()Is this needed?
@@ -5,3 +6,4 @@url = 'https://maven.fabricmc.net/'}gradlePluginPortal()mavenCentral()It appears that it's not necessary. It was necessary 6 hours ago as Gradle kept complaining it could not find the 3.9.0 version of this artifact. My guess is, Loom is using the maven central versions of the packages while the Gradle settings from the example mod are using JCenter to find these packages.
I would say that it's not strictly necessary except when Loom starts to use a new version of a dependency that hasn't been synced to JCenter yet. In my opinion, this would cut down on issues that relate to not finding an artifact that just hasn't synced yet (as I wasn't the only one on FabricMC's Discord today asking about it). After the artifact syncs, the link to Maven Central is no longer needed.
I'd say it's up to you whether or not I should remove the Maven Central link. Maybe we can just leave it commented out if you want.
Edit: I should mention, I noticed this issue as it caused my automated build script to fail. My build script is setup with an existing mod I've been working on for months, so I was surprised to find it all of a suddenly failing to compile (and the error message suggesting Loom out of all things). My strongest point to argue we should have Maven Central in the settings file is to help prevent dependency breakage from occurring to people who won't be able to discern the fix as easily. This will also help a lot with automated build scripts as the script failed without any dependency changes and certainly caught me by surprise. While I now personally keep Maven Central in my settings file, but this will be able to help other people who would not think about doing this. I didn't even think about it until I needed to today.
Pull request closed