[RFC][Icedtea-web]: validate cache location in itweb-settings
Omair Majid
omajid at redhat.com
Mon May 30 10:52:35 PDT 2011
Hi Andrew,
On 05/30/2011 01:23 PM, Andrew Su wrote:
> This patch will add validation when the user sets the cache directory
> location.
>
> It is not necessary to apply this patch, since the user can just as
> easily remove permissions after setting it, or enable permission
> after selecting a path without permission.
>
> Comments?
The patch looks fine to me. IMHO, it should be applied. It will help a
novice user avoid selecting a wrong directory for caching (which would
result in mysterious failures when running jnlp applications). An expert
user, on the other hand, can use the command line interface of
itweb-settings or his favourite text editor to change the settings in
the deployment.properties without running into this check.
Cheers,
Omair
More information about the distro-pkg-dev
mailing list