search

mllg / batchtools

Tools for computation on batch systems
https://mllg.github.io/batchtools/
GNU Lesser General Public License v3.0
170 stars 51 forks source link

Change package name after fork #238

Open kokyriakidis opened 5 years ago

kokyriakidis commented 5 years ago

I have forked batchtools and I have made some changes in order to fully work with our Cluster environment. Although I can not get it to change name. I want to load it as batchtools_cc to differentiate it against batchtools. How can I do that?

mllg commented 5 years ago

Renaming the package in DESCRIPTION should work.

kokyriakidis commented 5 years ago

No it doesn't. I changed the name in the DESCRIPTION to batchcc but when I try to install it I get the following error message:

> install_github("kokyriakidis/batchtools_cc")
Downloading GitHub repo kokyriakidis/batchtools_cc@master
✔  checking for file ‘/private/var/folders/bw/dpvcs2h51k7355d7w_81_dk40000gn/T/RtmpSsKisS/remotes1fcc28c4669d/kokyriakidis-batchtools_cc-508b9b1/DESCRIPTION’ ...
─  preparing ‘batchcc’:
✔  checking DESCRIPTION meta-information ... 
─  cleaning src
─  checking for LF line-endings in source and make files and shell scripts
─  checking for empty or unneeded directories
─  building ‘batchcc_0.9.11.tar.gz’

* installing *source* package ‘batchcc’ ...
** libs
clang -I"/Library/Frameworks/R.framework/Resources/include" -DNDEBUG   -I/usr/local/include   -fPIC  -Wall -g -O2  -c binpack.c -o binpack.o
clang -I"/Library/Frameworks/R.framework/Resources/include" -DNDEBUG   -I/usr/local/include   -fPIC  -Wall -g -O2  -c count_not_missing.c -o count_not_missing.o
clang -I"/Library/Frameworks/R.framework/Resources/include" -DNDEBUG   -I/usr/local/include   -fPIC  -Wall -g -O2  -c fill_gaps.c -o fill_gaps.o
clang -I"/Library/Frameworks/R.framework/Resources/include" -DNDEBUG   -I/usr/local/include   -fPIC  -Wall -g -O2  -c init.c -o init.o
clang -I"/Library/Frameworks/R.framework/Resources/include" -DNDEBUG   -I/usr/local/include   -fPIC  -Wall -g -O2  -c lpt.c -o lpt.o
clang -dynamiclib -Wl,-headerpad_max_install_names -undefined dynamic_lookup -single_module -multiply_defined suppress -L/Library/Frameworks/R.framework/Resources/lib -L/usr/local/lib -o batchcc.so binpack.o count_not_missing.o fill_gaps.o init.o lpt.o -F/Library/Frameworks/R.framework/.. -framework R -Wl,-framework -Wl,CoreFoundation
installing to /Library/Frameworks/R.framework/Versions/3.5/Resources/library/batchcc/libs
** R
** inst
** byte-compile and prepare package for lazy loading
** help
*** installing help indices
** building package indices
** installing vignettes
** testing if installed package can be loaded
Error: package or namespace load failed for ‘batchcc’ in library.dynam(lib, package, package.lib):
 shared object ‘batchtools.so’ not found
Error: loading failed
Execution halted
ERROR: loading failed
* removing ‘/Library/Frameworks/R.framework/Versions/3.5/Resources/library/batchcc’
Error: Failed to install 'batchcc' from GitHub:
  (converted from warning) installation of package ‘/var/folders/bw/dpvcs2h51k7355d7w_81_dk40000gn/T//RtmpSsKisS/file1fcc16bbd9da/batchcc_0.9.11.tar.gz’ had non-zero exit status
mllg commented 5 years ago

You need to change "batchtools" to "batchcc" in the roxygen-tags @useDynLib. If in doubt, you could maybe also replace all occurrences of "batchtools" with "batchcc"?

kokyriakidis commented 5 years ago

Changing the @useDynLib didn't work. There should be an easier way to change the load name than changing all batchtools to batchcc. This gets it really hard to update also..

Error: package or namespace load failed for ‘batchcc’ in FUN(X[[i]], ...):
 no such symbol c_binpack in package /home/kokyriakidis/R/x86_64-pc-linux-gnu-library/3.4/batchcc/libs/batchcc.so
Error: loading failed
Execution halted
ERROR: loading failed