yutannihilation,
@yutannihilation@fosstodon.org avatar

Some days ago, the CRAN check with R-devel started to raise "Found non-API calls to R" NOTE. I'm not sure if they are serious on disallowing these not-so-minor APIs, but what should I do? Do you take some action or just wait?

For example, rlang package now has these NOTE:

File ‘rlang/libs/rlang.so’:
Found non-API calls to R: ‘R_ClosureExpr’, ‘R_PromiseExpr’,
‘SETLENGTH’, ‘SET_ENCLOS’, ‘SET_ENVFLAGS’, ‘SET_TRUELENGTH’

https://cran.r-project.org/web/checks/check_results_rlang.html

coolbutuseless,
@coolbutuseless@fosstodon.org avatar

@yutannihilation what's the process for handling these checks?

Do I not need to do anything unless I get an email?

I guess next time I submit a package, I have to say that I've dealt with all these NOTEs

MikeMahoney218,
@MikeMahoney218@fosstodon.org avatar

@coolbutuseless @yutannihilation yeah, NOTEs won't trigger a CRAN email or any action (* until/unless they get upgraded to a warning or error), but will cause the automated checks to fail for incoming packages (sending you to human review and then probably to resubmission)

gaborcsardi,
@gaborcsardi@fosstodon.org avatar

@MikeMahoney218 @coolbutuseless @yutannihilation NOTEs can definitely trigger emails from CRAN, and also package archivals from CRAN, this happens all the time. (In fact CRAN might request an update even if your package check is clean.)

OTOH there is nothing to do until they do email you, sometimes they change their minds, or sometimes they only enforce new rules for new submissions.

For the case at hand my guess is that they will enforce it, and I am updating my packages. Only a guess...

yutannihilation,
@yutannihilation@fosstodon.org avatar

@gaborcsardi @MikeMahoney218 @coolbutuseless Really good to know, thanks. At the moment, do you feel you can live without those APIs?

gaborcsardi,
@gaborcsardi@fosstodon.org avatar

@yutannihilation @MikeMahoney218 @coolbutuseless I think my packages will be fine, yes. There is now R_NewEnv, which helps me.

For other packages it will be tougher, e.g. cpp11 uses SET_TRUELENGTH() to manage vectors with pre-allocated storage. I don't think there is a workaround there. (Well, except for nasty ones.) So every package that uses cpp11 will have that NOTE.

yutannihilation,
@yutannihilation@fosstodon.org avatar

@gaborcsardi @MikeMahoney218 @coolbutuseless I see... I don't use SET_TRUELENGTH() by myself, but have been wondering if there's any workaround. I hope there will be some alternative API.

  • All
  • Subscribed
  • Moderated
  • Favorites
  • random
  • khanakhh
  • DreamBathrooms
  • InstantRegret
  • magazineikmin
  • osvaldo12
  • mdbf
  • Youngstown
  • cisconetworking
  • slotface
  • rosin
  • thenastyranch
  • ngwrru68w68
  • kavyap
  • ethstaker
  • JUstTest
  • tacticalgear
  • modclub
  • cubers
  • Leos
  • everett
  • GTA5RPClips
  • Durango
  • anitta
  • normalnudes
  • provamag3
  • tester
  • megavids
  • lostlight
  • All magazines