arrayfire / arrayfire/arrayfire-ml

Possible anachronism in CMake `target_include_directories`

Open
#46 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
105
Forks
22
PR merge metrics
No merged PRs in 30d

Description

When trying to compile, [line 29 of the CMake file](https://github.com/arrayfire/arrayfire-ml/blob/master/CMakeLists.txt#L29) was giving me trouble. Changing the included item from `ArrayFire::af` to `af` solved the issue. I'm using CMake 3.10.2

See also:
1. https://cmake.org/cmake/help/latest/command/target_include_directories.html
2. https://gitlab.kitware.com/cmake/cmake/issues/15689

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at line 29 of CMakeLists.txt and reproduce the compile problem with CMake 3.10.2. Compare the ArrayFire::af and af forms using the linked CMake documentation and issue 15689. Done means the project compiles without requiring the reported workaround.

Written by the indexing model from the issue text.

Assessment

Tech stack
cmake, cpp
Domain
build-system
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.