[Rd] mcparallel (parallel:::mcexit) does not call finalizers

2016-06-16 Thread Karl Forner
Hello, In the context of trying to cover a package code that use parallelized tests using the covr package, I realized that code executed using mcparallel() was not covered, cf https://github.com/jimhester/covr/issues/189#issuecomment-226492623 >From my understanding, it seems that the package fi

Re: [Rd] new function to tools/utils package: dependencies based on DESCRIPTION file

2016-06-16 Thread Michael Lawrence
I agree that the utils package needs some improvements related to this, and hope to make them eventually. This type of feedback is very helpful. Thanks, Michael On Thu, Jun 16, 2016 at 1:42 AM, Jan Górecki wrote: > Dear Joris, > > So it does looks like the proposed function makes a lot sense t

Re: [Rd] new function to tools/utils package: dependencies based on DESCRIPTION file

2016-06-16 Thread Jan Górecki
Dear Joris, So it does looks like the proposed function makes a lot sense then, isn't it? Cheers, Jan On 16 June 2016 at 08:37, Joris Meys wrote: > Dear Jan, > > It is unavoidable to have OS and R dependencies for devtools. The building > process for packages is both OS and R dependent, so devt

Re: [Rd] new function to tools/utils package: dependencies based on DESCRIPTION file

2016-06-16 Thread Gábor Csárdi
FWIW, the `desc` package can extract dependencies from a local DESCRIPTION, and it is lightweight. It only depends on small, R-only packages (R6 and crayon). ❯ desc::desc_get_deps(".") type package version 1 Imports DBI * 2 Imports markdown * 3 Imports rhandso

Re: [Rd] new function to tools/utils package: dependencies based on DESCRIPTION file

2016-06-16 Thread Joris Meys
Dear Jan, It is unavoidable to have OS and R dependencies for devtools. The building process for packages is both OS and R dependent, so devtools has to be too according to my understanding. Cheers Joris On 14 Jun 2016 18:56, "Jan Górecki" wrote: Hi Thierry, I'm perfectly aware of it. Any idea