[R-pkg-devel] can't reproduce 'Additional issues' on CRAN with valgrind

2021-08-02 Thread Georgi Boshnakov
An update to my package rbibutils triggered 'Additional issues' on CRAN from valgrind, CRAN Package Check Results for Package rbibutils (r-project.org). However, running the checks with -use-valgrind on my machine gives zero

Re: [R-pkg-devel] I changed my vignette's file name to lowercase, then realized the url was case-sensitive

2021-08-02 Thread David Kepplinger
According to https://www.w3.org/TR/WD-html40-970708/htmlweb.html, "URLs in general are case-sensitive", it's not only a CRAN thing. Your vignette will be shipped with the package, hence if a user has a case-insensitive file system (like macOS has by default), your workaround would not work (see

Re: [R-pkg-devel] can't reproduce 'Additional issues' on CRAN with valgrind

2021-08-02 Thread Bill Dunlap
I ran the tests of rbibutils-2.2.2, using the latest devel version of R configured to use valgrind, with R --debugger=valgrind --debugger-args=--track-origins=yes --quiet -e 'testthat::test_package("rbibutils")' I saw a lot of 'conditional jump depends on uninitialized value' errors: ==27280==

[R-pkg-devel] I changed my vignette's file name to lowercase, then realized the url was case-sensitive

2021-08-02 Thread Dominic Comtois
I changed my "Introduction.html" vignette's name to "introduction.html", realizing only after the fact that CRAN's URLs are case sensitive. Would the solution of adding to my package's source a new Introduction.html file pointing to introduction.html using a be a viable one? Or is there maybe

Re: [R-pkg-devel] can't reproduce 'Additional issues' on CRAN with valgrind

2021-08-02 Thread Ivan Krylov
On Mon, 2 Aug 2021 17:11:14 + Georgi Boshnakov wrote: > What am I missing? Do I need to have a separate library for the > instrumented R-devel version? It doesn't look to me like you're missing anything, unless you start seeing the errors when you start `R -d valgrind` and then run the

Re: [R-pkg-devel] can't reproduce 'Additional issues' on CRAN with valgrind

2021-08-02 Thread Georgi Boshnakov
Thanks for looking into this. I probably didn’t make it clear that I am not questioning the errors on CRAN but my own configuration. I have pretty good idea where the error comes from, since the only change from the previous CRAN version was in medin.c (function medin_readf from line 94 and

Re: [R-pkg-devel] I changed my vignette's file name to lowercase, then realized the url was case-sensitive

2021-08-02 Thread Dominic Comtois
My question was in relation to the fact that my package has been around for a while using this link: https://cran.r-project.org/package=summarytools/vignettes/Introduction.html which is now broken, since my last update (uploaded to CRAN a few days ago) uses