nealrichardson commented on PR #13082: URL: https://github.com/apache/arrow/pull/13082#issuecomment-1122384912
I still support this change, but three notes for the historical record: * An R Core developer messaged me about this failure, suggesting that we needed to update our code to handle the classed error that the parser is now returning, and I responded that the code that was causing the issue was in the check for unstated dependencies in tests, i.e. it's not in `arrow`. He responded that there was indeed something in the check code that needed to be updated to handle this and is looking into it. So it is possible that this r-devel check failure will go away on its own soon. * As this [insightful blog post](https://enpiar.com/2018/02/01/back-to-the-future/#iii-way-back-to-the-distant-past) notes (and as alluded to in a code comment), the reason for the indirection of having a separate latin1.R file was because Solaris choked on non-ASCII files. Since CRAN doesn't check on Solaris anymore, this is no longer a concern, and consolidating the files should be ok. * When I submitted 8.0.0 to CRAN yesterday, I deleted latin1.R and test-utf.R to avoid the warning. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
