Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

10m_admin_0_boundary_lines_land.shp no longer used #8

Closed
dmorissette opened this issue Aug 3, 2012 · 1 comment
Closed

10m_admin_0_boundary_lines_land.shp no longer used #8

dmorissette opened this issue Aug 3, 2012 · 1 comment

Comments

@dmorissette
Copy link

The data/Makefile still has a ref and target to download and unzip the 10m_admin_0_boundary_lines_land.shp dataset, but the mapfiles no longer use it. It should probably be removed.

The reason why I found this out is that Natural Earth has renamed the contents of this zip file to include a "ne_" prefix, so the Makefile as it is now produces an error anyway for this dataset when it tries to run 'shptree' on the empty shapefile produced by the 'touch' command.

Here is what today's version of the Natural Earth 10m-admin-0-boundary-lines-land.zip looks like:

$ unzip -v 10m-admin-0-boundary-lines-land.zip 

Archive:  10m-admin-0-boundary-lines-land.zip
 Length   Method    Size  Ratio   Date   Time   CRC-32    Name
--------  ------  ------- -----   ----   ----   ------    ----
   11760  Defl:N      164  99%  07-24-11 22:34  73839fbc  ne_10m_admin_0_boundary_lines_land.dbf
     145  Defl:N      121  17%  07-24-11 21:47  af437f55  ne_10m_admin_0_boundary_lines_land.prj
 1199700  Defl:N   906630  24%  08-08-11 02:51  cdd91b9e  ne_10m_admin_0_boundary_lines_land.shp
    2844  Defl:N     1895  33%  08-08-11 02:51  3373b3b9  ne_10m_admin_0_boundary_lines_land.shx
--------          -------  ---                            -------
 1214449           908810  25%                            4 files
@maraev
Copy link

maraev commented Jan 15, 2013

I see the strange link in 31 line of data\Makefile that is why I get error when I make it.
Should change 10m-admin-0-boundary-lines-land to ne_10m_admin_0_boundary_lines_land in the link..right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants