This is a patch I would have just committed in normal times.
Contents:
- use a null device to avoid the extra Rplots.pdf file created by Sweave
- use directly setwd() instead of in the preamble of the module
- add lots of comments.
OK?
JMarc
Index: lib/scripts/lyxsweave.R
===================================================================
--- lib/scripts/lyxsweave.R (révision 38347)
+++ lib/scripts/lyxsweave.R (copie de travail)
@@ -1,3 +1,11 @@
+# file lyxsweave.R
+# This file is part of LyX, the document processor.
+# Licence details can be found in the file COPYING.
+
+# author Jean-Marc Lasgouttes
+
+# Full author contact details are available in file CREDITS
+
# Wrapper around Sweave that sets up some things for LyX
# argument 1 is the absolute name of the input file
# argument 2 is the absolute name of the output file
@@ -4,23 +12,30 @@
# argument 3 is the iconv name for the encoding of the file
# argument 4 is the original document directory
-ls.args <- commandArgs(trailingOnly=T)
+ls.args <- commandArgs(trailingOnly=TRUE)
-# check whether Sweave.sty is seen by LaTeX
-ls.sweavesty <- system("kpsewhich Sweave.sty", intern=T, ignore.stderr=T)
+# check whether Sweave.sty is seen by LaTeX. if it is not, we will
+# pass the option stylepath=TRUE to sweave so that a full path is given
+# to \usepackage.
+ls.sweavesty <- system("kpsewhich Sweave.sty", intern=TRUE, ignore.stderr=TRUE)
ls.sp <- (length(ls.sweavesty) == 0)
-# set default encoding to the one of the file; it will be reset to previous
-# default by the sweave module
+# set default encoding for input and output files; ls.enc is used in
+# the sweave module preamble to reset the encoding to what it was.
ls.enc <- getOption("encoding")
options(encoding=ls.args[3])
-# pass document dir to sweave module
-ls.dir <- ls.args[4]
+# Change current directory to the document directory, so that R can find
+# data files.
+setwd(ls.args[4])
-# this is used to tell where temporary files should go
+# this is passed as a prefix.string to tell where temporary files should go
ls.pr <- sub("\\.tex$", "", ls.args[2])
+# Replace the default pdf device by the null device (tip from Yihui Xie)
+# See: http://yihui.name/en/2010/12/a-special-graphics-device-in-r-the-null-device/
+.Call("R_GD_nullDevice", PACKAGE = "grDevices")
+
# finally run sweave
Sweave(file=ls.args[1], output=ls.args[2], syntax="SweaveSyntaxNoweb", stylepath=ls.sp, prefix.string=ls.pr)
Index: lib/layouts/sweave.module
===================================================================
--- lib/layouts/sweave.module (révision 38347)
+++ lib/layouts/sweave.module (copie de travail)
@@ -21,7 +21,6 @@
Preamble
<<echo=F>>=
- if(exists("ls.dir")) setwd(ls.dir)
if(exists("ls.enc")) options(encoding=ls.enc)
@
EndPreamble
