MagicMirror Forum
    • Recent
    • Tags
    • Unsolved
    • Solved
    • MagicMirror² Repository
    • Documentation
    • 3rd-Party-Modules
    • Donate
    • Discord
    • Register
    • Login
    A New Chapter for MagicMirror: The Community Takes the Lead
    Read the statement by Michael Teeuw here.

    new update/upgrade script, ready for testing

    Scheduled Pinned Locked Moved General Discussion
    328 Posts 27 Posters 659.1k Views 24 Watching
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • S Do not disturb
      sdetweil @bhepler
      last edited by

      @bhepler i added handling for the git lock file…

      if git is NOT running, I erase the dangling lock file
      if git IS running, and this is an APPLY run, i change the run back to test mode, and advise to correct the problem

      1 Reply Last reply Reply Quote 2
      • Mykle1M Offline
        Mykle1 Project Sponsor Module Developer
        last edited by

        I just did another run of your upgrade script, Sam. All went well.

        MM 2.8.0 --> 2.9.0
        Ubuntu 16.04.6 (downgraded from 18.04)

        Create a working config
        How to add modules

        S 1 Reply Last reply Reply Quote 0
        • S Do not disturb
          sdetweil @Mykle1
          last edited by

          @Mykle1 on the 1st run, no modules were updated… there should have been a message in the log… i didn’t see it… did u see any this time??

          Mykle1M 1 Reply Last reply Reply Quote 0
          • Mykle1M Offline
            Mykle1 Project Sponsor Module Developer @sdetweil
            last edited by

            @sdetweil

            Sorry, I was distracted here, Sam. I just emailed you the upgrade.log

            Create a working config
            How to add modules

            S 1 Reply Last reply Reply Quote 0
            • S Do not disturb
              sdetweil @Mykle1
              last edited by

              @Mykle1 thanks… they(last night and today) actually look good
              i did make one change, the merge results generated one LONE line… i added some formatting to make it readable

              1 Reply Last reply Reply Quote 1
              • S Do not disturb
                sdetweil @Mykle1
                last edited by sdetweil

                @Mykle1 well, there is installed modules, and there is active modules…

                i only update the ACTIVE (ie, not disabled: true) modules with package.json files

                Mykle1M 1 Reply Last reply Reply Quote 1
                • Mykle1M Offline
                  Mykle1 Project Sponsor Module Developer @sdetweil
                  last edited by

                  @sdetweil

                  Understood. :thumbsup:

                  Create a working config
                  How to add modules

                  1 Reply Last reply Reply Quote 0
                  • B Offline
                    bobwilliams
                    last edited by bobwilliams

                    for some reason, it updates me to version 2.8.0, although it seems like version 2.9.0 has already been released.

                    MagicMirror installation successful!

                    magicmirror@2.8.0 install-fonts /home/pi/MagicMirror
                    cd fonts && npm install

                    S 1 Reply Last reply Reply Quote 0
                    • S Do not disturb
                      sdetweil @bobwilliams
                      last edited by

                      @bobwilliams can you show me the MagicMirror/installers/upgrade.log

                      or send it to me mail, same userid as here on gmail, thanks

                      1 Reply Last reply Reply Quote 0
                      • B Offline
                        bobwilliams
                        last edited by bobwilliams

                        Upgrade started - Sun Oct 6 11:26:51 +05 2019
                        user requested to apply changes
                        doing test run = false

                        saving custom.css
                        remote name = origin
                        upgrading from version 2.8.0 to 2.9.0
                        fetching latest revisions
                        current branch = master
                        there are 7 local files that are different than the master repo

                        installers/mm.sh
                        modules/default/calendar/calendar.js
                        modules/default/currentweather/currentweather.js
                        modules/default/weather/weather.js
                        modules/default/weatherforecast/weatherforecast.js
                        package-lock.json
                        run-start.sh

                        save/restore files selection = y
                        test merge result rc=‘’ if empty, no conflicts
                        executing merge, apply specified
                        merge result rc= 0n Updating a31546b…500147e
                        updating MagicMirror runtime, please wait

                        magicmirror@2.8.0 install /home/pi/MagicMirror
                        cd vendor && npm install
                        npm WARN optional SKIPPING OPTIONAL DEPENDENCY: fsevents@1.2.4 (node_modules/fsevents):
                        npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@1.2.4: wanted {“os”:“darwin”,“arch”:“any”} (current: {“os”:“linux”,“arch”:“arm”})

                        audited 220 packages in 53.517s
                        found 5 vulnerabilities (4 low, 1 high)
                        run npm audit fix to fix them, or npm audit for details

                        magicmirror@2.8.0 postinstall /home/pi/MagicMirror
                        sh installers/postinstall/postinstall.sh && npm run install-fonts

                        ^[[32mMagicMirror installation successful!

                        magicmirror@2.8.0 install-fonts /home/pi/MagicMirror
                        cd fonts && npm install

                        audited 1 package in 3.78s
                        found 0 vulnerabilities

                        npm WARN grunt-stylelint@0.11.1 requires a peer of stylelint@^10.0.0 but none is installed. You must install peer dependencies yourself.
                        audited 3573 packages in 310.648s

                        found 16 vulnerabilities (1 moderate, 15 high)

                        run npm audit fix to fix them, or npm audit for details
                        processing dependency changes for active modules with package.json files
                        processing for module MMM-WeeklySchedule please wait
                        audited 1 package in 3.297s
                        found 0 vulnerabilities
                        processing complete for module MMM-WeeklySchedule
                        processing for module MMM-MyPrayerTimes please wait
                        audited 3 packages in 4.211s
                        ound 0 vulnerabilities
                        processing complete for module MMM-MyPrayerTimes
                        restoring custom.css
                        we stashed a set of files that appear changed from the latest repo versions. you should review them

                        S 1 Reply Last reply Reply Quote 0
                        • S Do not disturb
                          sdetweil @bobwilliams
                          last edited by

                          This post is deleted!
                          S 1 Reply Last reply Reply Quote 0
                          • S Do not disturb
                            sdetweil @sdetweil
                            last edited by sdetweil

                            @bobwilliams do a git pull, and show the results…

                            looks like git lied on the return code of the merge…

                            merge result rc= 0n Updating a31546b…500147e
                            

                            lets try the output of this

                            git merge-tree `git merge-base master HEAD` HEAD master
                            

                            thanks

                            1 Reply Last reply Reply Quote 0
                            • B Offline
                              bobwilliams
                              last edited by

                              no change … I’m a simple user so don’t judge strictly

                              S 1 Reply Last reply Reply Quote 0
                              • S Do not disturb
                                sdetweil @bobwilliams
                                last edited by

                                @bobwilliams no judging… my script should help with this

                                just gotta understand it…
                                the output of those two commands is the same?

                                cd ~/MagicMirror
                                git merge-tree `git merge-base master HEAD` HEAD master
                                then 
                                git pull
                                
                                1 Reply Last reply Reply Quote 0
                                • B Offline
                                  bobwilliams
                                  last edited by

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

                                  git pull

                                  pi@raspberrypi:~ $ cd ~/MagicMirror
                                  pi@raspberrypi:~/MagicMirror $ git merge-tree git merge-base master HEAD HEAD master
                                  pi@raspberrypi:~/MagicMirror $ git pull
                                  Updating a31546b…500147e
                                  error: Your local changes to the following files would be overwritten by merge:
                                  modules/default/currentweather/currentweather.js
                                  modules/default/weather/weather.js
                                  modules/default/weatherforecast/weatherforecast.js
                                  package-lock.json
                                  run-start.sh
                                  Please commit your changes or stash them before you merge.
                                  Aborting
                                  pi@raspberrypi:~/MagicMirror $

                                  S 1 Reply Last reply Reply Quote 0
                                  • S Do not disturb
                                    sdetweil @bobwilliams
                                    last edited by sdetweil

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

                                    git merge-tree git merge-base master HEAD HEAD master

                                    there were two little backtics (left of 1 on us keyboard)
                                    replace backtic in the next command with that keyboard character

                                     git merge-tree backtic git merge-base master HEAD backtic HEAD master
                                    

                                    git pull results hm… we did a stash, why are those files still listed

                                    try some more

                                    git stash show
                                    git stash 
                                    git pull
                                    
                                    1 Reply Last reply Reply Quote 0
                                    • B Offline
                                      bobwilliams
                                      last edited by bobwilliams

                                      pi@raspberrypi:~/MagicMirror $ git merge-tree ~ git merge-base master HEAD ~ HEAD master
                                      usage: git merge-tree
                                      pi@raspberrypi:~/MagicMirror $ git stash show
                                      No stash entries found.
                                      pi@raspberrypi:~/MagicMirror $ git stash

                                      *** Please tell me who you are.

                                      Run

                                      git config --global user.email “you@example.com”
                                      git config --global user.name “Your Name”

                                      to set your account’s default identity.
                                      Omit --global to set the identity only in this repository.

                                      fatal: unable to auto-detect email address (got ‘pi@raspberrypi.(none)’)
                                      Cannot save the current index state
                                      pi@raspberrypi:~/MagicMirror $ git pull
                                      Updating a31546b…500147e
                                      error: Your local changes to the following files would be overwritten by merge:
                                      modules/default/currentweather/currentweather.js
                                      modules/default/weather/weather.js
                                      modules/default/weatherforecast/weatherforecast.js
                                      package-lock.json
                                      run-start.sh
                                      Please commit your changes or stash them before you merge.
                                      Aborting
                                      pi@raspberrypi:~/MagicMirror $

                                      S 2 Replies Last reply Reply Quote 0
                                      • S Do not disturb
                                        sdetweil @bobwilliams
                                        last edited by

                                        @bobwilliams great!!.. let me see what I can do about that…

                                        also ~ git , that is the tilde character, i need the single quote leaning to the left (backtic), same keyboard key for me, just not with shift

                                        B 1 Reply Last reply Reply Quote 0
                                        • S Do not disturb
                                          sdetweil @bobwilliams
                                          last edited by

                                          @bobwilliams lets try this

                                          git -c user.name=upgrade_script -c user.email=script@upgrade.com stash
                                          then 
                                          git stash show
                                          
                                          1 Reply Last reply Reply Quote 0
                                          • B Offline
                                            bobwilliams @sdetweil
                                            last edited by

                                            @sdetweil English is not my native language … sorry but I did not quite understand what to do

                                            S 1 Reply Last reply Reply Quote 0

                                            Hello! It looks like you're interested in this conversation, but you don't have an account yet.

                                            Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.

                                            With your input, this post could be even better 💗

                                            Register Login
                                            • 1
                                            • 2
                                            • 3
                                            • 4
                                            • 5
                                            • 6
                                            • 16
                                            • 17
                                            • 4 / 17
                                            • First post
                                              Last post
                                            Enjoying MagicMirror? Please consider a donation!
                                            MagicMirror created by Michael Teeuw.
                                            Forum managed by Sam, technical setup by Karsten.
                                            This forum is using NodeBB as its core | Contributors
                                            Contact | Privacy Policy