[rfc][icedtea-web] moving rust launchers to cargo

Alex Kashchenko akashche at redhat.com
Fri Mar 16 21:19:21 UTC 2018


Hi Jiri,

On 03/16/2018 04:56 PM, Jiri Vanek wrote:
>
> [...]
>
>>
>> Hello!
>>
>> One more attempt to make default build pass, and still protect the 
>> launcher from being built without vars set.
>>
>> Using the original approach (no build.rs), havinf  option_env! to 
>> private fields, and having unwrapping gettets in hardcoded.rs.  With 
>> difference that the getter will return default instead of failing.
>>
>>
>> In addition I would keep the test, ensuring that the necessary values 
>> are not empty nor default.
>> By that, the default build you require will be kept, and  my fear that 
>> default-vars build will leak removed.
>>
>>
> 
> Patch with above approach attached.

Patch looks good to me. Compile-time checks may be better, but runtime 
ones will work too.

Minor problems:

1. redundant mod declaration in hardcoded_paths.rs

2. excessive trims in tests

3. variables_non_em*o*pty typo


-- 
-Alex


More information about the distro-pkg-dev mailing list