<div dir="auto"><div>hi,<br><br><div class="gmail_quote"><div dir="ltr">Le ven. 1 févr. 2019 à 20:39, Andrew C Aitchison <<a href="mailto:andrew@aitchison.me.uk">andrew@aitchison.me.uk</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, 1 Feb 2019, Even Rouault wrote:<br>
<br>
> (Re-adding the list)<br>
><br>
>>> Is the split in 2 directories for aesthetic reasons, or because it makes<br>
>>> the<br>
>>> conditional compiling logic easier ?<br>
>><br>
>> The latter. I'd like to avoid modifying the cpp files themselves to add<br>
>> ifdefs to them.<br>
<br>
Most of the drivers I have looked at have a suitable ifdef,<br>
so I checked and was surprised to see that most do not<br>
(a rough grep suggests that 28 frmts directories have such an ifdef and 93 <br>
do not).<br>
<br>
The pull-request talks about<br>
./configure<br>
        ... \<br>
        disable-driver-nnnn \<br>
        ...<br>
<br>
When using this proposal to generate minimal binaries, would<br>
it not be more natural to name the drivers that you wish to *enable*<br>
and have the configure system add whatever dependencies are required ?<br>
When a new driver ws added, I would like to assume that my minimal<br>
build would ignore it, without having to alter my build command.<br></blockquote></div></div><div dir="auto">I'm not sure you were looking at the latest version of the pull request. in your case, you'd use</div><div dir="auto">./configure --disable-all-drivers --enable-driver-nnnn</div><div dir="auto">and be assured that only the nnnn driver and the mandatory ones are included.</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I'm not sure about making "virt" one of the few required drivers;<br>
it would be hard to determine which virt files would be supported<br>
since a single target could use any of the other drivers.<br></blockquote></div></div><div dir="auto">that is already the case actually, although it currently only concerns drivers that would have been added by a --with-xxxx.</div><div dir="auto">I do believe it is useful to be able to use vrts along with the subset of drivers you have chosen to enable.</div><div dir="auto"><br></div><div dir="auto">Note that the target of this PR is not to generate a generic gdal build, but rather to generate a minimal build that contains only the formats you know will be needed, typically when building a custom docker image.</div><div dir="auto"><br></div><div dir="auto">best regards,</div><div dir="auto"> thomas</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
</blockquote></div></div></div>