[R-pkg-devel] R CMD check not finding my vignettes
David Hugh-Jones
davidhughjones at gmail.com
Thu Apr 20 15:58:03 CEST 2017
Hi guys,
Thanks very much for all of your comments. I now have a good sense of what
the possibilities are, and I will think about what works best for my
package.
Cheers,
David
On 20 April 2017 at 14:42, Duncan Murdoch <murdoch.duncan at gmail.com> wrote:
> On 20/04/2017 4:57 AM, Brian G. Peterson wrote:
>
>> David,
>>
>> I'd suggest creating a vignette for each of HTML and PDF, and including
>> a source file that contains the common code. e.g. have a pdf header and
>> an html header file, and then include the 'main' Rmd as a child doc from
>> each header Rmd. This way R CMD build could build both pdf and html
>> versions of the output.
>>
>
> That's another solution. My suggestion would be simpler: just pick one
> of PDF or HTML, and give the user instructions to produce the other if
> necessary, but don't produce it for them.
>
> Duncan Murdoch
>
>
>
>> Regards,
>>
>> Brian
>>
>> On 04/20/2017 03:38 AM, David Hugh-Jones wrote:
>>
>>> Hi Duncan,
>>>
>>> Thank you very much for taking the time to look at this.
>>>
>>> I tried rebuilding the tar file so as to include only the .Rmd files, not
>>> the HTML files, in 'vignettes':
>>>
>>> drwxr-xr-x 0 david staff 0 20 Apr 09:21 huxtable/vignettes/
>>>
>>> -rw-r--r-- 0 david staff 1633 6 Apr 16:26
>>> huxtable/vignettes/comparison.csv
>>>
>>> -rw-r--r-- 0 david staff 6697 6 Apr 14:44
>>> huxtable/vignettes/design-principles.Rmd
>>>
>>> -rw-r--r-- 0 david staff 5521 6 Apr 14:30
>>> huxtable/vignettes/huxreg.Rmd
>>>
>>> -rw-r--r-- 0 david staff 20552 6 Apr 16:19
>>> huxtable/vignettes/huxtable.Rmd
>>>
>>> -rw-r--r-- 0 david staff 22 17 Mar 00:19
>>> huxtable/vignettes/placeins-header.tex
>>>
>>>
>>> But when I run R CMD check, I still get the same warning:
>>>
>>> * checking package vignettes in ‘inst/doc’ ... WARNING
>>>
>>> Package vignettes without corresponding PDF/HTML:
>>>
>>> ‘design-principles.Rmd’
>>>
>>> ‘huxreg.Rmd’
>>>
>>> ‘huxtable.Rmd’
>>>
>>> So, the warning does not seem to be related to the presence of HTML files
>>> in vignettes.
>>>
>>> I also tried manually removing .Rmd files from inst/doc (leaving them
>>> only
>>> in vignettes) but this still gave the same error.
>>>
>>> My goal here is that for package-specific reasons, I would like both pdf
>>> and HTML versions of my vignettes to be available (I am writing code that
>>> prints HTML and LaTeX tables and want my users to have examples of how
>>> both
>>> output formats work). This is why I build those files manually, and place
>>> them in inst/doc.
>>>
>>> Side comment: at the moment, I feel as if I am running through the
>>> combinatorics of including and excluding files from vignettes and
>>> inst/doc,
>>> without much insight into what I am doing. Would it be fair to say that
>>> the
>>> current system is not very easy to comprehend?
>>>
>>> Cheers,
>>>
>>> David
>>>
>>> On 19 April 2017 at 21:24, Duncan Murdoch <murdoch.duncan at gmail.com>
>>> wrote:
>>>
>>> On 19/04/2017 1:00 PM, David Hugh-Jones wrote:
>>>>
>>>> Hi Uwe,
>>>>>
>>>>> I'm not sure if you ever got my off-list message with my tarball or
>>>>> subsequent messages. I can't send a tarball on-list - it gets rejected
>>>>> as
>>>>> too large - but here is a dropbox link. If you could confirm receipt,
>>>>> that
>>>>> would be extremely helpful!
>>>>>
>>>>> https://www.dropbox.com/s/179jrm19kx9o7dz/huxtable_0.2.0.tar.gz?dl=0
>>>>>
>>>>>
>>>>> Not sure if Uwe has had a chance to look, but I just ran R CMD check on
>>>> your tarball using the latest version of R-devel. It reported the
>>>> following problems:
>>>>
>>>> * checking CRAN incoming feasibility ... NOTE
>>>> Maintainer: ‘David Hugh-Jones <davidhughjones at gmail.com>’
>>>>
>>>> Found the following (possibly) invalid URLs:
>>>> URL: http://cran.rstudio.com/web/packages/huxtable/index.html
>>>> From: README.md
>>>> CRAN URL not in canonical form
>>>> Canonical CRAN.R-project.org URLs use https.
>>>>
>>>> The URL for your package should be in the form
>>>>
>>>> https://CRAN.R-project.org/package=huxtable
>>>>
>>>> * checking top-level files ... NOTE
>>>> Non-standard file/directory found at top level:
>>>> ‘multirow.rds’
>>>>
>>>> I'm not sure where that file belongs, but not there.
>>>>
>>>> * checking files in ‘vignettes’ ... WARNING
>>>> Files in the 'vignettes' directory newer than all files in 'inst/doc':
>>>> ‘huxreg.html’, ‘huxtable.html’
>>>>
>>>> Those files shouldn't be in the vignettes directory, they are products
>>>> of
>>>> building the vignettes. Only the source should normally be in the
>>>> vignettes directory.
>>>>
>>>> * checking package vignettes in ‘inst/doc’ ... WARNING
>>>> Package vignettes without corresponding PDF/HTML:
>>>> ‘design-principles.Rmd’
>>>> ‘huxreg.Rmd’
>>>> ‘huxtable.Rmd’
>>>>
>>>> I think this is the warning you were asking about. If I look at just
>>>> the
>>>> first of those files, I see that design-principles.Rmd exists in
>>>> vignettes
>>>> and is set to produce PDF output (only the first output entry in the
>>>> YAML
>>>> counts). You also have a .html file in vignettes; it shouldn't be
>>>> there.
>>>> In inst/doc, you have source, .html and .pdf versions of that file.
>>>>
>>>> The warning isn't very helpful, but I think it is triggered by the fact
>>>> that you've got the .html file in your vignettes directory. If I remove
>>>> everything but the source from that directory, and everything from the
>>>> inst/doc directory, then all the vignette warnings go away.
>>>>
>>>> (I haven't traced through the code, but I think the warning may be
>>>> literally correct. Since you had .html files that looked like vignette
>>>> outputs but weren't, you have vignettes without *corresponding* HTML. It
>>>> would have been nicer if it suggested how to fix this, but that vignette
>>>> code is quite tricky, because any file could be source, and any html,
>>>> tex
>>>> or pdf file could be output. At some point I may try to clean it up a
>>>> bit
>>>> and then maybe the error messages will be less obscure.)
>>>>
>>>> Duncan Murdoch
>>>>
>>>>
>>> [[alternative HTML version deleted]]
>>>
>>> ______________________________________________
>>> R-package-devel at r-project.org mailing list
>>> https://stat.ethz.ch/mailman/listinfo/r-package-devel
>>>
>>>
>>
>>
> ______________________________________________
> R-package-devel at r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/r-package-devel
>
[[alternative HTML version deleted]]
More information about the R-package-devel
mailing list