ThierryHFR / Big-Sur-StatusArea

macOS Big Sur "System Tray" look for Gnome Desktop
https://www.pling.com/p/1427290/
26 stars 4 forks source link

Latest version does not work. #79

Closed shadow5og closed 2 years ago

shadow5og commented 2 years ago

The latest shell version of the update does not work. Literally broke right after updating.

ThierryHFR commented 2 years ago

Hi @shadow5og Can you detail a little more your problem ? Version of the extension ? Version of gnome-shell ?

shadow5og commented 2 years ago

Extension version: 39

Gnome-shell version: 1:41.4-1

Literally got a notification this morning to upgrade the extension and it just blew up in my face. It upgraded and stopped working.

I went to the github page to try to build one from source but that failed too as the image

ThierryHFR commented 2 years ago

Hi @shadow5og, Here are the differences between version 37 and version 39. 1 - metadata.json version is incremented 2 - A correction on the display of the battery percentage (try to put back what was before and make me a return). If you are on the git version, it is not compatible with gnome-shell 41, it is for gnome-shell 42.

diff -ru 37/indicators/power.js 39/indicators/power.js
--- 37/indicators/power.js  2022-02-20 23:16:53.000000000 +0100
+++ 39/indicators/power.js  2022-03-26 01:59:13.000000000 +0100
@@ -77,7 +77,7 @@
         } else {
             this._power._indicator.show();
             this._percentageLabel.visible = this._power._desktopSettings.get_boolean("show-battery-percentage");
-            this._percentageLabel.clutter_text.set_markup('<span size="smaller">' + this._power._proxy.Percentage.toFixed(2) + " %</span>");
+            this._percentageLabel.clutter_text.set_markup('<span size="smaller">' + Math.round(this._power._proxy.Percentage) + " %</span>");
             this._separator.actor.show();
             this._label.show();
             this._settings.actor.show();
diff -ru 37/metadata.json 39/metadata.json
--- 37/metadata.json    2022-03-26 01:01:44.000000000 +0100
+++ 39/metadata.json    2022-03-26 01:01:44.000000000 +0100
@@ -8,5 +8,5 @@
   ],
   "url": "https://github.com/Ordissimo/Big-Sur-StatusArea",
   "uuid": "bigSur-StatusArea@ordissimo.com",
-  "version": 37
+  "version": 39
 }
ThierryHFR commented 2 years ago

also look at journalctl, any information is relevant ...

shadow5og commented 2 years ago

Please note that I reset to the commit 22a3c6470bddc12c1fe131b346c19bab2177a553 and built from there to test. It still did not work

Here are the requested logs:

Mar 29 16:49:52 devtool gnome-shell[6419]: JS ERROR: Extension bigSur-StatusArea@ordissimo.com: TypeError: this._client.get_devices is not a function _init@/home/nkuliguy0/.local/share/gnome-shell/extensions/bigSur-StatusArea@ordissimo.com/indicators/bluetooth.js:79:42 enable@/home/nkuliguy0/.local/share/gnome-shell/extensions/bigSur-StatusArea@ordissimo.com/extension.js:66:17 Mar 29 16:39:42 devtool gnome-shell[1202]: JS ERROR: Extension bigSur-StatusArea@ordissimo.com: TypeError: this._client.get_devices is not a function _init@/home/nkuliguy0/.local/share/gnome-shell/extensions/bigSur-StatusArea@ordissimo.com/indicators/bluetooth.js:79:42 enable@/home/nkuliguy0/.local/share/gnome-shell/extensions/bigSur-StatusArea@ordissimo.com/extension.js:66:17 Mar 29 16:30:45 devtool gnome-shell[1190002]: JS ERROR: Extension bigSur-StatusArea@ordissimo.com: Error: A different version was loaded previously. You need to log out f or changes to take effect. Mar 29 16:30:45 devtool gnome-shell[1192978]: extracting: /home/nkuliguy0/.local/share/gnome-shell/extensions/bigSur-StatusArea@ordissimo.com/metadata.json Mar 29 16:30:45 devtool gnome-shell[1192978]: inflating: /home/nkuliguy0/.local/share/gnome-shell/extensions/bigSur-StatusArea@ordissimo.com/icons/notification-new-symb olic.svg Mar 29 16:30:45 devtool gnome-shell[1192978]: inflating: /home/nkuliguy0/.local/share/gnome-shell/extensions/bigSur-StatusArea@ordissimo.com/icons/notification-symbolic .svg Mar 29 16:30:45 devtool gnome-shell[1192978]: inflating: /home/nkuliguy0/.local/share/gnome-shell/extensions/bigSur-StatusArea@ordissimo.com/icons/notifications-symboli

ThierryHFR commented 2 years ago

You are on the git version, it can not work, you must be on a gnome-shell 41 compatible version This function is not present in gnome-shell-41

this._client.get_devices
ThierryHFR commented 2 years ago

Give me a debug of the version 39 of the extension

shadow5og commented 2 years ago

Good Day, Brother/Sister

I had removed it before sending the logs.

Just to reiterate: I had been using version 39 which I got from https://extensions.gnome.org, it broke while I updated it via https://extensions.gnome.org. I then went to build from the git version after failing to come right with removing and re-adding the https://extensions.gnome.org version.

I just added it from https://extensions.gnome.org again and it works.

I suspect that I could not see the changes because I was not logging out after each change.

I do not know what broke it while it was updating. I'll have to check journalctl from this morning to find out.

Let me provide the journalctl till then and I'll close the issue.

shadow5og commented 2 years ago

Mar 29 07:35:06 devtool gnome-shell[96491]: JS ERROR: Extension bigSur-StatusArea@ordissimo.com: Error: A different version was loaded previously. You need to log out for changes to take effect.

From there, there is just information with the errors popping up again after I built from git.

This was the log made after it was updated in the morning on https://extensions.gnome.org. I did not log out to test and see. I am sorry for wasting your time. I am closing this issue now.

Thank you very much for your help.