[rfc] added preprocessing to rust launchers

Jiri Vanek jvanek at redhat.com
Fri Dec 8 10:21:53 UTC 2017


On 12/06/2017 07:46 PM, Alex Kashchenko wrote:
> On 12/06/2017 02:50 PM, Jiri Vanek wrote:
>> Hello!
>>
>> Thanx for feedback. I had week long personal battle whether to have it preprocessed with some 
>> solid defaults, or to use config file. It ehnded 51:49 for preprocess. No it is 99:1 for config 
>> file:)
>>
>> What location do you suggest?
> 
> The same directory, where launcher binary is, will be the most convenient. So it can be read with: 
> executable_directory() + "/conf_file_name".

But that is no way with linux fs conventions for distribution packages.

So the fall back will need to stay in place

1)xdg_confidir/icedtea-web/conf_file_name
2)executable_directory() + :/conf_file_name".
Anything more?

where conf_file_name I would vote for .itw_default_paths

And the file should be read only :)

Thanx for thoughts!
   J.
> 
>>
>>
>> On 12/06/2017 01:12 PM, Alex Kashchenko wrote:
>>> Hi,
>>>
>>> On 12/05/2017 04:11 PM, Jiri Vanek wrote:
>>>> This patch is adding the possiblity to pas classapth and simialr to the rust luncher.
>>>> If there is enybody knowing the better way, and especially how to get rid of this duplicated 
>>>> (triplicated x triplicated) snipept of
>>>> +    MAIN_CLASS=net.sourceforge.jnlp.runtime.Boot ;\
>>>> +    BIN_LOCATION=$(bindir)/$(javaws).sh ;\
>>>> +    PROGRAM_NAME=$(javaws) ;\
>>>> +    $(edit_launcher_script) < $< > $@
>>>>
>>>> And turn it into function of call(name, suffix)  < $< > $@, that would be awesome. I tried many 
>>>> times, various ways, but if I get rid of those three lines, I always duplicated  targets:(
>>>
>>> I can suggest not to pre-process .rs file, but to preprocess config file instead and read it on 
>>> startup.
>>>
>>>
>>
>>
> 
> 


-- 
Jiri Vanek
Senior QE engineer, OpenJDK QE lead, Mgr.
Red Hat Czech
jvanek at redhat.com    M: +420775390109


More information about the distro-pkg-dev mailing list