Skip to content

[CI] Validate Symbol Order - Take 2 - #66

Merged
SwareJonge merged 12 commits into
doldecomp:mainfrom
MB60893:ci-add-validate-symbol-order
Aug 5, 2026
Merged

[CI] Validate Symbol Order - Take 2#66
SwareJonge merged 12 commits into
doldecomp:mainfrom
MB60893:ci-add-validate-symbol-order

Conversation

@MB60893

@MB60893 MB60893 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Originally created by Mrkol as part of the Super Mario Sunshine Decompilation, this script is incredibly useful in helping identify the function symbol order based on the original linker map files that some decompilations have.

The next step is to add this script into the CI process for mkdd similarly to how sms already does this.

*First PR abandoned after submodules accidentally got included, leading to some errors which made me start over.

MB60893 added 2 commits August 5, 2026 23:57
…e [Super Mario Sunshine Decompilation](https://github.com/doldecomp/sms/blob/main/tools/validate-symbol-order.py), this script is incredibly useful in helping identify the function symbol order based on the original linker map files that some decompilations have.

The next step is to add this script into the CI process for mkdd similarly to how [sms already does this](doldecomp/sms@32a9cad).
@decomp-dev

decomp-dev Bot commented Aug 5, 2026

Copy link
Copy Markdown

Report for MarioClub_us (7a53e8d - f59bfa4)

No changes

@MB60893

MB60893 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

@SwareJonge this is now good to go.

Tested with a few files and it picked up the changed, compared things correctly and reverting the changes to the files I modified reset everything accordingly.

Now I can sleep peacefully. 😆😴

@MB60893
MB60893 marked this pull request as ready for review August 5, 2026 16:08
@MB60893 MB60893 changed the title [WIP][CI] Validate Symbol Order - Take 2 [CI] Validate Symbol Order - Take 2 Aug 5, 2026
@SwareJonge
SwareJonge merged commit 9cef9bb into doldecomp:main Aug 5, 2026
1 check passed
@MB60893
MB60893 deleted the ci-add-validate-symbol-order branch August 5, 2026 16:49
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.

2 participants