search

foonathan / lexy

C++ parsing DSL
https://lexy.foonathan.net
Boost Software License 1.0
991 stars 66 forks source link

Install only defined if lexy is top level project #205

Open adamdusty opened 2 months ago

adamdusty commented 2 months ago

Is there a reason that all the install rules are behind an if statement checking if source_dir is current_source_dir?

This makes it so if I use lexy in a library that also defines install targets, I have to manually include every lexy target (of which there are a few).

foonathan commented 2 months ago

I didn't imagine that people used it as add_subdirectory and then want to install their project. I thought for that use case you want to install lexy first, then call find_package.

But I welcome a PR that moves the install rules.