zanmato1984 commented on PR #45210:
URL: https://github.com/apache/arrow/pull/45210#issuecomment-2581918284
Hi @kou , sorry to bother you again.
The test-ubuntu-22.04-cpp-emscripten failure is related, because the
reordering from
```
"name": "ninja-debug-emscripten",
"inherits": [
"features-emscripten",
"base-debug"
],
```
to
```
"name": "ninja-debug-emscripten",
"inherits": [
"base-debug",
"features-emscripten"
],
```
messed up the `ARROW_BUILD_STATIC` (set to `OFF` in `base-debug` and to `ON`
in `features-emscripten`). This is essentially the same issue as the one the PR
is trying to fix.
We can again tune the overriding of `ARROW_BUILD_STATIC` using the same way
we did for allocators. However I want to take a step back considering about
this question: if the order of elements in a list matters functionally, maybe
we shouldn't enforce the alphabetical order at all? What do you think? Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
