[icedtea-web] itweb-settings launcher does not accept -J switch to pass args to JVM

[Jiri Vanek]

On 05/15/2013 10:41 AM, Jiri Vanek wrote:
> On 05/15/2013 01:52 AM, Jacob Wisor wrote:
>> "Adam Domurad"<adomurad at xxxxxxxxxx> wrote:
>>> On 05/14/2013 04:11 PM, Jacob Wisor wrote:
>>>> Hello there!
>>>>
>>>> The subject says it all. Is this the desired behavior? The javaws launcher does honor the -J switch.
>>>
>>> What sort of parameters could you possibly require ? It's a very simple
>>> application.
>>> If you're doing something this oddball it isn't too much of a stretch
>>> that you edit the shell script itself.
>>
>> Well, for example setting system properties with the -D switch is a common practice or some specific JVM switches that may be required on some configurations. Sure, administrators could edit the launch script, but it has become quite complex by now and administrators would need to waste time on adding this functionaliy while it is virtually identical to javaws launcher's in that respect. Would not it be nice, if life were easy and tools simply stuck to the "convention" to help users get thier jobs done? ;-) Although the -J switch is not required by spec, it probably would be good to interpret it as a convention.
>>
>>> As for javaws having it, I believe this is part of the spec.
>>
>> That is true. Nevertheless, itweb-settings is a proxy tool and as for such, it would not hurt or maybe even help users to have that functionality provided.
>>
>> All I am asking is, whether it would make sense to invest the time and effort to get it done, so that it is probably going to finds its way into the repository, or whether there are any objections.
>>
>> Regards,
>> Jacob
>>
> 		
> What about this suggestion - generate both itw and javaws launchers from one template? The only
> difference is then main class... And itw-settings will just benefit form this...

Eg like attached patch?


J.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: unifiedLaunchers.diff
Type: text/x-patch
Size: 7760 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20130517/8890e1d0/unifiedLaunchers.diff