[mkgmap-dev] [PATCH v1] Add defer relation support to PBF reading

Mon Oct 24 17:50:49 BST 2011

> Hello,
>
>> I downloaded the finland.osm.pbf extract from geofabrik, run splitter
>> and used mkgmap with the patch. As you described the patch did nothing.
>> I reconverted the tile containing the relation 155054 to osm XML using
>> osmconvert. In the resulting file no relation contained a member with
>> type="relation". Only members with type="node" or type="way" are contained.
>>
>> So there are three possible problems:
>> 1. The geofabrik dump does not contain subrelations
>> 2. Splitter does not support subrelations
>> 3. Osmconvert does not support subrelations
>>
>> My current favourite is that splitter is the problem. Maybe someone who
>> knows splitter better than me can have a look at it? Thanks!
>
> It's 2.) Splitter. Relations that only contain relations are dropped.
> I thought this was known and someone will work on it in the future.
>
> You can test all three cases by filtering out the critical relations with
> osmfilter
>
> With best regards
>
> Hasemann

Thanks for the analysis.
The fix in splitter seems to be very easy. Attached patch does the magic.

WanMil
-------------- next part --------------
A non-text attachment was scrubbed...
Name: splitter_pbf_subrelations_v1.patch
Type: text/x-patch
Size: 540 bytes
Desc: not available
Url : http://lists.mkgmap.org.uk/pipermail/mkgmap-dev/attachments/20111024/7a9d0c26/attachment.bin 