Allow access widener conversion in NeoForge

[ThexXTURBOXx]

Just a simple, but very effective change - works fine in my tests

[Juuxel]

Thanks for the PR, but this functionality is already available for Neo, just without this API. I intentionally excluded it from #166. (See the #loom-dev discussion on this)

The design is problematic since it uses unreliable code¹ for determining the relative AW file paths. It also should not be a Property<Boolean> IMO, just a toggle method like java.withSourcesJar() etc.

You can just add the AW names directly to remapJar.atAccessWideners:

architectury-loom/src/main/java/net/fabricmc/loom/task/RemapJarTask.java

Lines 113 to 122 in 8da7cc0

	/**
	* Gets the jar paths to the access wideners that will be converted to ATs for Forge runtime.
	* If you specify multiple files, they will be merged into one.
	*
	* <p>The specified files will be converted and removed from the final jar.
	*
	* @return the property containing access widener paths in the final jar
	*/
	@Input
	public abstract SetProperty<String> getAtAccessWideners();

Footnotes

1. This breaks if the AW is inside a subdirectory of something that's not directly in this project's resources, such as a common AW. I also can't change/remove this behaviour without making a breaking change. ↩

[ThexXTURBOXx]

Oh, I am terribly sorry - I didn't know that it "moved" for NeoForge!
That one works without further tinkering just fine, so I will use this for the future :)
Again, I am very sorry - I will close this PR then!

[Juuxel]

No problem. FYI the more direct API on RemapJarTask also works on Forge – people can use the same API on both platforms for consistency if they want to.