>> @@ -32,6 +33,10 @@ post-patch {
>>
>> use_configure no
>>
>> +if {[variant_isset ecm]} {
>> + build.args ECM=1
>> +}
>> +
>
> Why did you put this code here in a block by itself? Why not set (or better
> yet: append to) build.args inside the emc variant declaration, so that all
> the code relating to the variant is in one place?
>
>> build.env CFLAGS="${configure.cflags}" \
>> CPATH="${compiler.cpath}" \
>> LIBRARY_PATH="${compiler.library_path}" \
>> @@ -53,6 +58,10 @@ destroot {
>> ${destroot}${prefix}/share/${name}/demo
>> }
>>
>> +variant ecm description "Use elliptic curve method for factorisation" {
>> + depends_lib-append port:gmp-ecm
>> +}
>
You’re right, I forgot to finish to clean up after testing during development.
Fixed in ec35a394345e0f5ce04ba6d6167ef13870437f0b.
Cheers,
Nicolas
