Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add property_tree to get json_parser.hpp? #29

Closed
aydindemircioglu opened this issue Jun 26, 2016 · 4 comments
Closed

Add property_tree to get json_parser.hpp? #29

aydindemircioglu opened this issue Jun 26, 2016 · 4 comments

Comments

@aydindemircioglu
Copy link

sorry, probably i'm just confused, but why is property_tree/json_parser.hpp not part of BH?
is it a non-header-only? or is there any other reason?

(i'm definitely confused as my local update of RcppShark compiles well, but build_win cannot find the above file. )

@eddelbuettel
Copy link
Owner

Look at this section of the builder script: the explicitly selected Boost libraries and their dependencies are copied.

Boost property_tree is not yet part of this set.

I could look into adding it.

@eddelbuettel eddelbuettel changed the title json_parser.hpp? Add property_tree to get json_parser.hpp? Jun 26, 2016
@aydindemircioglu
Copy link
Author

i realized that in Shark sources json_parser is only used for creating a json file containing the version/build informations. this is not important for Rcppshark, so i removed the dependency.

for me the problem is solved for now, if you want to, you can close the issue.
thanks again for bh and all the other work you do!

@eddelbuettel
Copy link
Owner

I am on the fence. It is probably small relative to the large size we already have, so I may keep this open.

@eddelbuettel
Copy link
Owner

Fixed in 1.62.0-1 which is now master and which should be in CRAN "within days".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants