y-scope / clp

Compressed Log Processor (CLP) is a free log management tool capable of compressing logs and searching the compressed logs without decompression.
https://yscope.com
Apache License 2.0
871 stars 70 forks source link

string_utils requires an awkward include path in dependent projects #571

Open kirkrodrigues opened 2 weeks ago

kirkrodrigues commented 2 weeks ago

Bug

In dependent projects like clp-ffi-js, the string_utils library requires that we set the include path to components/core/src/clp/, but other clp-core headers require the include path to be set to components/core/src/. This leads to error-prone include statements since the former allows all top-level clp-core headers to be included directly without the clp/ prefix (e.g, <Array.hpp>).

CLP version

9f6a6ce

Environment

N/A

Reproduction steps

See https://github.com/y-scope/clp-ffi-js/blob/82079a5382230da5fe1b1cfbc0c1a4f66f34b98f/src/clp_ffi_js/ir/StreamReader.hpp#L4