Skip to content

chore(scripts): add compile scripts in ubuntu and macos#864

Merged
yangxk1 merged 2 commits intoapache:mainfrom
SYaoJun:0215_add_scripts
Feb 25, 2026
Merged

chore(scripts): add compile scripts in ubuntu and macos#864
yangxk1 merged 2 commits intoapache:mainfrom
SYaoJun:0215_add_scripts

Conversation

@SYaoJun
Copy link
Contributor

@SYaoJun SYaoJun commented Feb 15, 2026

Reason for this PR

Every time I compile the C++ project, I have to input a long list of compilation options, which is time-consuming and completely unnecessary.

Regarding the legacy issue #628:

On macOS, we must explicitly specify the Arrow include path; otherwise, the compilation will fail.

What changes are included in this PR?

Add two scripts on Ubuntu and MacOS

Are these changes tested?

yes

Are there any user-facing changes?

no

@codecov-commenter
Copy link

codecov-commenter commented Feb 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.21%. Comparing base (89c167a) to head (b30da2a).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #864   +/-   ##
=========================================
  Coverage     80.21%   80.21%           
  Complexity      615      615           
=========================================
  Files            93       93           
  Lines         10255    10255           
  Branches       1049     1049           
=========================================
  Hits           8226     8226           
  Misses         1789     1789           
  Partials        240      240           
Flag Coverage Δ
cpp 71.51% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@yangxk1
Copy link
Contributor

yangxk1 commented Feb 24, 2026

Hi @SYaoJun , thank you for doing this. Imo, it would be better to introduce the role of the script synchronously in readme/docs

@yangxk1
Copy link
Contributor

yangxk1 commented Feb 25, 2026

Hi @SYaoJun, We need to merge from main to let ci pass~

@yangxk1 yangxk1 merged commit 5657616 into apache:main Feb 25, 2026
12 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