Hi,
als Vorbereitung für die Unit Tests die ich einbauen möchte müssen wir mal ein bisschen was segmentieren im hoodselector und besser trennen
Gruß
Johannes
From 28f0fa7aa55b732585197c261c8a242fd2d60a0c Mon Sep 17 00:00:00 2001 From: Johannes Rudolph johannes.rudolph@gmx.com Date: Thu, 29 Dec 2016 22:30:59 +0100 Subject: [PATCH] split getGeoLocationFromWebService
files/usr/sbin/hoodselector | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/files/usr/sbin/hoodselector b/files/usr/sbin/hoodselector index 072217b..a70a186 100755 --- a/files/usr/sbin/hoodselector +++ b/files/usr/sbin/hoodselector @@ -45,7 +45,7 @@ local function get_geolocation() local lat = uci:get('gluon-node-info', uci:get_first('gluon-node-info', 'location'), 'latitude') local lng = uci:get('gluon-node-info', uci:get_first('gluon-node-info', 'location'), 'longitude') if ( lat == nil or lng == nil ) then
for scan in io.popen(string.format("lwtrace -t 2> /dev/null"), 'r'):lines() do
for scan in get_geoLocationFromWebService():lines() do
Der Patch besteht nicht auf dem aktuellen Code. Die Positions-Ermittlung wurde abstrahiert, Ein Aufruf von lwtrace findet garnicht mehr statt.
if string.find(scan,"(lat)") then local last_val = nil for geo in string.gmatch(scan,"[^%s]+") do@@ -66,6 +66,11 @@ local function get_geolocation() return ret end +-- Get GeoPositon from Webservice +local function get_geoLocationFromWebService()
- return io.popen(string.format("lwtrace -t 2> /dev/null"), 'r')
+end
Hm, eine bessere Lösung ist denke ich, lwtrace zu emulieren z.b. mit einem shell Script z.B. Die O.g Änderung halte ich nicht für sinvoll.
vg Tarek