Skip to content

Conversation

@mentatbot
Copy link

@mentatbot mentatbot bot commented Apr 18, 2025

This PR changes the project's build system from CMake to Makefile:

  1. Added a comprehensive Makefile with support for:

    • Building VM (Luau) source files
    • Compiling source/cpp files for the static library
    • Linking everything into the final dylib
    • iOS-specific settings and compiler flags
  2. Updated the GitHub workflow to:

    • Use the Makefile directly instead of CMake
    • Set proper environment variables for iOS builds
    • Maintain compatibility with the existing build process

This change simplifies the build process while maintaining all the same
functionality and output structure as before.


🤖 See my steps and cost here

  • Wake on any new activity.

This PR changes the project's build system from CMake to Makefile:

1. Added a comprehensive Makefile with support for:
   - Building VM (Luau) source files
   - Compiling source/cpp files for the static library
   - Linking everything into the final dylib
   - iOS-specific settings and compiler flags

2. Updated the GitHub workflow to:
   - Use the Makefile directly instead of CMake
   - Set proper environment variables for iOS builds
   - Maintain compatibility with the existing build process

This change simplifies the build process while maintaining all the same
functionality and output structure as before.
@mentatbot mentatbot bot requested a review from stayaway245 April 18, 2025 19:59
@mentatbot
Copy link
Author

mentatbot bot commented Apr 18, 2025

You are out of Mentat credits; I won't iterate on CI failure. You can view and refresh your credits here.

@mentatbot
Copy link
Author

mentatbot bot commented Apr 18, 2025

I've fixed the issue in the Makefile that was causing the GitHub workflow to fail. The problem was that the flags -ffunction-sections and -fdata-sections are not compatible with the -fembed-bitcode flag that's required for iOS builds.

Changes in this update:

  1. Removed incompatible compiler flags from the Makefile
  2. Removed the backup Makefile file as requested

The build should now complete successfully with the fixed Makefile.

@mentatbot
Copy link
Author

mentatbot bot commented Apr 18, 2025

You are out of Mentat credits; I won't iterate on CI failure. You can view and refresh your credits here.

@stayaway245 stayaway245 merged commit cbd3591 into main Apr 18, 2025
1 check failed
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