<div dir="ltr">Hi Evan,<div>oh, thanks for the typo, fixed in my code ;-)<br></div><div><br>There is driver check in the code<br><br><a href="https://svn.osgeo.org/grass/grass/trunk/vector/v.out.ogr/main.c">https://svn.osgeo.org/grass/grass/trunk/vector/v.out.ogr/main.c</a></div><div><br></div><div>line 457</div><div><br></div><div>    for (i = 0; i < OGRGetDriverCount(); i++) {</div><div>        ...<br><br>I could probably commit, but before I do, I would like to get confirmed by PSC, this is OK to GRASS community.</div><div><br></div><div>Jachym</div><div><br></div><div>P.S. I've prepared similar patch for QGIS for those, who are interested <a href="https://github.com/qgis/QGIS/pull/5385">https://github.com/qgis/QGIS/pull/5385</a><br><br></div><br><div class="gmail_quote"><div dir="ltr">st 18. 10. 2017 v 14:36 odesílatel Even Rouault <<a href="mailto:even.rouault@spatialys.com">even.rouault@spatialys.com</a>> napsal:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On mercredi 18 octobre 2017 12:28:36 CEST Jachym Cepicky wrote:<br>
> Hi,<br>
><br>
> I would like to submit following commit to GRASS GIS - master<br>
><br>
> Index: vector/v.out.ogr/args.c<br>
> ===================================================================<br>
> --- vector/v.out.ogr/args.c (revision 71566)<br>
> +++ vector/v.out.ogr/args.c (working copy)<br>
> @@ -32,7 +32,7 @@<br>
>      options->format->type = TYPE_STRING;<br>
>      options->format->required = YES;<br>
>      options->format->multiple = NO;<br>
> -    options->format->answer = "ESRI_Shapefile";<br>
> +    options->format->answer = "DPKG";<br>
>      options->format->options = OGR_list_write_drivers();<br>
>      options->format->description = _("Data format to write");<br>
><br>
><br>
><br>
> any objections?<br>
><br>
<br>
I'm not a GRASS dev, but do I have a major objection... There's a typo in your<br>
patch ;-) Should be GPKG<br>
<br>
Joke aside, the SQLite dependency in GDAL builds is optional, so in theory you<br>
could have a GDAL build without SQLite/GPKG support. But this is really<br>
unlikely in practice as no sane packager would do that.  Perhaps checking if<br>
the GPKG driver is there would be safer, and if not fallback to good old<br>
shapefile.<br>
<br>
Even<br>
<br>
<br>
--<br>
Spatialys - Geospatial professional services<br>
<a href="http://www.spatialys.com" rel="noreferrer" target="_blank">http://www.spatialys.com</a><br>
</blockquote></div></div>