Re: [Rd] Small bug in install.packages?

2011-06-29 Thread Hadley Wickham
 R.home(bin)

 [1] C:/Program Files/R/R-2.13.0/bin/i386

 Weird.  Like others, I see 8.3 pathnames.  R gets those from a Windows
 call; what version of Windows are you using?

 ... and how are you starting R?  Startup goes through some contortions to
 handle all the different possibilities; maybe one of them is missing the
 path-shortening step.

Ok, this was coming from R running within eclipse, via rj.  (This is
not my setup, but a collaborator's)

Hadley

-- 
Assistant Professor / Dobelman Family Junior Chair
Department of Statistics / Rice University
http://had.co.nz/

__
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel


[Rd] Small bug in install.packages?

2011-06-28 Thread Hadley Wickham
When capturing the path to the current R binary, install.packages does:

  cmd0 - paste(file.path(R.home(bin), R), CMD INSTALL)

shouldn't that be

  cmd0 - shQuote(paste(file.path(R.home(bin), R), CMD INSTALL))

to allow paths with spaces in them (which would be very common on windows)?

Hadley


-- 
Assistant Professor / Dobelman Family Junior Chair
Department of Statistics / Rice University
http://had.co.nz/

__
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel


Re: [Rd] Small bug in install.packages?

2011-06-28 Thread Simon Urbanek

On Jun 28, 2011, at 3:03 PM, Hadley Wickham wrote:

 When capturing the path to the current R binary, install.packages does:
 
  cmd0 - paste(file.path(R.home(bin), R), CMD INSTALL)
 
 shouldn't that be
 
  cmd0 - shQuote(paste(file.path(R.home(bin), R), CMD INSTALL))
 
 to allow paths with spaces in them (which would be very common on windows)?
 

Isn't R.home() 8.3 path anyway?

BTW: I think you probably meant more something like 
paste(shQuote(file.path(R.home(bin), R)), CMD INSTALL) since the above 
won't work ..

Cheers,
S

__
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel


Re: [Rd] Small bug in install.packages?

2011-06-28 Thread Joshua Ulrich
It is 8.3 on my install of 2.13.0 (release) and I get the same results
with the current R-patched.

 R.home(bin)
[1] C:/PROGRA~1/R/R-213~1.0PA/bin/i386
 sessionInfo()
R version 2.13.0 Patched (2011-06-20 r56188)
Platform: i386-pc-mingw32/i386 (32-bit)

locale:
[1] LC_COLLATE=English_United States.1252
[2] LC_CTYPE=English_United States.1252
[3] LC_MONETARY=English_United States.1252
[4] LC_NUMERIC=C
[5] LC_TIME=English_United States.1252

attached base packages:
[1] stats graphics  grDevices utils datasets  methods   base

--
Joshua Ulrich  |  FOSS Trading: www.fosstrading.com



On Tue, Jun 28, 2011 at 4:42 PM, Hadley Wickham had...@rice.edu wrote:
 Isn't R.home() 8.3 path anyway?

 I don't think so:

 R.home(bin)
 [1] C:/Program Files/R/R-2.13.0/bin/i386
 sessionInfo()
 R version 2.13.0 Patched (2011-06-09 r56106)
 Platform: i386-pc-mingw32/i386 (32-bit)

 locale:
 [1] LC_COLLATE=English_United States.1252
 [2] LC_CTYPE=English_United States.1252
 [3] LC_MONETARY=English_United States.1252
 [4] LC_NUMERIC=C
 [5] LC_TIME=English_United States.1252

 attached base packages:
 [1] grDevices datasets  utils     stats     graphics  methods   base

 other attached packages:
 [1] RMark_2.0.5     rj_0.5.0-5      devtools_0.2    codetools_0.2-8
 [5] plotrix_3.2-2   nlme_3.1-101    msm_1.0.1

 loaded via a namespace (and not attached):
 [1] grid_2.13.0      lattice_0.19-26  mvtnorm_0.9-9991 rJava_0.8-8
 [5] splines_2.13.0   survival_2.36-9  tools_2.13.0

 BTW: I think you probably meant more something like 
 paste(shQuote(file.path(R.home(bin), R)), CMD INSTALL) since the above 
 won't work ..

 Ooops, yes.

 Hadley


 --
 Assistant Professor / Dobelman Family Junior Chair
 Department of Statistics / Rice University
 http://had.co.nz/

 __
 R-devel@r-project.org mailing list
 https://stat.ethz.ch/mailman/listinfo/r-devel


__
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel


Re: [Rd] Small bug in install.packages?

2011-06-28 Thread Duncan Murdoch

On 28/06/2011 5:42 PM, Hadley Wickham wrote:

Isn't R.home() 8.3 path anyway?


I don't think so:


R.home(bin)

[1] C:/Program Files/R/R-2.13.0/bin/i386


Weird.  Like others, I see 8.3 pathnames.  R gets those from a Windows 
call; what version of Windows are you using?


Duncan Murdoch


sessionInfo()

R version 2.13.0 Patched (2011-06-09 r56106)
Platform: i386-pc-mingw32/i386 (32-bit)

locale:
[1] LC_COLLATE=English_United States.1252
[2] LC_CTYPE=English_United States.1252
[3] LC_MONETARY=English_United States.1252
[4] LC_NUMERIC=C
[5] LC_TIME=English_United States.1252

attached base packages:
[1] grDevices datasets  utils stats graphics  methods   base

other attached packages:
[1] RMark_2.0.5 rj_0.5.0-5  devtools_0.2codetools_0.2-8
[5] plotrix_3.2-2   nlme_3.1-101msm_1.0.1

loaded via a namespace (and not attached):
[1] grid_2.13.0  lattice_0.19-26  mvtnorm_0.9-9991 rJava_0.8-8
[5] splines_2.13.0   survival_2.36-9  tools_2.13.0


BTW: I think you probably meant more something like paste(shQuote(file.path(R.home(bin), 
R)), CMD INSTALL) since the above won't work ..


Ooops, yes.

Hadley




__
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel


Re: [Rd] Small bug in install.packages?

2011-06-28 Thread Duncan Murdoch

On 28/06/2011 8:17 PM, Duncan Murdoch wrote:

On 28/06/2011 5:42 PM, Hadley Wickham wrote:

Isn't R.home() 8.3 path anyway?


I don't think so:


R.home(bin)

[1] C:/Program Files/R/R-2.13.0/bin/i386


Weird.  Like others, I see 8.3 pathnames.  R gets those from a Windows
call; what version of Windows are you using?


... and how are you starting R?  Startup goes through some contortions 
to handle all the different possibilities; maybe one of them is missing 
the path-shortening step.


Duncan Murdoch



Duncan Murdoch


sessionInfo()

R version 2.13.0 Patched (2011-06-09 r56106)
Platform: i386-pc-mingw32/i386 (32-bit)

locale:
[1] LC_COLLATE=English_United States.1252
[2] LC_CTYPE=English_United States.1252
[3] LC_MONETARY=English_United States.1252
[4] LC_NUMERIC=C
[5] LC_TIME=English_United States.1252

attached base packages:
[1] grDevices datasets  utils stats graphics  methods   base

other attached packages:
[1] RMark_2.0.5 rj_0.5.0-5  devtools_0.2codetools_0.2-8
[5] plotrix_3.2-2   nlme_3.1-101msm_1.0.1

loaded via a namespace (and not attached):
[1] grid_2.13.0  lattice_0.19-26  mvtnorm_0.9-9991 rJava_0.8-8
[5] splines_2.13.0   survival_2.36-9  tools_2.13.0


BTW: I think you probably meant more something like paste(shQuote(file.path(R.home(bin), 
R)), CMD INSTALL) since the above won't work ..


Ooops, yes.

Hadley






__
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel