grass
grass copied to clipboard
OSGeo
v.out.ogr: Fix exporting vector layers from a location whose CRS has no EPSG code
This PR addresses issue #2187: the v.out.ogr module fails to export vector layers from a location whose CRS does not have an entry in the EPSG registry. The root cause is the inadequate discarding of the Ogr_projection object in vector/v.out.ogr/main.c. In addition a new test is provided for this module verifying the correct creation of layers from such locations. This test fails with the current master branch (release 8.3).
This PR is incremental on PR #3848. Please merge that one before merging this one.
@ldesousa Can you please:
- update the description to include what this is fixing and how and when it is failing (I suppose that the new test is failing without the change and passing with it, correct?)
- shorten the title
Once it will be ready for merging, the title isn't clear to me what it does (an issue number is great for PR descriptions, but less as a changelog entry and PR titles). Our PR titles are changelog entries, and are what we will see in the code in for the rest of the lifetime of the project
Once it will be ready for merging, the title isn't clear to me what it does (an issue number is great for PR descriptions, but less as a changelog entry and PR titles). Our PR titles are changelog entries, and are what we will see in the code in for the rest of the lifetime of the project
@echoix I already lengthened/shortened the title a few times. Could you confer with @wenzeslaus which kind of title you wish for this PR?
@rouault I already lengthened/shortened the title a few times. Could you confer with @wenzeslaus which kind of title you wish for this PR?
maybe "v.out.ogr: fix exporting vector layers from a location whose CRS has no EPSG code" ?
@rouault Sorry, wrong tag. Let's wait for all the internal GRASS procedures to be ready. Then I will ask for your review again. Thanks.
@rouault I already lengthened/shortened the title a few times. Could you confer with @wenzeslaus which kind of title you wish for this PR?
maybe "v.out.ogr: fix exporting vector layers from a location whose CRS has no EPSG code" ?
I actually find that great, I can't reword it any better way that doesn't end up sounding weird