nealrichardson commented on a change in pull request #9898:
URL: https://github.com/apache/arrow/pull/9898#discussion_r611915984
##########
File path: r/pkgdown/extra.js
##########
@@ -0,0 +1,166 @@
+/**
+ * adapted from rmarkdown/inst/rmd/h/navigation-1.1/tabsets.js
Review comment:
Why do we have to copy this from rmarkdown? Does `rmarkdown` not make it
available? Can we load
https://raw.githubusercontent.com/rstudio/rmarkdown/master/inst/rmd/h/navigation-1.1/tabsets.js
at page load time in the pkgdown site?
I'm not sure we can do this. [`rmarkdown` is GPL-3
license](https://github.com/rstudio/rmarkdown/blob/master/DESCRIPTION#L113),
which is copyleft and on the [forbidden
list](https://www.apache.org/legal/resolved.html#category-x). We can of course
*use* `rmarkdown`, we just can't copy its code inside of apache/arrow.
##########
File path: r/pkgdown/extra.js
##########
@@ -0,0 +1,166 @@
+/**
+ * adapted from rmarkdown/inst/rmd/h/navigation-1.1/tabsets.js
Review comment:
Why do we have to copy this from rmarkdown? Does `rmarkdown` not make it
available? Can we load
https://raw.githubusercontent.com/rstudio/rmarkdown/master/inst/rmd/h/navigation-1.1/tabsets.js
at page load time in the pkgdown site?
I'm not sure we can copy this code here. [`rmarkdown` is GPL-3
license](https://github.com/rstudio/rmarkdown/blob/master/DESCRIPTION#L113),
which is copyleft and on the [forbidden
list](https://www.apache.org/legal/resolved.html#category-x). We can of course
*use* `rmarkdown`, we just can't copy its code inside of apache/arrow.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
