Replace existing shapefile module with shapefile-ng #176

Merged
merged 11 commits into from Apr 13, 2013

Conversation

Projects
None yet
2 participants
@aaime
Member

aaime commented Apr 7, 2013

No description provided.

@jodygarnett

This comment has been minimized.

Show comment
Hide comment
@jodygarnett

jodygarnett Apr 8, 2013

Thanks Andrea it looks good

Thanks Andrea it looks good

@jodygarnett

This comment has been minimized.

Show comment
Hide comment
@aaime

This comment has been minimized.

Show comment
Hide comment
@aaime

aaime Apr 8, 2013

Member

Roger about the upgrade guide, for that I'll have to wait until the PMC votes on the deprecated methods stuff. For cvs2shp, I don't see anything that needs changing?

Member

aaime commented Apr 8, 2013

Roger about the upgrade guide, for that I'll have to wait until the PMC votes on the deprecated methods stuff. For cvs2shp, I don't see anything that needs changing?

@jodygarnett

This comment has been minimized.

Show comment
Hide comment
@jodygarnett

jodygarnett Apr 8, 2013

Member

That may indeed be the case, it is just an example in the docs that uses ShapefileDataStore explicitly.

I searched google for: ShapefileDataStore site:docs.geotools.org/latest/userguide

Only one I can still see is:

Jody Garnett

On Monday, 8 April 2013 at 6:23 PM, Andrea Aime wrote:

Roger about the upgrade guide, for that I'll have to wait until the PMC votes on the deprecated methods stuff. For cvs2shp, I don't see anything that needs changing?


Reply to this email directly or view it on GitHub (#176 (comment)).

Member

jodygarnett commented Apr 8, 2013

That may indeed be the case, it is just an example in the docs that uses ShapefileDataStore explicitly.

I searched google for: ShapefileDataStore site:docs.geotools.org/latest/userguide

Only one I can still see is:

Jody Garnett

On Monday, 8 April 2013 at 6:23 PM, Andrea Aime wrote:

Roger about the upgrade guide, for that I'll have to wait until the PMC votes on the deprecated methods stuff. For cvs2shp, I don't see anything that needs changing?


Reply to this email directly or view it on GitHub (#176 (comment)).

@aaime

This comment has been minimized.

Show comment
Hide comment
@aaime

aaime Apr 9, 2013

Member

That one still also uses the single URL constructor, which is still supported, no issues there

Member

aaime commented Apr 9, 2013

That one still also uses the single URL constructor, which is still supported, no issues there

aaime added a commit that referenced this pull request Apr 13, 2013

Merge pull request #176 from aaime/shapefile-ng
Replace existing shapefile module with shapefile-ng

@aaime aaime merged commit 7ea4466 into geotools:master Apr 13, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment