[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