rdatsci / rtcl

R tools for the command line
65 stars 3 forks source link

Missing dependency to `stringr`? #1

Closed surmann closed 8 years ago

surmann commented 9 years ago

The message 'regex' is not an exported object from 'namespace:stringr' appears when calling rt in the console. Is a dependency to stringr missing?

surmann commented 9 years ago

Talked to Michel, nobody knows why the dependency from docopt to stringr is not installed.

surmann commented 8 years ago

After the update to R-3.3.1 x64, I try to install rt. I get the two missing dependencies to backports and chron. After calling rt from the PowerShell (yes, I use Windows) the stringr is missing. Does it make sense to add these three packages to the dependencies?

berndbischl commented 8 years ago

not really, as the package uses stringi for string ops.

can you see what exactly triggers the stringr problem? i greped for stringr in the code base and cannot find it. also r cmd check should pick this up?

surmann commented 8 years ago

Good question. I get the message "there is no package called 'stringr'" when I call rupdate from the command prompt. The corresponding call from inside R works properly.

Should we rank these problems as Windows related? Currently, I add them to the package file and rt installs them.

berndbischl commented 8 years ago

i dont think this is a windows issue. (but just guessing)

but i have no idea where stringr should be needed. if you find this, it would be simple to convert the class into stringi calls and resolve this.

sorry

adding string to DEPENDS without knowing WHY is really not a good option....

surmann commented 8 years ago

Do you have a hint how to find out where stringr is used, because I get the message "there is no package called 'stringr'" only in the console.

surmann commented 8 years ago

Here we are! roxygen2 depends on stringr.

I don`t know why, but dependencies are sometimes not installed correctly.