Skip to content

Conversation

@Eyizoha
Copy link
Contributor

@Eyizoha Eyizoha commented Jan 26, 2026

Purpose

  1. Fix CMAKE_OPTIONS argument passing: The original script incorrectly passed all CMake options as a single string instead of separate arguments

  2. Fix build directory naming convention: Use lowercase for build type in directory names (build-release and build-debug instead of build-Release and build-Debug)

  3. Switch default generator from Make to Ninja: Replace Make with Ninja as the default CMake generator to improve build performance

  4. Add configurable parallel build jobs: Introduce -j/--jobs option to allow customization of parallel compilation threads

Tests

API and Format

Documentation

Copy link
Collaborator

@zjw1111 zjw1111 left a comment

Choose a reason for hiding this comment

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

+1

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@lxy-9602 lxy-9602 merged commit cd35fc1 into alibaba:main Jan 27, 2026
13 checks passed
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