Commit: r2906: Merge the mergeroads branch.

Version mkgmap-r2906 was committed by wanmil on Sun, 22 Dec 2013 Merge the mergeroads branch. * Roads are merged which reduces the number of road entries by 7-25% using the default style. This results in slightly better routing calculation times and slightly smaller img file sizes. * The four labels to pois/lines/roads/polygons are no longer assigned automatically. They must be assigned using the 'name' and 'addlabel' function. This also means that ref tags are no longer used as labels automatically. * Access restrictions of roads are now defined by setting special mkgmap:** access tags (e.g. mkgmap:car, mkgmap:foot, mkgmap:bicycle etc.). * mkgmap:carpool does no longer automatically set all access restrictions. In the trunk mkgmap:carpool=yes/1/true lead to set all access restrictions to "no" except emergency, bus and carpool. * The road speed is no longer automatically calculated using the maxspeed tag. New style functions maxspeedkmh() and maxspeedmph() allow style developers to perform the same by setting the mkgmap:road-speed-class tag. * Styles can now have a <finalize> block. This block is executed for each element if an element style definition matches. The finalize block must contain actions only.

Hi Steve, the new release does not appear on the download page of www.mkgmap.org.uk. I can only guess that a junit test fails? Did you receive any error that need to be fixed? Thanks! WanMil
Version mkgmap-r2906 was committed by wanmil on Sun, 22 Dec 2013
Merge the mergeroads branch. * Roads are merged which reduces the number of road entries by 7-25% using the default style. This results in slightly better routing calculation times and slightly smaller img file sizes. * The four labels to pois/lines/roads/polygons are no longer assigned automatically. They must be assigned using the 'name' and 'addlabel' function. This also means that ref tags are no longer used as labels automatically. * Access restrictions of roads are now defined by setting special mkgmap:** access tags (e.g. mkgmap:car, mkgmap:foot, mkgmap:bicycle etc.). * mkgmap:carpool does no longer automatically set all access restrictions. In the trunk mkgmap:carpool=yes/1/true lead to set all access restrictions to "no" except emergency, bus and carpool. * The road speed is no longer automatically calculated using the maxspeed tag. New style functions maxspeedkmh() and maxspeedmph() allow style developers to perform the same by setting the mkgmap:road-speed-class tag. * Styles can now have a <finalize> block. This block is executed for each element if an element style definition matches. The finalize block must contain actions only. _______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

Yes a test is failing. I will forward the message. ..Steve Sent with AquaMail for Android http://www.aqua-mail.com On 23 December 2013 09:21:09 WanMil <wmgcnfg@web.de> wrote:
Hi Steve,
the new release does not appear on the download page of www.mkgmap.org.uk.
I can only guess that a junit test fails? Did you receive any error that need to be fixed?
Thanks! WanMil
Version mkgmap-r2906 was committed by wanmil on Sun, 22 Dec 2013
Merge the mergeroads branch. * Roads are merged which reduces the number of road entries by 7-25% using the default style. This results in slightly better routing calculation times and slightly smaller img file sizes. * The four labels to pois/lines/roads/polygons are no longer assigned automatically. They must be assigned using the 'name' and 'addlabel' function. This also means that ref tags are no longer used as labels automatically. * Access restrictions of roads are now defined by setting special mkgmap:** access tags (e.g. mkgmap:car, mkgmap:foot, mkgmap:bicycle etc.). * mkgmap:carpool does no longer automatically set all access restrictions. In the trunk mkgmap:carpool=yes/1/true lead to set all access restrictions to "no" except emergency, bus and carpool. * The road speed is no longer automatically calculated using the maxspeed tag. New style functions maxspeedkmh() and maxspeedmph() allow style developers to perform the same by setting the mkgmap:road-speed-class tag. * Styles can now have a <finalize> block. This block is executed for each element if an element style definition matches. The finalize block must contain actions only. _______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
_______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

Hi Steve, that is strange. The tests did not fail on my machine, but I think they should have, e.g. ScriptedStyleTest with get-tag.test should report many errors because name, labels and refs are different now. I have no idea why the tests did not fail with r2906 Gerd
From: steve@parabola.me.uk To: mkgmap-dev@lists.mkgmap.org.uk Date: Mon, 23 Dec 2013 09:48:46 +0000 Subject: Re: [mkgmap-dev] Commit: r2906: Merge the mergeroads branch.
Yes a test is failing. I will forward the message.
..Steve
Sent with AquaMail for Android http://www.aqua-mail.com
On 23 December 2013 09:21:09 WanMil <wmgcnfg@web.de> wrote:
Hi Steve,
the new release does not appear on the download page of www.mkgmap.org.uk.
I can only guess that a junit test fails? Did you receive any error that need to be fixed?
Thanks! WanMil
Version mkgmap-r2906 was committed by wanmil on Sun, 22 Dec 2013
Merge the mergeroads branch. * Roads are merged which reduces the number of road entries by 7-25% using the default style. This results in slightly better routing calculation times and slightly smaller img file sizes. * The four labels to pois/lines/roads/polygons are no longer assigned automatically. They must be assigned using the 'name' and 'addlabel' function. This also means that ref tags are no longer used as labels automatically. * Access restrictions of roads are now defined by setting special mkgmap:** access tags (e.g. mkgmap:car, mkgmap:foot, mkgmap:bicycle etc.). * mkgmap:carpool does no longer automatically set all access restrictions. In the trunk mkgmap:carpool=yes/1/true lead to set all access restrictions to "no" except emergency, bus and carpool. * The road speed is no longer automatically calculated using the maxspeed tag. New style functions maxspeedkmh() and maxspeedmph() allow style developers to perform the same by setting the mkgmap:road-speed-class tag. * Styles can now have a <finalize> block. This block is executed for each element if an element style definition matches. The finalize block must contain actions only. _______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
_______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
_______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

Hi Gerd OK. I don't have many ideas of why it might be different. The build is done from a completely fresh checkout each time and from a bare environment is the only differences I can think of. These days all builds are created on checking so that if the build does not appear in the download list within a few minutes then it has failed. (The big green download now button don't update straight away) I can't look into it just at the moment as I am on a train with standing room only. ..Steve Gerd Petermann <gpetermann_muenchen@hotmail.com> wrote:
Hi Steve,
that is strange. The tests did not fail on my machine, but I think they should have, e.g. ScriptedStyleTest with get-tag.test should report many errors because name, labels and refs are different now. I have no idea why the tests did not fail with r2906
Gerd
From: steve@parabola.me.uk To: mkgmap-dev@lists.mkgmap.org.uk Date: Mon, 23 Dec 2013 09:48:46 +0000 Subject: Re: [mkgmap-dev] Commit: r2906: Merge the mergeroads branch.
Yes a test is failing. I will forward the message.
..Steve
Sent with AquaMail for Android http://www.aqua-mail.com
On 23 December 2013 09:21:09 WanMil <wmgcnfg@web.de> wrote:
Hi Steve,
the new release does not appear on the download page of www.mkgmap.org.uk.
I can only guess that a junit test fails? Did you receive any error that need to be fixed?
Thanks! WanMil
Version mkgmap-r2906 was committed by wanmil on Sun, 22 Dec 2013
Merge the mergeroads branch. * Roads are merged which reduces the number of road entries by
7-25%
using the default style. This results in slightly better routing calculation times and slightly smaller img file sizes. * The four labels to pois/lines/roads/polygons are no longer assigned automatically. They must be assigned using the 'name' and 'addlabel' function. This also means that ref tags are no longer used as labels automatically. * Access restrictions of roads are now defined by setting special mkgmap:** access tags (e.g. mkgmap:car, mkgmap:foot, mkgmap:bicycle etc.). * mkgmap:carpool does no longer automatically set all access restrictions. In the trunk mkgmap:carpool=yes/1/true lead to set all access restrictions to "no" except emergency, bus and carpool. * The road speed is no longer automatically calculated using the maxspeed tag. New style functions maxspeedkmh() and maxspeedmph() allow style developers to perform the same by setting the mkgmap:road-speed-class tag. * Styles can now have a <finalize> block. This block is executed for each element if an element style definition matches. The finalize block must contain actions only. _______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
_______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
_______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
------------------------------------------------------------------------
_______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

Hi Steve, sorry, my fault. I ran the tests on r2905 with success. It seems that during the merge to trunk some changes got lost. Gerd Steve Ratcliffe wrote
Hi Gerd
OK. I don't have many ideas of why it might be different. The build is done from a completely fresh checkout each time and from a bare environment is the only differences I can think of.
These days all builds are created on checking so that if the build does not appear in the download list within a few minutes then it has failed. (The big green download now button don't update straight away)
I can't look into it just at the moment as I am on a train with standing room only.
..Steve
Gerd Petermann <
gpetermann_muenchen@
> wrote:
Hi Steve,
that is strange. The tests did not fail on my machine, but I think they should have, e.g. ScriptedStyleTest with get-tag.test should report many errors because name, labels and refs are different now. I have no idea why the tests did not fail with r2906
Gerd
From:
steve@.me
To:
mkgmap-dev@.org
Date: Mon, 23 Dec 2013 09:48:46 +0000 Subject: Re: [mkgmap-dev] Commit: r2906: Merge the mergeroads branch.
Yes a test is failing. I will forward the message.
..Steve
Sent with AquaMail for Android http://www.aqua-mail.com
On 23 December 2013 09:21:09 WanMil <
wmgcnfg@
> wrote:
Hi Steve,
the new release does not appear on the download page of www.mkgmap.org.uk.
I can only guess that a junit test fails? Did you receive any error that need to be fixed?
Thanks! WanMil
Version mkgmap-r2906 was committed by wanmil on Sun, 22 Dec 2013
Merge the mergeroads branch. * Roads are merged which reduces the number of road entries by
7-25%
using the default style. This results in slightly better routing calculation times and slightly smaller img file sizes. * The four labels to pois/lines/roads/polygons are no longer assigned automatically. They must be assigned using the 'name' and 'addlabel' function. This also means that ref tags are no longer used as labels automatically. * Access restrictions of roads are now defined by setting special mkgmap:** access tags (e.g. mkgmap:car, mkgmap:foot, mkgmap:bicycle etc.). * mkgmap:carpool does no longer automatically set all access restrictions. In the trunk mkgmap:carpool=yes/1/true lead to set all access restrictions to "no" except emergency, bus and carpool. * The road speed is no longer automatically calculated using the maxspeed tag. New style functions maxspeedkmh() and maxspeedmph() allow style developers to perform the same by setting the mkgmap:road-speed-class tag. * Styles can now have a <finalize> block. This block is executed for each element if an element style definition matches. The finalize block must contain actions only. _______________________________________________ mkgmap-dev mailing list
mkgmap-dev@.org
_______________________________________________ mkgmap-dev mailing list
mkgmap-dev@.org
_______________________________________________ mkgmap-dev mailing list
mkgmap-dev@.org
------------------------------------------------------------------------
_______________________________________________ mkgmap-dev mailing list
mkgmap-dev@.org
_______________________________________________ mkgmap-dev mailing list
mkgmap-dev@.org
-- View this message in context: http://gis.19327.n5.nabble.com/Commit-r2906-Merge-the-mergeroads-branch-tp57... Sent from the Mkgmap Development mailing list archive at Nabble.com.

I have commited three tests that got lost during the merge. I have no idea why they got lost but now tests should work. WanMil
Hi Steve,
sorry, my fault. I ran the tests on r2905 with success. It seems that during the merge to trunk some changes got lost.
Gerd
Steve Ratcliffe wrote
Hi Gerd
OK. I don't have many ideas of why it might be different. The build is done from a completely fresh checkout each time and from a bare environment is the only differences I can think of.
These days all builds are created on checking so that if the build does not appear in the download list within a few minutes then it has failed. (The big green download now button don't update straight away)
I can't look into it just at the moment as I am on a train with standing room only.
..Steve
Gerd Petermann <
gpetermann_muenchen@
> wrote:
Hi Steve,
that is strange. The tests did not fail on my machine, but I think they should have, e.g. ScriptedStyleTest with get-tag.test should report many errors because name, labels and refs are different now. I have no idea why the tests did not fail with r2906
Gerd
From:
steve@.me
To:
mkgmap-dev@.org
Date: Mon, 23 Dec 2013 09:48:46 +0000 Subject: Re: [mkgmap-dev] Commit: r2906: Merge the mergeroads branch.
Yes a test is failing. I will forward the message.
..Steve
Sent with AquaMail for Android http://www.aqua-mail.com
On 23 December 2013 09:21:09 WanMil <
wmgcnfg@
> wrote:
Hi Steve,
the new release does not appear on the download page of www.mkgmap.org.uk.
I can only guess that a junit test fails? Did you receive any error that need to be fixed?
Thanks! WanMil
Version mkgmap-r2906 was committed by wanmil on Sun, 22 Dec 2013
Merge the mergeroads branch. * Roads are merged which reduces the number of road entries by
7-25%
using the default style. This results in slightly better routing calculation times and slightly smaller img file sizes. * The four labels to pois/lines/roads/polygons are no longer assigned automatically. They must be assigned using the 'name' and 'addlabel' function. This also means that ref tags are no longer used as labels automatically. * Access restrictions of roads are now defined by setting special mkgmap:** access tags (e.g. mkgmap:car, mkgmap:foot, mkgmap:bicycle etc.). * mkgmap:carpool does no longer automatically set all access restrictions. In the trunk mkgmap:carpool=yes/1/true lead to set all access restrictions to "no" except emergency, bus and carpool. * The road speed is no longer automatically calculated using the maxspeed tag. New style functions maxspeedkmh() and maxspeedmph() allow style developers to perform the same by setting the mkgmap:road-speed-class tag. * Styles can now have a <finalize> block. This block is executed for each element if an element style definition matches. The finalize block must contain actions only. _______________________________________________ mkgmap-dev mailing list
mkgmap-dev@.org
_______________________________________________ mkgmap-dev mailing list
mkgmap-dev@.org
_______________________________________________ mkgmap-dev mailing list
mkgmap-dev@.org
------------------------------------------------------------------------
_______________________________________________ mkgmap-dev mailing list
mkgmap-dev@.org
_______________________________________________ mkgmap-dev mailing list
mkgmap-dev@.org
-- View this message in context: http://gis.19327.n5.nabble.com/Commit-r2906-Merge-the-mergeroads-branch-tp57... Sent from the Mkgmap Development mailing list archive at Nabble.com. _______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

Hi Steve, the new mkgmap version now appears at other builds but the main download still links to r2889. Can you please have a look on it if you are back again? Thanks! WanMil
Hi Gerd
OK. I don't have many ideas of why it might be different. The build is done from a completely fresh checkout each time and from a bare environment is the only differences I can think of.
These days all builds are created on checking so that if the build does not appear in the download list within a few minutes then it has failed. (The big green download now button don't update straight away)
I can't look into it just at the moment as I am on a train with standing room only.
..Steve
Gerd Petermann <gpetermann_muenchen@hotmail.com> wrote:
Hi Steve,
that is strange. The tests did not fail on my machine, but I think they should have, e.g. ScriptedStyleTest with get-tag.test should report many errors because name, labels and refs are different now. I have no idea why the tests did not fail with r2906
Gerd
> From: steve@parabola.me.uk > To: mkgmap-dev@lists.mkgmap.org.uk > Date: Mon, 23 Dec 2013 09:48:46 +0000 > Subject: Re: [mkgmap-dev] Commit: r2906: Merge the mergeroads branch. > > Yes a test is failing. I will forward the message. > > ..Steve > > Sent with AquaMail for Android > http://www.aqua-mail.com > > > On 23 December 2013 09:21:09 WanMil <wmgcnfg@web.de> wrote: > > Hi Steve, > > > > the new release does not appear on the download page of www.mkgmap.org.uk. > > > > I can only guess that a junit test fails? Did you receive any error that > > need to be fixed? > > > > Thanks! > > WanMil > > > > > > > > Version mkgmap-r2906 was committed by wanmil on Sun, 22 Dec 2013 > > > > > > Merge the mergeroads branch. > > > * Roads are merged which reduces the number of road entries by 7-25% > > > using the default style. This results in slightly better routing > > > calculation times and slightly smaller img file sizes. > > > * The four labels to pois/lines/roads/polygons are no longer assigned > > > automatically. They must be assigned using the 'name' and 'addlabel' > > > function. This also means that ref tags are no longer used as labels > > > automatically. > > > * Access restrictions of roads are now defined by setting special > > > mkgmap:** access tags (e.g. mkgmap:car, mkgmap:foot, mkgmap:bicycle etc.). > > > * mkgmap:carpool does no longer automatically set all access > > > restrictions. In the trunk mkgmap:carpool=yes/1/true lead to set all > > > access restrictions to "no" except emergency, bus and carpool. > > > * The road speed is no longer automatically calculated using the > > > maxspeed tag. New style functions maxspeedkmh() and maxspeedmph() allow > > > style developers to perform the same by setting the > > > mkgmap:road-speed-class tag. > > > * Styles can now have a <finalize> block. This block is executed for > > > each element if an element style definition matches. The finalize block > > > must contain actions only. > > > _______________________________________________ > > > mkgmap-dev mailing list > ; > > mkgmap-dev@lists.mkgmap.org.uk > > > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev > > > > > > > _______________________________________________ > > mkgmap-dev mailing list > > mkgmap-dev@lists.mkgmap.org.uk > > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev > > > _______________________________________________ > mkgmap-dev mailing list > mkgmap-dev@lists.mkgmap.org.uk > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
------------------------------------------------------------------------
mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
_______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

Hi WanMil
the new mkgmap version now appears at other builds but the main download still links to r2889. Can you please have a look on it if you are back again?
OK I have now set it so that it should be updated tonight. And thanks for posting the news item about it. I was going to suggest that we need such a post and that you (and Gerd) are able to post updates yourselves, but then I saw that you discovered that anyway ;) I'll quickly explain what is happening with the download page. A while back someone on the forums said that it was confusing to know which build to download. I thought that was a fair comment for someone coming to the project for the first time. So I added the green download button which is meant to be for a reasonably stable version. On the other hand a new build is created for every commit to trunk and the branches immediately after they are committed. I don't really know the best way to update the 'green' download so suggestions are welcome. Currently it updates every night to the latest version unless a particular version has been pinned. I was thinking of a way to mark a particular build as bad if it had regressions and a way to stop updates for a few days after a merge or something simple like that. For reference: the following binary and source formats are produced for every build. They all exist for each build (trunk and branch) whether linked to or not. mkgmap-rNNN.jar mkgmap-rNNN.tar.gz mkgmap-rNNN.zip mkgmap-rNNN-src.zip mkgmap-rNNN-src.tar.gz The -rNNN- part can be replaced with -latest- in any of these to get the most up to date version. Everything is exactly the same for splitter. ..Steve

Hi Steve,
And thanks for posting the news item about it. I was going to suggest that we need such a post and that you (and Gerd) are able to post updates yourselves, but then I saw that you discovered that anyway ;)
sorry, I don't know how to do it, please let me know what is needed. Gerd

Hi WanMil
the new mkgmap version now appears at other builds but the main download still links to r2889. Can you please have a look on it if you are back again?
OK I have now set it so that it should be updated tonight.
And thanks for posting the news item about it. I was going to suggest that we need such a post and that you (and Gerd) are able to post updates yourselves, but then I saw that you discovered that anyway ;)
I'll quickly explain what is happening with the download page.
A while back someone on the forums said that it was confusing to know which build to download. I thought that was a fair comment for someone coming to the project for the first time.
So I added the green download button which is meant to be for a reasonably stable version. On the other hand a new build is created for every commit to trunk and the branches immediately after they are committed.
I don't really know the best way to update the 'green' download so suggestions are welcome. Currently it updates every night to the latest version unless a particular version has been pinned. I was thinking of a way to mark a particular build as bad if it had regressions and a way to stop updates for a few days after a merge or something simple like that.
For reference: the following binary and source formats are produced for every build. They all exist for each build (trunk and branch) whether linked to or not.
mkgmap-rNNN.jar mkgmap-rNNN.tar.gz mkgmap-rNNN.zip mkgmap-rNNN-src.zip mkgmap-rNNN-src.tar.gz
The -rNNN- part can be replaced with -latest- in any of these to get the most up to date version.
Everything is exactly the same for splitter.
..Steve
Hi Steve, thanks for the explanation! I think the big green download button is a good idea! Maybe we can track the latest stable version number in Subversion? Create a file called stable.version that contains the stable Subversion revision. This revision is checked out for building the stable release. This would allow all committers to change the stable release. As a rule of thumb we should update the stable release around once a month. People who want to work with the latest release can download them in the list below. That sounds fair enough. WanMil
participants (5)
-
Gerd Petermann
-
GerdP
-
Steve Ratcliffe
-
svn commit
-
WanMil