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

CLI: Dump and replace support #860

Merged
merged 3 commits into from
Apr 26, 2022
Merged

Conversation

Miepee
Copy link
Contributor

@Miepee Miepee commented Apr 26, 2022

This PR adds the following functionality to the CLI:

  • dump and replace individual code entries
  • dump and replace all code entries
  • dump all embedded textures
  • replace all or individual embedded textures
  • dump all strings

It does currently not add the functionality to the interactive menu. The only way right now to access these is via the parameters.

@Grossley Grossley merged commit 034250b into UnderminersTeam:master Apr 26, 2022
UndertaleModCli/Program.cs Show resolved Hide resolved
UndertaleModCli/Program.cs Show resolved Hide resolved
UndertaleModCli/Program.cs Show resolved Hide resolved
UndertaleModCli/Program.cs Show resolved Hide resolved
UndertaleModCli/Program.cs Show resolved Hide resolved
UndertaleModCli/Program.cs Show resolved Hide resolved
@Miepee Miepee deleted the johnny branch May 8, 2022 07:35
@suphamster
Copy link

suphamster commented Jul 26, 2022

I can't get to work cli mass embed textures import after cli dump command working (using Win10x64).
To reproduce bug:

  1. dump embed textures: "UndertaleModCli dump -t data.win"
  2. import 'em back with: "UndertaleModCli replace -t UMT_REPLACE_ALL=./EmbeddedTextures -o temp.win data.win"
    Getting errors like:
    Data file does not contain an embedded texture named Texture 0.png!
    Data file does not contain an embedded texture named Texture 1.png!

If I'm doing import all emb textures in GUI it works (but after reneming "Texture 0.png" -> "0.png" and etc.) but in cli still not.

@Miepee
Copy link
Contributor Author

Miepee commented Jul 26, 2022

Commenting under a merged PR is the wrong place for this, open a new issue for it please.

@suphamster
Copy link

suphamster commented Jul 26, 2022

Commenting under a merged PR is the wrong place for this, open a new issue for it please.

Ok, sorry. Plz delete it. Created new issue there #1058

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.

None yet

4 participants