new update/upgrade script, ready for testing

sdetweil

@spitzlbergerj thx. Can u try the last command again

spitzlbergerj

Here are the results from the just newly started command:

curl -sL https://www.dropbox.com/s/80fevh39jme3iig/testlang.sh?dl=0 | bash
not english or locale not set, set git alias
alias git='LANG=en_US.UTF-8 git'
Auf Branch master
Ihr Branch ist zu 'origin/master' um 117 Commits hinterher, und kann vorgespult werden.
  (benutzen Sie "git pull", um Ihren lokalen Branch zu aktualisieren)
Unversionierte Dateien:
  (benutzen Sie "git add <Datei>...", um die Änderungen zum Commit vorzumerken)

        core
        installers/dumpactivemodules.js

nichts zum Commit vorgemerkt, aber es gibt unversionierte Dateien (benutzen Sie "git add" zum Versionieren)

The result is the same as last night, right?

Regards
Sepp

evroom

@spitzlbergerj

Hi there, I do not want to interfere, as Sam is most capable, but can you give the output of this command?

locale -a

And tell if you see English or German output for those 2 commands?

LC_ALL=C git status

LC_MESSAGES=C git status

sdetweil

@evroom i remove the alias so, the output will not tell us anything…

I was hoping there was some priority of the settings…

LC_ALL doers override LANGUAGE…
but LANG doesn’t appear to be dynamically changeable per command

try the testlang script again, the locale doc says this should work…

evroom

@sdetweil said in new update/upgrade script, ready for testing:

i remove the alias so, the output will not tell us anything…

It will tell you if you get English as output, despite any language & variable settings.

I temporary changed the locale to de_DE.utf8 for testing and git status gave me German output.
Then I did LC_ALL=C git status and it replied in English.
LC_MESSAGES=C git status continued to reply in German.

Just FYI, perhaps it will help you, perhaps not :-)

sdetweil

@evroom said in new update/upgrade script, ready for testing:

Then I did LC_ALL=C git status and it replied in English.

cool… thats what the test script does… sets alias for git to ‘LC_ALL=C git’

I think I will just force alias

sdetweil

@spitzlbergerj great, thanks… u can run the update script again now…

if it says already at 2.10, the change the parm from apply to force

evroom

@sdetweil

Sam,

for some reason your testlang.sh script, using alias, is still responding in German, whereas the plain command responses in English:

pi@raspberrypi:~/MagicMirror $ curl -sL https://www.dropbox.com/s/80fevh39jme3iig/testlang.sh?dl=0 | bash
Auf Branch master
Ihr Branch ist auf demselben Stand wie 'origin/master'.
:
:

pi@raspberrypi:~/MagicMirror $ LC_ALL=C git status
On branch master
Your branch is up to date with 'origin/master'.
:
:

pi@raspberrypi:~/MagicMirror $ git status
Auf Branch master
Ihr Branch ist auf demselben Stand wie 'origin/master'.
:
:

Also interesting; using sh iso bash:

pi@raspberrypi:~/MagicMirror $ curl -sL https://www.dropbox.com/s/80fevh39jme3iig/testlang.sh?dl=0 | sh
On branch master
Your branch is up to date with 'origin/master'.

EDIT - recommendations / friendly advice:

1/ Use #!/bin/bash in your script, not ~/bin/bash
2/ Do not alias the git command, just plain LC_ALL=C git status

I am on 2.10.0, but can try your testlang.sh script when needed.
Will keep the language settings on German for a while.

sdetweil

@evroom thanks… I have multiple commands to issue, and wanted to use alias for it…

thanks for the catch on #~…

can u try the test script again?
u can also look at the alias list , to make sure the one for git is set

alias

spitzlbergerj

@evroom said in new update/upgrade script, ready for testing:

locale -a

the output is:

C
C.UTF-8
de_DE.utf8
POSIX

spitzlbergerj

@sdetweil Hi, I run the script just now again. The output is the same I guess:

curl -sL https://www.dropbox.com/s/80fevh39jme3iig/testlang.sh?dl=0 | bash
Auf Branch master
Ihr Branch ist zu 'origin/master' um 117 Commits hinterher, und kann vorgespult werden.
  (benutzen Sie "git pull", um Ihren lokalen Branch zu aktualisieren)
Unversionierte Dateien:
  (benutzen Sie "git add <Datei>...", um die Änderungen zum Commit vorzumerken)

        core
        installers/dumpactivemodules.js

nichts zum Commit vorgemerkt, aber es gibt unversionierte Dateien (benutzen Sie "git add" zum Versionieren)

sdetweil

@spitzlbergerj don’t yah love it… what is supposed to work, doesn’t…

arg

sop I changed test script to do it your way… try it out

spitzlbergerj

@sdetweil Sorry if I misunderstood something. Should I have corrected something on my own?

OK, I started the script again. Now the output is as follows:

curl -sL https://www.dropbox.com/s/80fevh39jme3iig/testlang.sh?dl=0 | bash
On branch master
Your branch is behind 'origin/master' by 117 commits, and can be fast-forwarded.
  (use "git pull" to update your local branch)
Untracked files:
  (use "git add <file>..." to include in what will be committed)

        core
        installers/dumpactivemodules.js

nothing added to commit but untracked files present (use "git add" to track)

And the update script returns the following output:

bash -c  "$(curl -sL https://raw.githubusercontent.com/sdetweil/MagicMirror_scripts/master/upgrade-script.sh)"
the log will be /home/pi/MagicMirror/installers/upgrade.log
No LSB modules are available.
doing test run = true
update log will be in /home/pi/MagicMirror/installers/upgrade.log

saving custom.css
upgrading from version 2.9.0 to 2.10.0
fetching latest revisions
processing dependency changes for active modules with package.json files

processing for module MMM-Remote-Control please wait
----------------------------------
skipped processing for MMM-Remote-Control, doing test run
processing complete for module MMM-Remote-Control

processing for module MMM-NetworkScanner please wait
----------------------------------
skipped processing for MMM-NetworkScanner, doing test run
processing complete for module MMM-NetworkScanner

processing for module MMM-DWD-WarnWeather please wait
----------------------------------
skipped processing for MMM-DWD-WarnWeather, doing test run
processing complete for module MMM-DWD-WarnWeather

processing for module MMM-SystemStats please wait
----------------------------------
skipped processing for MMM-SystemStats, doing test run
processing complete for module MMM-SystemStats

restoring custom.css

So it all worked out now, right?

Thanks for your help @sdetweil and @evroom

sdetweil

@spitzlbergerj said in new update/upgrade script, ready for testing:

upgrading from version 2.9.0 to 2.10.0

all looks good… thank you so much for your time and patience…

but… the upgrader says u are still on 2.9… maybe a different folder? (as u said u were on 2.10 already)

spitzlbergerj

This post is deleted!

sdetweil

@spitzlbergerj no attachment… send via mail… same ID as here on gmail

evroom

@sdetweil said in new update/upgrade script, ready for testing:

I changed test script to do it your way… try it out

pi@raspberrypi:~/MagicMirror $ curl -sL https://www.dropbox.com/s/80fevh39jme3iig/testlang.sh?dl=0 | bash
On branch master
Your branch is up to date with 'origin/master'.

Sam. I think the issue is, that the command is git status and you alias LC_ALL=C git to the name git, not the command git.
You cannot use the alias as a replacement for a part of your command, but I might be wrong.

I think the only solution is to check for language en_US and then for every git command either use plain git or else by using LC_ALL=C.

Or just use LC_ALL=C for every git call.

sdetweil

@evroom

name git, not the command git

i don’t understand this…

its been working fine for quite a few months, handling LC_ALL and LANGUAGE, just hs system has LANG too…

anyhow, alias is supposed to work on ‘commands’ issued, and replace the started command with the replacement string (once)

this is how the git doc suggested dealing with this issue and stackexchange posts over quite some time echoed it…

I changes to the ‘LC_ALL=C git’ approach anyhow

I don’t even check now, just do it… on english system, it doesn’t hurt

spitzlbergerj

no, I didn’t say I was on 2.10.

i have now run the script in apply mode. The output is very long and apparently it was not updated, right?

Link Text

Sorry, I’ve always done the update via git. Now I am confused …

sdetweil

@spitzlbergerj yes, please erase this file

~/MagicMirror/installers/dumpactivemodules.js

it was copied early by the official updater in error