[ARROW-8125] [C++] "arrow-tests" target broken with ninja build

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.17.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/24333
• Assignee: Wes McKinney
• Reporter: Wes McKinney
• Created: 16/Mar/20 01:40
• Updated: 11/Jan/23 07:58
• Resolved: 16/Mar/20 02:24

Description

$ ninja arrow-tests
ninja: no work to do.

According to git bisect this was introduced by

$ git bisect bad
7db3855cd4a2e2f704b8715af3a36cbef0bb2a27 is the first bad commit
commit 7db3855cd4a2e2f704b8715af3a36cbef0bb2a27
Author: Benjamin Kietzman <[e-mail]>
Date:   Mon Mar 9 16:40:21 2020 +0100

    ARROW-8014: [C++] Provide CMake targets exercising tests with a label
    
    To run a subset of the tests, use:
    ```shell-session
    $ ninja -C ~/arrow/cpp/debug-build test-arrow_dataset
    ```
    
    Closes #6547 from bkietz/8014-Provide-CMake-targets-to- and squashes the following commits:
    
    cf9bbb06a <Benjamin Kietzman> test-lable- => test-
    90a1a7f3b <Benjamin Kietzman> ARROW-8014:  Provide Cmake targets exercising tests with a label
    
    Authored-by: Benjamin Kietzman <[e-mail]>
    Signed-off-by: Antoine Pitrou <[e-mail]>

 cpp/cmake_modules/BuildUtils.cmake | 15 +++++++++++++++
 cpp/src/arrow/CMakeLists.txt       |  2 --
 2 files changed, 15 insertions(+), 2 deletions(-)

Issue Links

• links to: GitHub Pull Request #6627