search

abrensch / brouter

configurable OSM offline router with elevation awareness, Java + Android
MIT License
490 stars 118 forks source link

App version check #526

Closed afischerdev closed 1 year ago

afischerdev commented 1 year ago

According to @polyscias update lookups comment a app version check is added.

Rework of the handling if profile param is used.

afischerdev commented 1 year ago

I think it would be good to do a release of the App soon after this has been checked in and the profile update only later.

Thanks for the test. Yes, I also think app first.

afischerdev commented 1 year ago

@polyscias Could you do some tests? It is hard to test when there is no server to download a new and changed lookups.dat For this situation I used this logic: When downloaded a new lookups.dat from server it is first stored in a file named lookups.dat.tmp And then the checks are done. You could transfer a manipulated tmp file from PC and then start BRouter download dialog. When there already is a tmp file this is checked. Current version code is 45

polyscias commented 1 year ago

I could not find time this long weekend and yesterday did build the brouter App with the patch. Now trying to test it with a servconfig.txt file pointing to my own web server with some test files but that needs still some debug, will update once I have results.

Not sure how much time I have coming days, so better not wait for me if you want to continue.