cameron314 / cameron314/readerwriterqueue
Add CMake option for install into moodycamel
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 4.6k
- Forks
- 736
- Avg merge
- 10d 5h
- Merged PRs (30d)
- 1
Description
Hi,
I tend to use your excellent queues in various placements.
I recently noticed that the concurrentqueue cmake build installs with the ".../moodycamel/" sub-directory.
However, this readerwriterqueue build does not have the moodycamel part.
I tend to root the include directories to work as:
#include <moodycamel/concurrentqueue.h>
As you wouldn't want to break existing code, any chance of a cmake option that would add the "moodycamel/" part into the readerwriterqueue install location?
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by comparing the readerwriterqueue and concurrentqueue CMake install configurations to find where the include destination is defined. Add an option for the moodycamel/ install prefix while preserving the current default, then verify that installed headers support both requested layouts.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cmake, cpp
- Domain
- build-system
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100