rhub icon indicating copy to clipboard operation
rhub copied to clipboard

R CMD check requires 'checkbashisms' for compiled packages

Open gothub opened this issue 5 years ago • 8 comments

https://builder.r-hub.io/status/redland_1.0.17-13.tar.gz-0a87263b135749f386e1c44259ba3fe8

R CMD check now requires that a configure script need to pass a 'checkbashism' check to help determine if the script is portable.

The R redland package build at https://builder.r-hub.io/status/redland_1.0.17-13.tar.gz-0a87263b135749f386e1c44259ba3fe8 generated this warning, as checkbashisms wasn't available:

1880#> * checking top-level files ... WARNING
1881#> A complete check needs the 'checkbashisms' script.
1882#> See section ‘Configure and cleanup’ in the ‘Writing R Extensions’
1883#> manual.

This command is available from:

  • macOS/brew - checkbashisms'
  • Ubuntu - devscripts
  • Debian - devscripts
  • Fedora - devscripts-checkbashisms

How should I proceed in order to implement a fix? Is this an internal fix to Rhub, or should I submit a PR to 'https://github.com/r-hub/sysreqsdb' for 'checkbashisms.json' and add 'checkbashisms' to my R package DESCRIPTION system requirements?

gothub avatar Sep 29 '20 21:09 gothub

@gaborcsardi related PR https://github.com/r-hub/rhub-linux-builders/pull/47

maelle avatar Sep 30 '20 04:09 maelle

checkbashisms script seems to be present on most machines now.

Only for Platform: | Fedora Linux, R-devel, clang, gfortran I encountered the error today.

Platform: | Fedora Linux, R-devel, clang, gfortran

WARNINGS:* checking top-level files ... WARNING A complete check needs the 'checkbashisms' script. See section ‘Configure and cleanup’ in the ‘Writing R Extensions’ manual.

SteffenMoritz avatar Jan 18 '21 22:01 SteffenMoritz

@SteffenMoritz @maelle thanks!

gothub avatar Jan 20 '21 17:01 gothub

The problem happens also on Oracle Solaris 10, x86, 32 bit, R-release and Oracle Solaris 10, x86, 32 bit, R release, Oracle Developer Studio 12.6.

MatthieuStigler avatar Jul 27 '21 14:07 MatthieuStigler

@MatthieuStigler AFAIK there is no such script for Solaris, so just ignore that failure. Or do not run that check on Solaris at all.

gaborcsardi avatar Jul 27 '21 15:07 gaborcsardi

@gaborcsardi Thanks for the quick answer! I still need to run solaris as it is one of the platforms monitored on CRAN (and was the only platform creating a problem), but will ignore that warning in the future!

Thanks for the great work with rhub, very useful!

MatthieuStigler avatar Jul 27 '21 16:07 MatthieuStigler

What I meant is to skip that check when checking on Solaris. I.e. do not use check_for_cran() or --as-cran, CRAN does not use it, either, on Solaris. Or yeah, just ignore that result.

gaborcsardi avatar Jul 27 '21 18:07 gaborcsardi

Oh I see! So I am using it from devtools::check_rhub(), should I then pass check_args to it, as in devtools::check_rhub(..., check_args="")? Thanks!

MatthieuStigler avatar Jul 27 '21 19:07 MatthieuStigler

This issue is about the previous R-hub system, it does not apply to the new system, so I am closing it now. Please see https://r-hub.github.io/rhub/ for the new system, R-hub v2.

gaborcsardi avatar May 09 '24 12:05 gaborcsardi