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

Encoding issue #120

Closed
dfalbel opened this Issue Jul 4, 2018 · 2 comments

Comments

Projects
None yet
2 participants
@dfalbel
Copy link

dfalbel commented Jul 4, 2018

I am experiencing encoding issues with fs::path_ext:

For example:

> fs::path_ext("ááá.zip")
Error: `m` must be a match object from `regexpr()`

I am using the latest fs version from Github:

> packageVersion("fs")
[1] ‘1.2.3’

On CRAN version I see:

> fs::path_ext("ááá.zip")
Error in fs_error(msg, ..., class = class) : unused argument (FALSE)

My system Info

> sessionInfo()
R version 3.4.3 (2017-11-30)
Platform: x86_64-apple-darwin15.6.0 (64-bit)
Running under: macOS High Sierra 10.13.6

Matrix products: default
BLAS: /System/Library/Frameworks/Accelerate.framework/Versions/A/Frameworks/vecLib.framework/Versions/A/libBLAS.dylib
LAPACK: /Library/Frameworks/R.framework/Versions/3.4/Resources/lib/libRlapack.dylib

locale:
[1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base     

other attached packages:
[1] usethis_1.3.0

loaded via a namespace (and not attached):
 [1] Rcpp_0.12.17     dplyr_0.7.5      assertthat_0.2.0 R6_2.2.2         backports_1.1.2 
 [6] magrittr_1.5     pillar_1.2.3     rlang_0.2.0.9001 fs_1.2.3         bindrcpp_0.2.2  
[11] tools_3.4.3      glue_1.2.0.9000  purrr_0.2.5      yaml_2.1.19      compiler_3.4.3  
[16] pkgconfig_2.0.1  tidyselect_0.2.4 bindr_0.1.1      tibble_1.4.2

@jimhester jimhester closed this in ffb0400 Jul 5, 2018

@jimhester

This comment has been minimized.

Copy link
Member

jimhester commented Jul 5, 2018

Thanks, should now be fixed.

@dfalbel

This comment has been minimized.

Copy link
Author

dfalbel commented Jul 5, 2018

Thank you! Works great now! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment