std::cout in R package BH about bh

[eddelbuettel]

We should be able to narrow this down to somewhere in your package / its use of BH.

By way of background:

• BH currently has about 50 reverse depends, these all build without the std::cout issue -- as they are all on CRAN
• I was wrestling for long time with another Boost using package of mine, and a colleague eventually tracked it down deep inside of Boost Fusion where std::cout was used as a fallback when no other output was define -- and we were able to override with Rcpp::Rcout which then uses the R stream so that the package is now on CRAN

So can I suggest that you try a minimal usage case, and build it up until you reach the full complexity of your package. Somewhere on the way we will probably find the Boost component tickling this.

Patching BH itself is somewhat out of the question, and as mentioned above, has not been needed.

[jisuk1]

I will try to analyze my package in detail. Thank you!

Could you point me to the package you mentioned above? I would like to check their source and how they overrided with Rcpp::Rcout.

[eddelbuettel]

It was RcppStreams and this change -- but I consider this to be somewhat specific to the package.

[eddelbuettel]

Also, when I grep for std::cout in your sources I still see some not-commented out uses (though I did only look very briefly and may have missed #define statements around them). But if that is the case, you may want to make sure you take care of those before turning to Boost...

[jisuk1]

I see. Thanks! I will try to analyze my code more carefully.

[eddelbuettel]

No worries. I will close this now, please do feel free to re-open it when you find something needing a change in BH.