[R-pkg-devel] windows and configure scripts

2021-08-06 Thread Tim Keitt
As configure is not run on windows, but according to the writing extensions docs you can run commands in Makevars, is that not preferable for simple cases? What shell is active in Makevars? Is it "/bin/sh" or is make the interpreter here? Can we use $(shell ...cmd...) in Makevars on windows? I'm

Re: [R-pkg-devel] Ongoing CXX17 woes

2021-08-06 Thread Tim Keitt
Fixed. Thanks. THK On Fri, Aug 6, 2021 at 4:33 PM Tim Keitt wrote: > > > On Fri, Aug 6, 2021 at 4:22 PM Tim Keitt wrote: > >> >> >> On Fri, Aug 6, 2021 at 4:05 PM Ivan Krylov wrote: >> >>> On Fri, 6 Aug 2021 15:51:37 -0500 >>> Tim Kei

Re: [R-pkg-devel] Ongoing CXX17 woes

2021-08-06 Thread Tim Keitt
On Fri, Aug 6, 2021 at 4:22 PM Tim Keitt wrote: > > > On Fri, Aug 6, 2021 at 4:05 PM Ivan Krylov wrote: > >> On Fri, 6 Aug 2021 15:51:37 -0500 >> Tim Keitt wrote: >> >> > CXX17=`${R_HOME}/bin/R CMD config --all | awk '{print $1}' | grep \ >&g

Re: [R-pkg-devel] Ongoing CXX17 woes

2021-08-06 Thread Tim Keitt
On Fri, Aug 6, 2021 at 4:05 PM Ivan Krylov wrote: > On Fri, 6 Aug 2021 15:51:37 -0500 > Tim Keitt wrote: > > > CXX17=`${R_HOME}/bin/R CMD config --all | awk '{print $1}' | grep \ > > "^CXX17$"` > > This seems to output "CXX17" on a ma

[R-pkg-devel] Ongoing CXX17 woes

2021-08-06 Thread Tim Keitt
Hi Folks, I have a package that compiles fine under C++11 but it throws an "Error: C++17 standard requested but CXX17 is not defined" error when checking under Solaris. My configure looks like: #!/bin/sh # Borrowed from https://github.com/eddelbuettel/rcppsimdjson/blob/master/configure : ${R_H

Re: [R-pkg-devel] C++17

2021-07-30 Thread Tim Keitt
Kind of figured that out looking around on github. I added a config (borrowed from one of your repos) to test for CXX17. I'm not sure its quite working for me yet but I will figure it out. A couple of questions: 1) I had C++17 in the system requirements in the description file but might remove it

[R-pkg-devel] C++17

2021-07-26 Thread Tim Keitt
Hi Folks, It appears that c++17 is not supported on all platforms used by CRAN. Is there a workaround? I have CXX_STD = CXX17 in the package Makevars file, but see https://cran.r-project.org/web/checks/check_results_kdtools.html I assume if CXX17 is not defined, it is not available on that build

Re: [R-pkg-devel] import 'as' from another package

2020-07-20 Thread Tim Keitt
classic confusing R message. Try adding methods > >to the Imports: statement in your DESCRIPTION file and see if that > >helps. (Maybe I should file a bug report about that error message - it > >confuses me every time.) > > > >On 7/20/20 12:34 PM, Tim Keitt wrote: &g

Re: [R-pkg-devel] import 'as' from another package

2020-07-20 Thread Tim Keitt
sage - it confuses me > every time.) > On 7/20/20 12:34 PM, Tim Keitt wrote: > > It works but "check" gives > > > checking package dependencies ... ERROR > Namespace dependency not required: ‘methods’ > > THK > > On Mon, Jul 20, 2020 at 11:2

Re: [R-pkg-devel] import 'as' from another package

2020-07-20 Thread Tim Keitt
It works but "check" gives > checking package dependencies ... ERROR Namespace dependency not required: ‘methods’ THK On Mon, Jul 20, 2020 at 11:24 AM Ben Bolker wrote: > @importFrom methods as > > ? > > On 7/20/20 12:06 PM, Tim Keitt wrote: > >

[R-pkg-devel] import 'as' from another package

2020-07-20 Thread Tim Keitt
I have if (!inherits(x, "RasterLayer")) x <- as(x, "RasterLayer") in a package and its not finding the coerce definition from the raster package. I know I need to add an @import roxygen2 directive of some kind, but I'm not sure the correct syntax. My first try generated a warning that it was no

[R-pkg-devel] Vignette depends on BH

2020-06-02 Thread Tim Keitt
I have an RMarkdown vignette in a package that uses Rcpp and depends on BH. I added BH to the Suggests list. It is still generating warnings about BH. What is the correct way to handle this? Thanks. THK [[alternative HTML version deleted]] __

Re: [R-pkg-devel] Travis/appveyor with CXX14/17

2019-06-07 Thread Tim Keitt
On Fri, Jun 7, 2019 at 11:09 AM Dirk Eddelbuettel wrote: > > On 7 June 2019 at 10:13, Tim Keitt wrote: > | I bumped a package to CXX14/17 and it failed on Travis/Appveyor while > | checking fine locally. I assume I need to request different compilers or > > Correct. > >

[R-pkg-devel] Travis/appveyor with CXX14/17

2019-06-07 Thread Tim Keitt
I bumped a package to CXX14/17 and it failed on Travis/Appveyor while checking fine locally. I assume I need to request different compilers or something like that. Anyone have a quick recipe for that? THK [[alternative HTML version deleted]] __

Re: [R-pkg-devel] Avoid reprocessing Rmd vignette

2018-03-19 Thread Tim Keitt
http://www.keittlab.org/ On Mon, Mar 19, 2018 at 6:41 AM, Dirk Eddelbuettel wrote: > > On 18 March 2018 at 21:57, Tim Keitt wrote: > | That's a great idea, however my problem is with building a vignette, not > | running tests, unless they are linked in some way I&#x

Re: [R-pkg-devel] Avoid reprocessing Rmd vignette

2018-03-19 Thread Tim Keitt
ing desire for an answer does not > ensure that a reasonable answer can be extracted from a given body of data. > ~ John Tukey > > /// > > <https://www.inbo.be> > > 2018-03-19 8:18 GMT+01:00 Iñaki Úcar : > > > 2018-03-19 3:57 GMT+01:00 Tim Keitt : > > > h

Re: [R-pkg-devel] Avoid reprocessing Rmd vignette

2018-03-19 Thread Tim Keitt
http://www.keittlab.org/ On Mon, Mar 19, 2018 at 2:18 AM, Iñaki Úcar wrote: > 2018-03-19 3:57 GMT+01:00 Tim Keitt : > > http://www.keittlab.org/ > > > > On Sun, Mar 18, 2018 at 8:14 PM, Dirk Eddelbuettel > wrote: > > > >> > >> Tim, > >>

Re: [R-pkg-devel] Avoid reprocessing Rmd vignette

2018-03-18 Thread Tim Keitt
http://www.keittlab.org/ On Sun, Mar 18, 2018 at 8:14 PM, Dirk Eddelbuettel wrote: > > Tim, > > On 18 March 2018 at 18:58, Tim Keitt wrote: > | I have an Rmd vignette that runs some benchmarks. It takes long enough > (20+ > | minutes) that eg TravisCI will choke. I'

[R-pkg-devel] Avoid reprocessing Rmd vignette

2018-03-18 Thread Tim Keitt
I have an Rmd vignette that runs some benchmarks. It takes long enough (20+ minutes) that eg TravisCI will choke. I've not tried submitting to CRAN. What is the best practice for handling this situation? Do I generate HTML/PDF output locally and try to make them static? The vignette builder is knit

[R-pkg-devel] (no subject)

2018-01-09 Thread Tim Keitt
I'm seeing: * checking re-building of vignette outputs ... [3s] WARNING > Error in re-building vignettes: > ... > Quitting from lines 24-60 (strider.Rmd) > Error: processing vignette 'strider.Rmd' failed with diagnostics: > package 'dplyr' is not installed for 'arch = x64' > Execution halted I'

Re: [R-pkg-devel] -Wlong-long

2017-10-29 Thread Tim Keitt
http://www.keittlab.org/ On Sun, Oct 29, 2017 at 2:56 PM, Uwe Ligges wrote: > > > On 29.10.2017 19:25, Tim Keitt wrote: > >> Anyone have a fix for: >> >> Found the following significant warnings: >>../windows/libpq-9.5.2/include/pg_config_ext.h:8:2

[R-pkg-devel] -Wlong-long

2017-10-29 Thread Tim Keitt
Anyone have a fix for: Found the following significant warnings: ../windows/libpq-9.5.2/include/pg_config_ext.h:8:28: warning: ISO C++ 1998 does not support 'long long' [-Wlong-long] ? I tried prepending __extension__ to the definition and adding CXX_STD = CXX11 to Makevars. This is external c

Re: [R-pkg-devel] Package Submission: issues with R_registerRoutines', 'R_useDynamicSymbols'

2017-04-13 Thread Tim Keitt
On Thu, Apr 13, 2017 at 10:14 AM, Vineetha Warriyar Kodalore Vijayan < vineethawarriyar@ucalgary.ca> wrote: > Can someone help to resolve this issue? I wrote a short R script that will read your RcppExports.cpp and emit the corresponding init.c file. I call it from my configure script, but i

Re: [R-pkg-devel] AppVeyor with LinkingTo: field

2017-02-13 Thread Tim Keitt
On Mon, Feb 13, 2017 at 10:31 AM, Tim Keitt wrote: > I am almost there -- only failing on devel in Travis CI. It failing at the > "R CMD build" phase. > > * installing *source* package ‘odeintr’ ... >> ** libs >> I/home/travis/R-bin/lib/R/include -DNDEBUG -I&quo

Re: [R-pkg-devel] AppVeyor with LinkingTo: field

2017-02-13 Thread Tim Keitt
On Mon, Feb 13, 2017 at 10:36 AM, Dirk Eddelbuettel wrote: > Not true. > It seems to be a work-around for Travis, but as you say not a general packaging requirement. THK http://www.keittlab.org/ [[alternative HTML version deleted]] __ R-pac

Re: [R-pkg-devel] AppVeyor with LinkingTo: field

2017-02-13 Thread Tim Keitt
On Mon, Feb 13, 2017 at 9:19 AM, Dirk Eddelbuettel wrote: > [1] http://eddelbuettel.github.io/r-travis/ > This looks useful. I'll have to investigate. THK http://www.keittlab.org/ [[alternative HTML version deleted]] __ R-package-devel@r-pr

Re: [R-pkg-devel] AppVeyor with LinkingTo: field

2017-02-13 Thread Tim Keitt
On Mon, Feb 13, 2017 at 9:11 AM, Gábor Csárdi wrote: > You need to put it in 'Imports' as well. Whatever is in LinkingTo, must be > in Imports. > That did the trick. Thanks. I am almost there -- only failing on devel in Travis CI. It failing at the "R CMD build" phase. * installing *source* pa

[R-pkg-devel] AppVeyor with LinkingTo: field

2017-02-13 Thread Tim Keitt
My tests are failing on AppVeyor because BH is not available. How do I force AppVeyor to install the package? It is in the LinkingTo: field, but my tests call sourceCpp and it fails there. The Cpp code has the "depends" directive for BH. It is either not being installed or is absent from the test e

Re: [R-pkg-devel] Force namespace prefix for a loaded package function

2016-06-28 Thread Tim Keitt
http://www.keittlab.org/ On Mon, Jun 27, 2016 at 7:04 PM, Tim Keitt wrote: > > > http://www.keittlab.org/ > > On Mon, Jun 27, 2016 at 5:18 PM, Duncan Murdoch > wrote: > >> On 27/06/2016 5:46 PM, Tim Keitt wrote: >> >>> >>> >>> http:

Re: [R-pkg-devel] Force namespace prefix for a loaded package function

2016-06-28 Thread Tim Keitt
http://www.keittlab.org/ On Mon, Jun 27, 2016 at 5:18 PM, Duncan Murdoch wrote: > On 27/06/2016 5:46 PM, Tim Keitt wrote: > >> >> >> http://www.keittlab.org/ >> >> On Mon, Jun 27, 2016 at 10:19 AM, Duncan Murdoch >> mailto:murdoch.dun...@gmail.com>&

Re: [R-pkg-devel] Force namespace prefix for a loaded package function

2016-06-27 Thread Tim Keitt
http://www.keittlab.org/ On Mon, Jun 27, 2016 at 4:46 PM, Tim Keitt wrote: > > > http://www.keittlab.org/ > > On Mon, Jun 27, 2016 at 10:19 AM, Duncan Murdoch > wrote: > >> On 27/06/2016 11:08 AM, Tim Keitt wrote: >> >>> http://www.keittlab.org/ >

Re: [R-pkg-devel] Force namespace prefix for a loaded package function

2016-06-27 Thread Tim Keitt
http://www.keittlab.org/ On Mon, Jun 27, 2016 at 10:19 AM, Duncan Murdoch wrote: > On 27/06/2016 11:08 AM, Tim Keitt wrote: > >> http://www.keittlab.org/ >> >> On Mon, Jun 27, 2016 at 3:22 AM, Joris Meys wrote: >> >> > If you want to call a non e

Re: [R-pkg-devel] Force namespace prefix for a loaded package function

2016-06-27 Thread Tim Keitt
trying to suggest that it be fixed. (Probably should have sent this to r-devel actually.) THK > Cheers > Joris > On 26 Jun 2016 19:37, "Tim Keitt" wrote: > >> It would be rather nice if we could define functions in our packages that >> must be called w

[R-pkg-devel] Force namespace prefix for a loaded package function

2016-06-26 Thread Tim Keitt
It would be rather nice if we could define functions in our packages that must be called with the namespace prefix. I'd like to do #' @protected (or some such) f = function(...) list(...) in package scope and then library(x) f(1) # fails x::f(1) # succeeds Currently unless

Re: [R-pkg-devel] What to do with build-time-only utility scripts in package

2016-04-15 Thread Tim Keitt
http://www.keittlab.org/ On Fri, Apr 15, 2016 at 11:40 AM, Dirk Eddelbuettel wrote: > > On 15 April 2016 at 10:27, Tim Keitt wrote: > | I created a couple of R scripts to assist my package 'configure' script. > I > | put them in the top-level of the package and

[R-pkg-devel] What to do with build-time-only utility scripts in package

2016-04-15 Thread Tim Keitt
I created a couple of R scripts to assist my package 'configure' script. I put them in the top-level of the package and that gives a check error obviously. So I added them to .Rbuildignore and then the package would not build (at least on Travis CI) saying the script files were not found. That's co

Re: [R-pkg-devel] CRAN and CMake

2015-06-10 Thread Tim Keitt
http://www.keittlab.org/ On Wed, Jun 10, 2015 at 2:21 AM, Martin Maechler wrote: > > Gábor Csárdi > > on Tue, 9 Jun 2015 18:39:47 -0400 writes: > > > On Tue, Jun 9, 2015 at 6:23 PM, Duncan Murdoch < > murdoch.dun...@gmail.com> wrote: > > [...] > >> The DESCRIPTION file s

Re: [R-pkg-devel] CRAN and CMake

2015-06-10 Thread Tim Keitt
http://www.keittlab.org/ On Tue, Jun 9, 2015 at 5:23 PM, Duncan Murdoch wrote: > On 09/06/2015 6:10 PM, Gregory Jefferis wrote: > > > > > >> On 9 Jun 2015, at 22:08, Gábor Csárdi wrote: > >> > >> Thanks! I just looked at it, but it seems that it actually requires an > >> external system cmake.

[R-pkg-devel] CRAN and CMake

2015-06-09 Thread Tim Keitt
I am thinking of writing an R wrapper for some code that uses cmake. If I include that code under the src directory (ie in src//), am I likely to run into problems with CRAN? Never been down that road and would like to know if this is feasible. I guess I would have to call cmake in the configure sc