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

[MWCC]: mwcceppc file path behavior broken #78

Closed
Cuyler36 opened this issue Jul 2, 2024 · 5 comments · Fixed by #81
Closed

[MWCC]: mwcceppc file path behavior broken #78

Cuyler36 opened this issue Jul 2, 2024 · 5 comments · Fixed by #81

Comments

@Cuyler36
Copy link

Cuyler36 commented Jul 2, 2024

#75 (wibo 0.6.14+) causes issues with mwcceppc paths that worked in previous versions. This causes issues on all Linux environments.

Our setup uses assetrip + assetinc to rip binary assets from the source binaries into build/include/assets/... and we have -i build/include passed into mwcceppc.exe. Since the issue started occurring with wibo 0.6.14, the issue must be these lines: https://github.com/decompals/wibo/pull/75/files#diff-35916b9fc9415b9ed5c6c9e5cb55c16f9c61754053de1ac138aa4456378610dfR685-R691

Example error:

[422/1595] CC build/iconv/src/static/version.c
FAILED: build/src/static/version.o 
/opt/devkitpro/devkitPPC/bin/powerpc-eabi-cpp -M build/iconv/src/static/version.c -MF build/src/static/version.o.d -nostdinc -I include -I tools/ppcdis/include -I build/include -I /N64_SDK/ultra/usr/include && wibo tools/1.3.2/mwcceppc.exe -O4 -char unsigned -fp hard -inline on -enum int -sdata 0 -sdata2 0 -d _LANGUAGE_C -d F3DEX_GBI_2 -d NDEBUG -d DEBUG=0 -d MUST_MATCH -nostdinc -proc gekko -maxerrors 1 -Cpp_exceptions off -I- -i include -i tools/ppcdis/include -i build/include -ir /N64_SDK/ultra/usr/include -c build/iconv/src/static/version.c -o build/src/static/version.o
### mwcceppc.exe Compiler:

User break, cancelled...
#    File: build\iconv\src\static\version.c
# -----------------------------------------
#       4: #include "assets/__Creator__.inc"
#   Error:                                 ^
#   the file 'assets/__Creator__.inc' cannot be opened
#   Too many errors printed, aborting program
[434/1595] Order floats config/rel.yml 806432c0 806432c8
ninja: build stopped: subcommand failed.
Dumping dump/forest_1st.arc
Dumping dump/forest_2nd.arc
Error running Ninja build
@jpburnett
Copy link
Contributor

Thanks for opening the issue, I'll take a look into this and see what I can find out.

@MisterSheeple
Copy link

Any updates on this?

@mkst
Copy link
Contributor

mkst commented Jul 27, 2024

Have you tried the latest commit?

@Cuyler36
Copy link
Author

Have you tried the latest commit?

Unfortunately it did not fix the issue for us. In fact, it appears even more broken than before. Now it can't even find top level include directives. These are in our /include/ directory.

[8052/9273] CC src/static/initial_menu.c
FAILED: build/src/static/initial_menu.o 
/opt/devkitpro/devkitPPC/bin/powerpc-eabi-cpp -M src/static/initial_menu.c -MF build/src/static/initial_menu.o.d -nostdinc -I include -I tools/ppcdis/include -I build/include -I ~/n64sdk/ultra/usr/include && wibo tools/sjiswrap.exe tools/1.3.2/mwcceppc.exe -O4 -char unsigned -fp hard -inline on -sdata 0 -sdata2 0 -d _LANGUAGE_C -d F3DEX_GBI_2 -d NDEBUG -d DEBUG=0 -d MUST_MATCH -nostdinc -proc gekko -maxerrors 1 -Cpp_exceptions off -I- -i include -i tools/ppcdis/include -i build/include -ir ~/n64sdk/ultra/usr/include -c src/static/initial_menu.c -o build/src/static/initial_menu.o
### mwcceppc.exe Compiler:

User break, cancelled...
#    File: src\static\initial_menu.c
# ----------------------------------
#       1: #include "initial_menu.h"
#   Error:                         ^
#   the file 'initial_menu.h' cannot be opened
#   Too many errors printed, aborting program

@Cuyler36
Copy link
Author

I can confirm that reverting to wibo 0.6.13 fixes both the issues with 0.6.14 and 0.6.15.

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 a pull request may close this issue.

4 participants