On 14 June 2013 at 16:17, Simon Urbanek wrote: | I think it would be better to have a bit more sane handling of this. The decision is rarely at the install time of the package -- e.g. did you adjust the flag for INSTALL in update.packages based on the permissions? Otherwise it fails the next time and confuses the hell out of users! Although it's certainly a step up from the previously hard-coded default, I would argue that the most desired behavior is to follow the permissions already set up. If the library is setup to be group-writable then the package should be installed group-writable. The point is that anything else makes less sense - the user can still use rm -rf and re-install it even if the package is not writable. It's just update.packages() that fails.
One could also consider respecting the umask which is ignored in the older behaviour, ignore in what was just committed, and ignored in what you suggest here. Not exactly right either. But up until right now I could not update a package a colleague installed, and vice versa -- unless we sudo. The patch improves on this, and the only other comment was a 'thumbs up' from Andreas Leha who described an elaborate workaround scheme required by the existing shortcoming. Dirk -- Dirk Eddelbuettel | e...@debian.org | http://dirk.eddelbuettel.com ______________________________________________ R-devel@r-project.org mailing list https://stat.ethz.ch/mailman/listinfo/r-devel
