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

Port to Latest Template Dev Env #68

Open
wants to merge 1 commit into
base: 12.x/forge
Choose a base branch
from

Conversation

Levviata
Copy link

@Levviata Levviata commented Feb 2, 2025

(template is latest from this commit on main c19ab24)

(most, if not all, changes are made to restructure to the latest Cleanroom dev template as stated, didn't really change much myself!)

Important:

  • Migrated mcmod.info and other related options to "gradle.properties", these should be the same unless stated.
  • Migrated dependencies to "dependencies.gradle" (under "../gradle/scripts/").
  • "Fixed" not being able to run the dev client ("runClient" gradle task), there is a big note on "dependencies.gradle" as to how I partially "fixed" (hence the quotes) it, though its not perfect! (very shittily done but hey I breakdown what I think is the problem so that's fine I guess)
  • Renamed "vintagium.mixin.json" to "mixin.vintagium.json" and related (e.i: refmap I think?). Propagate changes to SodiumMixinTweaker::getMixinConfigs) to fit style enforced "by gradle.build" by default as well.

Other minor changes:

  • Removed support for version 1.13 as the template is too embedded into Cleanroom (1.12.2 only).
  • Upgraded Gradle to 8.12
  • Upgraded foojay-resolver-convention to 0.8.0
  • Upgrade to minimum Mixin version of 0.8.5
  • In "mixin.vintagium.json" file I added the follow entry-> "target": "@env(DEFAULT)"
    This might not be wanted but the template had it so I might as well add it (no idea what it does).

Why?:

  • Partially fixes runClient task so that running the client in dev environment is possible
  • Keep stuff up to date / Maintenance

(template is latest from this commit on main [c19ab24](CleanroomMC/TemplateDevEnv@c19ab24))

Important:

- Migrated mcmod.info and other related options to "gradle.properties", these should be the same unless stated.
- Migrated dependencies to "dependencies.gradle" (under "../gradle/scripts/")
- "Fixed" not being able to run the dev client ("runClient" gradle task), there is a big note on "dependencies.gradle" as to how I partially "fixed" (hence the quotes) it.
- Renamed "vintagium.mixin.json" to "mixin.vintagium.json" (and propagate changes to SodiumMixinTweaker::getMixinConfigs) to fit style enforced by gradle.build by default

Other minor changes:

- Removed support for version 1.13 as the template is too embedded into Cleanroom (1.12.2 only so far)
- Upgraded Gradle to 8.12
- Upgraded foojay-resolver-convention to 0.8.0
- Upgrade to minimum Mixin version of 0.8.5
- In "mixin.vintagium.json" file I added the follow entry->
"target": "@env(DEFAULT)"
This might not be wanted but the template had it so I might as well add it (no idea what it does)
@Levviata
Copy link
Author

Levviata commented Feb 2, 2025

This definitely needs a review by the way, didn't do things 100% correct as I'm not that experienced in the field

@Levviata
Copy link
Author

Levviata commented Feb 3, 2025

Seems like Vintagium is hard depending both on LolASM and FermiumASM thats why runClient was failing before bdb5d0c#diff-57c8077621420d8708b8b41ff49da0f2688ba9f380fd4a9b01d04b34ba235220

Thats pretty goofy.

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.

1 participant