jhallman at frb.gov
2006-Nov-28 23:40 UTC
[Rd] Bug in tkStartGUI in package tcltk (PR#9396)
Full_Name: Jeff Hallman
Version: 2.4.0
OS: Linux
Submission from: (NULL) (132.200.32.34)
When the R Internals manual was added to the list of manuals in tkStartGUI,
whoever did it left an extra comma in the code. This creates an ugly warning
when tkStartGUI is invoked. Here is a patch file to fix it:
--- src/library/tcltk/R/tkGUI.R.orig 2006-11-21 15:15:52.000000000 -0500
+++ src/library/tcltk/R/tkGUI.R 2006-11-21 15:16:40.000000000 -0500
@@ -93,7 +93,7 @@
"R-data", "R Data Import/Export",
"R-exts", "Writing R extensions",
"R-lang", "R Language Reference",
- "refman", "R Reference Manual",
+ "refman", "R Reference Manual"
), ncol=2, byrow=TRUE)
helpPDFMenu <- tkmenu(helpMenu)
jhallman at frb.gov wrote:> Full_Name: Jeff Hallman > Version: 2.4.0 > OS: Linux > Submission from: (NULL) (132.200.32.34) > > > When the R Internals manual was added to the list of manuals in tkStartGUI, > whoever did it left an extra comma in the code. This creates an ugly warning > when tkStartGUI is invoked. Here is a patch file to fix it: > > --- src/library/tcltk/R/tkGUI.R.orig 2006-11-21 15:15:52.000000000 -0500 > +++ src/library/tcltk/R/tkGUI.R 2006-11-21 15:16:40.000000000 -0500 > @@ -93,7 +93,7 @@ > "R-data", "R Data Import/Export", > "R-exts", "Writing R extensions", > "R-lang", "R Language Reference", > - "refman", "R Reference Manual", > + "refman", "R Reference Manual" > ), ncol=2, byrow=TRUE) > > helpPDFMenu <- tkmenu(helpMenu) >Thanks. I think the extra comma has been there for a while, but c() has become more picky about missing args. (Another issue is that the actual manuals go walkabout in some binary distributions. E.g. FC6 has them in /usr/share/doc/R-2.4.0/ but the GUI looks for them in file.path(R.home("doc"), "manual"), which only has the HTML versions)