Thanks for the pointers! I think I fixed it. Here is what I did [1]. To
be honest, I am not sure what was wrong before, but I followed [2],
especially the code box at the very end of the section. I wonder if this
was always wrong or if the R internals changed at some point.
[1]
https://github.com/gdkrmr/coRanking/commit/012c3e340ba64e7da0adab38c8927a92dfebe432#diff-993bc9d43b232315c8f4edcd1823237e621593cd1186436fe42783c6240d4d1fL128
[2]
https://cran.r-project.org/doc/manuals/r-release/R-exts.html#Converting-a-package-to-use-registration
On 3/20/23 17:56, Dirk Eddelbuettel wrote:
On 20 March 2023 at 12:43, Duncan Murdoch wrote:
| It appears that you have some uses of .Call() where you are passing a
| character variable as the first argument, instead of registering the
| entry point and passing the variable containing the registration
| information.
Correct. And to make the case worse, the package then calls (on line 130 of
coranking_main.cpp) the very 'R_forceSymbols(info, TRUE);' that is now
telling it about the error of using _quoted_ ("text") rather than symbol
references. Just removing line 130 may work.
| From your second link, it looks like Rcpp has some special way to
| handle entry point registration. I'm not familiar with that, I use the
| basic way described in Writing R Extensions, section 5.4 "Registering
| native routines".
Rcpp checks if a useDynLib("nameofpackage") has `, .registration=TRUE)` and
if so, creates glue code with symbols (ie what Guido desires here) instead of
text variables (as was common a long time ago).
That was the case of package 'sf' which I helped yesterday; all it took there
was to ensure it used `useDynLib("sf", .registration=TRUE)` and to re-create
the generated RcppExports.R via Rcpp's own `compileAttributes()` function (as
I did in PR that Edzer merged hours later).
But none of this helps Guido as his package prefers to do it all by hand --
which is perfectly fine and valid, of course -- but a little bit more work.
Hth, Dirk
--
Dr. Guido Kraemer
Remote Sensing Center for Earth System Research
Uni Leipzig
Talstr. 35
D-04103 Leipzig
Germany
https://rsc4earth.de/authors/gkraemer/
https://www.guido-kraemer.com
https://twitter.com/gdkrmr
Phone: +49 1577 6099439
Email:guido.krae...@uni-leipzig.de
______________________________________________
R-package-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-package-devel
