Modified timesetting procedure again

With our current code we get problems when the user want to set datetime
manually in our dialog. Revisiting last changes i see: it's a bad thing
to force any timesetting based on a *selected* timezone in an still UTC
environment.
So the easiest way (and i hope the last change on this!) is:
a) user selected a timezone
b) we set this timezonefile to /etc/localtime
c) Now we ask: is your hwclock on UTC or localtime?
c) UTC BIOSclock user now have both correct system and hw time.
d) For localtime users we must re-read the hwclock into our new
timezone, cause copy'ing above timezone file has recalculated the
systemtime to a false time.
I've tested these commit on my PC, both with hwclock on UTC and
localtime, also with manual setting and via ntp(date) in the aif dialog.

Signed-off-by: Gerhard Brauer <gerbra@archlinux.de>

1 files changed, 12 insertions, 10 deletions

diff --git a/src/core/libs/lib-ui-interactive.sh b/src/core/libs/lib-ui-interactive.sh
index e66403a..3774707 100644
--- a/src/core/libs/lib-ui-interactive.sh
+++ b/src/core/libs/lib-ui-interactive.sh
@@ -94,20 +94,28 @@ interactive_timezone () {
 	ask_timezone || return 1
         TIMEZONE=$ANSWER_TIMEZONE
         infofy "Setting Timezone to $TIMEZONE"
+		if [ -n "$TIMEZONE" -a -e "/usr/share/zoneinfo/$TIMEZONE" ]
+		then
+			# This changes probably also the systemtime (UTC->$TIMEZONE)!
+			# localtime users will have a false time after that!
+			/bin/rm -f /etc/localtime
+			/bin/cp "/usr/share/zoneinfo/$TIMEZONE" /etc/localtime
+		fi
 }
 
 
-
 interactive_time () {
         # utc or localtime?
         ask_option UTC "Clock configuration" "Is your hardware clock in UTC or local time? UTC is recommended" required "UTC" " " "localtime" " " || return 1
         HARDWARECLOCK=$ANSWER_OPTION
-        [ -e /etc/localtime ] && rm -f /etc/localtime
+		# To avoid a false time for localtime users after above
+		# we must re-read the hwclock value again, but now into the
+		# correct timezone.
+		[ "$HARDWARECLOCK" == "localtime" ] && dohwclock $HARDWARECLOCK hctosys
 
 	NEXTITEM=
         while true; do
-		dohwclock $HARDWARECLOCK hctosys $TIMEZONE
-		current=$(TZ=$TIMEZONE date)
+		current=$(date)
                 default=no
                 [ -n "$NEXTITEM" ] && default="$NEXTITEM"
                 #TODO: only propose if network ok
@@ -138,12 +146,6 @@ interactive_time () {
 		fi
                 [ "$ANSWER_OPTION" = return ] && break
         done
-
-	if [ "$TIMEZONE" != "" -a -e "/usr/share/zoneinfo/$TIMEZONE" ]
-	then
-		/bin/rm -f /etc/localtime
-		/bin/cp "/usr/share/zoneinfo/$TIMEZONE" /etc/localtime
-	fi
 }