Unified the target output directory. #504

Closed
hkeepl wants to merge 1 commit from master into master
hkeepl commented 2023-02-11 08:54:44 -06:00 (Migrated from github.com)

It's hard to run or debug the the test executable program because the library and test target are compiled in seperate directories

It's hard to run or debug the the test executable program because the library and test target are compiled in seperate directories
Youw commented 2023-02-11 13:25:13 -06:00 (Migrated from github.com)

This will not be merged.

  • I have been developing/debugging HIDAPI for several years, and it hasn't been an issue for me even once;
    I presume you're the Windows-only user. On other platforms that is not even required due to mechanisms like embedded RUNPATH.
  • You may set those variable variables for you own CMake build yourself. I don't see reasons why this should be a default for all users.
  • Once you set it here - no other users may override it w/o source modifications. We don't want to have such limitation.
  • If you use an IDE with mature CMake support - it automatically adds all library paths to your PATH, so it runs/debugs just fine (e.g. I use Qt Creator Community edition).
This will not be merged. - I have been developing/debugging HIDAPI for several years, and it hasn't been an issue for me even once; I presume you're the Windows-only user. On other platforms that is not even required due to mechanisms like embedded RUNPATH. - You may set those variable variables for you own CMake build yourself. I don't see reasons why this should be a default for all users. - Once you set it here - no other users may override it w/o source modifications. We don't want to have such limitation. - If you use an IDE with mature CMake support - it automatically adds all library paths to your PATH, so it runs/debugs just fine (e.g. I use Qt Creator Community edition).

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
gh-libusb/hidapi!504
No description provided.