search

puppetlabs / cpp-hocon

A C++ port of the Typesafe Config library.
Other
81 stars 57 forks source link

Why is include name hocon/xxx but library is libcpp-hocon? #103

Open dvhwgumby opened 6 years ago

dvhwgumby commented 6 years ago

Normally the library onject and header names are consistent -- is this an accident? Are you likely to change this?

I can happily make a pull request but if this would require other changes to your process so that you would ignore it, I won't bother. (I understand the priority of cpp-hocon to puppetlabs is not high -- understandable)

MikaelSmith commented 6 years ago

The library could probably be named libhocon. I think the library name followed from the repository, but hocon makes more sense to me when including headers for the library.

MikaelSmith commented 5 years ago

It wouldn't be much trouble to adapt our process to find libhocon. I'm in favor of this change.