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

Clean up tools directory (setpaths.R) #10

Closed
PolMine opened this issue Feb 21, 2019 · 1 comment
Closed

Clean up tools directory (setpaths.R) #10

PolMine opened this issue Feb 21, 2019 · 1 comment

Comments

@PolMine
Copy link
Collaborator

PolMine commented Feb 21, 2019

The new RcppCWB version 0.2.8 will rely on a temporary registry directory as the location for a registry file with correct paths to the corpora within a package. This is necessary to cope with the planned 'staged installs' of the CRAN build machinery.

I wanted to see whether the new RcppCWB version passes all CRAN checks. If yes, some cleaning up is possible with the configure script (delete Step 3 commented out at this time), and the file setpaths.R in the tools directory can be removed.

@ablaette
Copy link
Collaborator

Indeed this file is unused and I have deleted it. Just as a mental note: The nice thing about setting paths upon installation was that you would not need the temporary registry directory. However, there is no way how you could set the paths with the distribution of binaries (for Windows and macOS). So even thought there may be hickups when you load RcppCWB and polmineR, the temporary registry is the way to go.

ablaette added a commit that referenced this issue Jul 13, 2021
ablaette added a commit that referenced this issue Jul 13, 2021
ablaette added a commit that referenced this issue Jul 15, 2021
ablaette added a commit that referenced this issue Jul 15, 2021
ablaette added a commit that referenced this issue Jul 15, 2021
ablaette added a commit that referenced this issue Jul 15, 2021
ablaette added a commit that referenced this issue Jan 5, 2022
ablaette added a commit that referenced this issue Jan 15, 2022
ablaette added a commit that referenced this issue Jan 15, 2022
ablaette added a commit that referenced this issue Jan 15, 2022
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

1 participant