[Marble-devel] Review Request: add application/x-gpx+xml & application/x-esri-shape to the mime-type database

[Torsten Rahn]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106977/#review20654
-----------------------------------------------------------


Shouldn't we also have .OSM covered? :-)
Anyways the most important thing right now is to fix the license of the file that seems to be licensed under the GPL.


data/mimetypes/geo.xml
<http://git.reviewboard.kde.org/r/106977/#comment16303>

    I think "Shapefile" should be written with a capital "S", "ESRI Shapefile". It's also written like that by ESRI.



data/mimetypes/geo.xml
<http://git.reviewboard.kde.org/r/106977/#comment16304>

    "ESRI Shapefile Index", see above


- Torsten Rahn


On Oct. 22, 2012, 8:50 a.m., Friedrich W. H. Kossebau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106977/
> -----------------------------------------------------------
> 
> (Updated Oct. 22, 2012, 8:50 a.m.)
> 
> 
> Review request for Marble.
> 
> 
> Description
> -------
> 
> This patch installs an extension of the mimetype database for both file types.
> 
> This way such files get a nicer display in the file dialogs/managers and can also finally be bound to the Marble program by the mimetype ids.
> 
> Still to do after this patch:
> * have the view zoom to the loaded shape/track
> 
> Patch additionally installs a mimetype for the Esri shapefile index, so this file at least looks better in the filemanager.
> That mimetype is made up by me. Not sure if this is useful, could also remove this from the patch if you prefer. For that reason I have also not yet added entries for the two other additional file types with shapefiles.
> 
> PS: After applying the patch and installing somehow the new info about the mimetypes and that marble supports them seems to not make it to all running programs. For me it worked only completely after a restart of the workspace session, so please try that before reporting problems.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 7f8777e 
>   data/CMakeLists.txt 316d005 
>   data/mimetypes/XmlMessages.sh PRE-CREATION 
>   data/mimetypes/geo.xml PRE-CREATION 
>   src/marble_part.desktop 9a75d07 
>   src/plugins/runner/shp/CMakeLists.txt 0ef8c65 
>   src/plugins/runner/shp/marble_part_shp.desktop PRE-CREATION 
>   src/plugins/runner/shp/marble_shp.desktop PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/106977/diff/
> 
> 
> Testing
> -------
> 
> Clicking an gpx file or a shape file starts Marble and loads the file.
> 
> 
> Thanks,
> 
> Friedrich W. H. Kossebau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20121022/9b3a0377/attachment.html>