Skip to content

[AURON #1690] Improving build-native.sh#1691

Merged
richox merged 1 commit intoapache:masterfrom
yew1eb:improve_build_native
Dec 4, 2025
Merged

[AURON #1690] Improving build-native.sh#1691
richox merged 1 commit intoapache:masterfrom
yew1eb:improve_build_native

Conversation

@yew1eb
Copy link
Copy Markdown
Contributor

@yew1eb yew1eb commented Dec 3, 2025

Which issue does this PR close?

Closes #1690.

Rationale for this change

The checksum() function currently emits two lines (one for features_arg and one for files), which makes comparisons and logging awkward and brittle.

What changes are included in this PR?

Are there any user-facing changes?

How was this patch tested?

Master:
Pasted Graphic 2

PR:
Pasted Graphic 1

@richox richox merged commit 832b9b7 into apache:master Dec 4, 2025
172 of 173 checks passed
# limitations under the License.
#

set -ex
Copy link
Copy Markdown
Contributor

@cxzl25 cxzl25 Dec 22, 2025

Choose a reason for hiding this comment

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

This PR removes set -e which causes rust compilation to fail and will not cause the final build to fail.

[INFO] [main] warning: build failed, waiting for other jobs to finish...
[ERROR] [main] cp: target/pre/libauron.dylib: No such file or directory
[INFO] [main] build-checksum updated: 702989478eabd7003d4354a0b19ad24d
[INFO] [main] Native build completed successfully

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improving build-native.sh checksum handling

3 participants