[icedtea-web] RFC: add support for validating the deployment configuration

Deepak Bhole dbhole at redhat.com
Thu Dec 9 12:43:39 PST 2010


* Omair Majid <omajid at redhat.com> [2010-12-09 12:12]:
> Hi,
> 
> The attached patch adds support for validating the deployment
> configuration. This patch does not add any GUI or command line tools
> to do it manually (that will be addressed later).
> 
> What this patch does is reject invalid values in all deployment
> configuration files and use the default ones. A
> deployment.properties file like:
> deployment.debug=foobar
> deployment.user.cachedir=file:///localhost/
> foo.bar=foo
> 
> will make javaws detect the problems and use the default values instead:
> 
> Unknown setting "deployment.debug"
> Unknown setting "foo.bar"
> Error: setting "deployment.user.cachedir" has incorrect value
> "file:///localhost/". Using the default value
> "/home/omajid/.icedtea/cache" instead. Possible values include the
> absolute location of a file - it must begin with a /.
> 

Looks okay to me. Not sure if we should be using a 'default' value
instead though.. what does the reference impl. do?

Deepak



More information about the distro-pkg-dev mailing list