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

remove msys2 support #30

Closed
wants to merge 3 commits into from
Closed

Conversation

diamant3
Copy link
Member

  • It's hard to maintain the toolchain in msys2
  • I suspect this --disable-libstdcxx-pch is the cause of the std::filesystem::directory_iterator and other libstdc++ issues

@@ -59,7 +59,7 @@ rm -rf build-$TARGET-stage2 && mkdir build-$TARGET-stage2 && cd build-$TARGET-st
--disable-libssp \
--disable-multilib \
--enable-threads=posix \
--disable-libstdcxx-pch \
# --disable-libstdcxx-pch \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will cause some issues, since the line below it will no longer be used. I suspect that's why the Mac build fails.

Copy link
Member Author

@diamant3 diamant3 Mar 31, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much better if totally remove it instead of commenting out for future usee

edit: maybe there's an issue with the missing package, can't test it because i don't have mac :(

@fjtrujy
Copy link
Member

fjtrujy commented Mar 31, 2024

This is strange... although I don't have a special interest in dropping msys2 support, on PS2 we have currently it compiling properly.
ps2toolchain-ee doesn't differ that much from psptoolchain-allegrex, so it must compile for sure too.
https://github.com/ps2dev/ps2toolchain-ee

@diamant3
Copy link
Member Author

I will close this now because the msys2 support is up and running, thanks @fjtrujy!

@diamant3 diamant3 closed this Mar 31, 2024
@diamant3 diamant3 deleted the remove-msys2-support branch July 21, 2024 00:45
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.

3 participants