search

ericprud / SWObjects

Semantic Web swiss army knife C++ libraries
MIT License
15 stars 4 forks source link

Remove Boost.log, because recent Boost versions already have it. #2

Closed RubenVerborgh closed 9 years ago

RubenVerborgh commented 10 years ago

The current version is difficult to build on systems with more recent versions of Boost, because they already include Boost.Log.

Could the code be removed from this repo, so that the system Boost.Log can be used?

ericprud commented 10 years ago

On Mar 9, 2014 2:07 PM, "Ruben Verborgh" notifications@github.com wrote:

The current version is difficult to build on systems with more recent versions of Boost, because they already include Boost.Log.

Could the code be removed from this repo, so that the system Boost.Log can be used?

The API has changed a fair amount. My attempts this far have eliminated logging altogether. I hope to get to the point where I can commit something the week after next. If you're keen to fight with this, I can send you what I've tried so far. There's an idiom in the new API which seems like it should work but it was commented out in the 1.55 examples.

— Reply to this email directly or view it on GitHub.

RubenVerborgh commented 10 years ago

Thanks, @ericprud, good to see you're working on this. Unfortunately, I'm not a C expert myself so it would be better for the library if I don't touch this :-) I'll wait for your commits;

ericprud commented 9 years ago

I should have mentioned this months ago, but this now links against the Boost.Log in boost 1.54 or later.