Bug#844194: InfoPane.qml:54:22: Unable to assign [undefined] to int

2016-11-17 Thread Helge Kreutzmann
Hello Maximiliano,
On Thu, Nov 17, 2016 at 10:30:00AM +0100, Maximiliano Curia wrote:
> El 2016-11-16 a las 20:39 +0100, Helge Kreutzmann escribió:
> >Hello Maximiliano, On Wed, Nov 16, 2016 at 10:25:38AM +0100,
> >Maximiliano Curia wrote:
> >>El 2016-11-15 a las 19:41 +0100, Helge Kreutzmann escribió:
> >Since Sep. 21 I repeatedly see the following error (?) message in my 
> >logs:
> >Sep 21 21:42:48 samd sddm-greeter[17492]:
> >file:///usr/share/sddm/themes/breeze/components/InfoPane.qml:54:22:
> >Unable to assign [undefined] to int Everything (except the
> >PID and the date) is always the same.
> 
> That file
> (/usr/share/sddm/themes/breeze/components/InfoPane.qml) is
> not included in our packages, where did it come from?
> 
> >>>I never manipulate anything within /usr directly, only via apt/dpkg.
> 
> >>Ah, ok, it was present in the 5.7.4 version. The report was made
> >>listing sddm-theme-breeze 5.8.2 hence the confusion.
> 
> >>Anyway, the message is still present if you restart sddm
> >>(remember to close your graphical session before doing this)?
> 
> >Ok, I'm still processing my logs, so yes, you are right, this
> >particular message is no longer present since October 13th.
> >However, this file is still present -- if it is no longer packaged,
> >should'nd it be removed? (Maybe a bug in dpkg?)
> 
> I wouldn't bet on a dpkg error (ocaml's razor and trying not to break
> the sharks free [1] :) ).
> 
> Could you provide the output of a "ls -l" of
> /usr/share/sddm/themes/breeze/components/ ?

-rw-r--r-- 1 root root 2360 Okt 18 13:47 ActionButton.qml
drwxr-xr-x 2 root root 4096 Okt 28 15:37 artwork
-rw-r--r-- 1 root root 1686 Okt 18 13:47 Battery.qml
-rw-r--r-- 1 root root 1455 Okt 18 13:47 Clock.qml
-rw-r--r-- 1 root root 2166 Okt 18 13:47 KeyboardLayoutButton.qml
-rw-r--r-- 1 root root 3776 Okt 18 13:47 SessionManagementScreen.qml
-rw-r--r-- 1 root root 5403 Okt 18 13:47 UserDelegate.qml
-rw-r--r-- 1 root root 3011 Okt 18 13:47 UserList.qml

> Would it be possible that the file was restored from a backup? Are

No

> you using a special filesystem? Has your system crashed while

Ext4

> installing packages? Do you have fsyncs disabled in any way?

No. 

I did not set up anything regarding fscycns, any output you'll need?

> >>>Nov 14 22:28:23 samd sddm-greeter[3210]: 
> >>>file:///usr/share/sddm/themes/breeze/components/Battery.qml:39:18: Unable 
> >>>to assign [undefined] to int
> 
> >>>And this file is clearly in the current package:
> >>>sddm-theme-breeze: /usr/share/sddm/themes/breeze/components/Battery.qml
> 
> >This message is present since October 13th.
> 
> Ok, is there any battery in the system? (I guess not). I'm attaching
> a patch for the Battery.qml file that takes into account undefined
> values. Can you test this?

No, this machine is a old fashioned tower system, no battary (besides the
clock one, of course).

> To apply the patch you'll need to run as root:
> cd /usr/share/sddm/themes/breeze ; patch -p3 < 
> /tmp/WIP_fix_undefined_to_int_log.patch

Worked. I'll monitor the logs in the next days and report back.

> This is asuming you saved the patch in /tmp. You can also use patch
> -R to revert the patch, or just reinstall the sddm-theme-breeze
> package.

Thanks & Greetings

  Helge



-- 
  Dr. Helge Kreutzmann deb...@helgefjell.de
   Dipl.-Phys.   http://www.helgefjell.de/debian.php
64bit GNU powered gpg signed mail preferred
   Help keep free software "libre": http://www.ffii.de/


signature.asc
Description: Digital signature


Bug#844194: InfoPane.qml:54:22: Unable to assign [undefined] to int

2016-11-17 Thread Maximiliano Curia

¡Hola Helge!

El 2016-11-16 a las 20:39 +0100, Helge Kreutzmann escribió:
Hello Maximiliano, 
On Wed, Nov 16, 2016 at 10:25:38AM +0100, Maximiliano Curia wrote:

El 2016-11-15 a las 19:41 +0100, Helge Kreutzmann escribió:

Since Sep. 21 I repeatedly see the following error (?) message in my logs:
Sep 21 21:42:48 samd sddm-greeter[17492]:
file:///usr/share/sddm/themes/breeze/components/InfoPane.qml:54:22:
Unable to assign [undefined] to int Everything (except the PID 
and the date) is always the same.


That file (/usr/share/sddm/themes/breeze/components/InfoPane.qml) 
is not included in our packages, where did it come from?



I never manipulate anything within /usr directly, only via apt/dpkg.


Ah, ok, it was present in the 5.7.4 version. The report was made 
listing sddm-theme-breeze 5.8.2 hence the confusion.


Anyway, the message is still present if you restart sddm (remember to 
close your graphical session before doing this)?


Ok, I'm still processing my logs, so yes, you are right, this 
particular message is no longer present since October 13th. However, 
this file is still present -- if it is no longer packaged, should'nd 
it be removed? (Maybe a bug in dpkg?)


I wouldn't bet on a dpkg error (ocaml's razor and trying not to break the 
sharks free [1] :) ).


Could you provide the output of a "ls -l" of 
/usr/share/sddm/themes/breeze/components/ ?


Would it be possible that the file was restored from a backup? Are you using a 
special filesystem? Has your system crashed while installing packages? Do you 
have fsyncs disabled in any way?



Nov 14 22:28:23 samd sddm-greeter[3210]: 
file:///usr/share/sddm/themes/breeze/components/Battery.qml:39:18: Unable to 
assign [undefined] to int



And this file is clearly in the current package:
sddm-theme-breeze: /usr/share/sddm/themes/breeze/components/Battery.qml



This message is present since October 13th.


Ok, is there any battery in the system? (I guess not). I'm attaching a patch 
for the Battery.qml file that takes into account undefined values. Can you 
test this?


To apply the patch you'll need to run as root:
cd /usr/share/sddm/themes/breeze ; patch -p3 < 
/tmp/WIP_fix_undefined_to_int_log.patch

This is asuming you saved the patch in /tmp. You can also use patch -R to 
revert the patch, or just reinstall the sddm-theme-breeze package.


Happy hacking,
[1]: https://xkcd.com/349/

--
"It's a well-known fact that computing devices such as the abacus were
invented thousands of years ago. But it's not well known that the first use of
common computer protocol occurred in the Old Testament. This, of course, was
when Moses aborted the Egyptians' process with a control-sea."
-- Tom Galloway (recs.arts.comics, February 1992)
Saludos /\/\ /\ >< `/
commit 1ea89f08cce14e7e4017d6d3dc5b8cc22e1e902d
Author: Maximiliano Curia 
Date:   Thu Nov 17 10:01:49 2016 +0100

WIP: fix "Unable to assign [undefined] to int" log

The sddm breeze theme battery icon is trying to access the battery
percent. In a batteryless system it produces the syslog message:
Nov 14 22:28:23 samd sddm-greeter[3210]: file:///usr/share/sddm/themes/breeze/components/Battery.qml:39:18: Unable to assign [undefined] to int

Based on https://gist.github.com/Zren/4e5709d842965227088f6e1d3fd42016

Debian-Bug: https://bugs.debian.org/844194

diff --git a/lookandfeel/contents/components/Battery.qml b/lookandfeel/contents/components/Battery.qml
index c4a94eb..c1bbf73 100644
--- a/lookandfeel/contents/components/Battery.qml
+++ b/lookandfeel/contents/components/Battery.qml
@@ -33,10 +33,19 @@ Row {
 connectedSources: ["Battery", "AC Adapter"]
 }
 
+function getBatteryData(key, def) {
+var value = pmSource.data.Battery[key]
+if (typeof value === "undefined") {
+return def;
+} else {
+return value;
+}
+}
+
 PW.BatteryIcon {
 id: battery
-hasBattery: true
-percent: pmSource.data["Battery"]["Percent"]
+hasBattery: getBatteryData("Has Battery", false)
+percent: getBatteryData("Percent", 100)
 pluggedIn: pmSource.data["AC Adapter"] ? pmSource.data["AC Adapter"]["Plugged in"] : false
 
 height: batteryLabel.height


signature.asc
Description: PGP signature


Bug#844194: InfoPane.qml:54:22: Unable to assign [undefined] to int

2016-11-16 Thread Helge Kreutzmann
Hello Maximiliano,
On Wed, Nov 16, 2016 at 10:25:38AM +0100, Maximiliano Curia wrote:
> El 2016-11-15 a las 19:41 +0100, Helge Kreutzmann escribió:
> >>>Since Sep. 21 I repeatedly see the following error (?) message in my logs:
> >>>Sep 21 21:42:48 samd sddm-greeter[17492]:
> >>>file:///usr/share/sddm/themes/breeze/components/InfoPane.qml:54:22:
> >>>Unable to assign [undefined] to int Everything (except the PID
> >>>and the date) is always the same.
> 
> >>That file (/usr/share/sddm/themes/breeze/components/InfoPane.qml)
> >>is not included in our packages, where did it come from?
> 
> >I never manipulate anything within /usr directly, only via apt/dpkg.
> 
> Ah, ok, it was present in the 5.7.4 version. The report was made
> listing sddm-theme-breeze 5.8.2 hence the confusion.
> 
> Anyway, the message is still present if you restart sddm (remember to
> close your graphical session before doing this)?

Ok, I'm still processing my logs, so yes, you are right, this
particular message is no longer present since October 13th. However, 
this file is still present -- if it is no longer packaged, should'nd
it be removed? (Maybe a bug in dpkg?)

> >Nov 14 22:28:23 samd sddm-greeter[3210]: 
> >file:///usr/share/sddm/themes/breeze/components/Battery.qml:39:18: Unable to 
> >assign [undefined] to int
> 
> >And this file is clearly in the current package:
> >sddm-theme-breeze: /usr/share/sddm/themes/breeze/components/Battery.qml

This message is present since October 13th.

> Mmh, what's the version of your currently installed plasma-workspace?

ii  plasma-workspace   4:5.8.2-1
amd64Plasma Workspace for KF5

Greetings

   Helge
-- 
  Dr. Helge Kreutzmann deb...@helgefjell.de
   Dipl.-Phys.   http://www.helgefjell.de/debian.php
64bit GNU powered gpg signed mail preferred
   Help keep free software "libre": http://www.ffii.de/


signature.asc
Description: Digital signature


Bug#844194: InfoPane.qml:54:22: Unable to assign [undefined] to int

2016-11-16 Thread Maximiliano Curia

¡Hola Helge!

El 2016-11-15 a las 19:41 +0100, Helge Kreutzmann escribió:

Since Sep. 21 I repeatedly see the following error (?) message in my logs:
Sep 21 21:42:48 samd sddm-greeter[17492]:
file:///usr/share/sddm/themes/breeze/components/InfoPane.qml:54:22:
Unable to assign [undefined] to int Everything (except the PID and 
the date) is always the same.


That file (/usr/share/sddm/themes/breeze/components/InfoPane.qml) is 
not included in our packages, where did it come from?



I never manipulate anything within /usr directly, only via apt/dpkg.


Ah, ok, it was present in the 5.7.4 version. The report was made listing 
sddm-theme-breeze 5.8.2 hence the confusion.


Anyway, the message is still present if you restart sddm (remember to close 
your graphical session before doing this)?



Nov 14 22:28:23 samd sddm-greeter[3210]: 
file:///usr/share/sddm/themes/breeze/components/Battery.qml:39:18: Unable to 
assign [undefined] to int



And this file is clearly in the current package:
sddm-theme-breeze: /usr/share/sddm/themes/breeze/components/Battery.qml


Mmh, what's the version of your currently installed plasma-workspace? It 
could be that we are missing a versioned depend in sddm-theme-breeze against 
plasma-workspace.


Happy hacking,
--
"By definition, when you are investigating the unknown, you do not know what
you will find"
-- The Ultimate Principle
Saludos /\/\ /\ >< `/


signature.asc
Description: PGP signature


Bug#844194: InfoPane.qml:54:22: Unable to assign [undefined] to int

2016-11-15 Thread Helge Kreutzmann
Hello Maximiliano,
On Tue, Nov 15, 2016 at 09:42:08AM +0100, Maximiliano Curia wrote:
> El 2016-11-13 a las 09:50 +0100, Helge Kreutzmann escribió:
> >Package: sddm Version: 0.13.0-1 Severity: minor
> 
> >Since Sep. 21 I repeatedly see the following error (?) message in my logs:
> >Sep 21 21:42:48 samd sddm-greeter[17492]:
> >file:///usr/share/sddm/themes/breeze/components/InfoPane.qml:54:22:
> >Unable to assign [undefined] to int Everything (except the PID and
> >the date) is always the same.
> 
> That file (/usr/share/sddm/themes/breeze/components/InfoPane.qml) is
> not included in our packages, where did it come from?

I never manipulate anything within /usr directly, only via apt/dpkg. 

So let's see:
It appeared on September 21st (the occurence above is the first one).
On that day, according to "history.log" the follwing happened:
Start-Date: 2016-09-21  19:46:11
Commandline: apt-get dist-upgrade
Install: libtaskmanager6:amd64 (4:5.7.4-1, automatic), libscim8v5:amd64 
(1.4.17-1, automatic), liblegacytaskmanager5:amd64 (4:5.7.4-1, automatic), 
libpackagekitqt5-0:amd64 (0.9.5-3, automatic), libkf5activitiesstats1:amd64 
(5.25.0-1, automatic)
Upgrade: milou:amd64 (4:5.7.0-1, 4:5.7.4-1), plasma-workspace:amd64 
(4:5.6.5.1-1, 4:5.7.4-1), libgles2-mesa:amd64 (12.0.2-1, 12.0.3-1), 
libplasma-geolocation-interface5:amd64 (4:5.6.5.1-1, 4:5.7.4-1), 
kde-cli-tools:amd64 (4:5.7.0-1, 4:5.7.4-1), libegl1-mesa-dev:amd64 (12.0.2-1, 
12.0.3-1), breeze-cursor-theme:amd64 (4:5.7.0-1, 4:5.7.4-1), 
libgles1-mesa:amd64 (12.0.2-1, 12.0.3-1), plasma-dataengines-addons:amd64 
(4:5.6.5-1, 4:5.7.4-1), libprocesscore7:amd64 (4:5.7.0-1, 4:5.7.4-1), 
libglapi-mesa:amd64 (12.0.2-1, 12.0.3-1), kwin-style-breeze:amd64 (4:5.7.0-1, 
4:5.7.4-1), kwin-data:amd64 (4:5.7.0-1, 4:5.7.4-1), 
plasma-wallpapers-addons:amd64 (4:5.6.5-1, 4:5.7.4-1), kde-style-breeze:amd64 
(4:5.7.0-1, 4:5.7.4-1), libxatracker2:amd64 (12.0.2-1, 12.0.3-1), breeze:amd64 
(4:5.7.0-1, 4:5.7.4-1), hyphen-de:amd64 (1:5.2.1-1, 1:5.2.1-2), 
libegl1-mesa:amd64 (12.0.2-1, 12.0.3-1), plasma-desktop:amd64 (4:5.6.5-1, 
4:5.7.4-2), kactivitymanagerd:amd64 (5.7.0-1, 5.7.4-1), 
plasma-desktop-data:amd64 (4:5.6.5-1, 4:5.7.4-2), libksgrd7:amd64 (4:5.7.0-1, 
4:5.7.4-1), libgbm1:amd64 (12.0.2-1, 12.0.3-1), libkworkspace5-5:amd64 
(4:5.6.5.1-1, 4:5.7.4-1), kde-cli-tools-data:amd64 (4:5.7.0-1, 4:5.7.4-1), 
plasma-runners-addons:amd64 (4:5.6.5-1, 4:5.7.4-1), oxygen-sounds:amd64 
(4:5.7.0-1, 4:5.7.4-1), libkscreenlocker5:amd64 (5.7.0-1, 5.7.4-1), 
sddm-theme-breeze:amd64 (4:5.6.5.1-1, 4:5.7.4-1), libkwinglutils8:amd64 
(4:5.7.0-1, 4:5.7.4-1), libwayland-egl1-mesa:amd64 (12.0.2-1, 12.0.3-1), 
libkf5screen-bin:amd64 (4:5.7.0-1, 4:5.7.4-1), libkf5screen7:amd64 (4:5.7.0-1, 
4:5.7.4-1), plasma-integration:amd64 (5.7.0-1, 5.7.4-1), 
xserver-xorg-video-ati:amd64 (1:7.7.0-1, 1:7.7.1-1), libgl1-mesa-dri:amd64 
(12.0.2-1, 12.0.3-1), kwin-x11:amd64 (4:5.7.0-1, 4:5.7.4-1), 
libkwin4-effect-builtins1:amd64 (4:5.7.0-1, 4:5.7.4-1), libgl1-mesa-glx:amd64 
(12.0.2-1, 12.0.3-1), xserver-xorg-video-radeon:amd64 (1:7.7.0-1, 1:7.7.1-1), 
libkwinxrenderutils8:amd64 (4:5.7.0-1, 4:5.7.4-1), libkfontinstui5:amd64 
(4:5.6.5-1, 4:5.7.4-2), libkfontinst5:amd64 (4:5.6.5-1, 4:5.7.4-2), 
polkit-kde-agent-1:amd64 (4:5.7.0-1, 4:5.7.4-1), libkf5sysguard-data:amd64 
(4:5.7.0-1, 4:5.7.4-1), kdeplasma-addons-data:amd64 (4:5.6.5-1, 4:5.7.4-1), 
libprocessui7:amd64 (4:5.7.0-1, 4:5.7.4-1), libkwineffects8:amd64 (4:5.7.0-1, 
4:5.7.4-1), kwin-common:amd64 (4:5.7.0-1, 4:5.7.4-1), 
qml-module-qtquick-controls-styles-breeze:amd64 (4:5.7.0-1, 4:5.7.4-1), 
plasma-widgets-addons:amd64 (4:5.6.5-1, 4:5.7.4-1), mesa-va-drivers:amd64 
(12.0.2-1, 12.0.3-1), libweather-ion7:amd64 (4:5.6.5.1-1, 4:5.7.4-1)
End-Date: 2016-09-21  19:46:35

I.e. "plasma-workspace:amd64" was updated. 

I checked, and yes, in the sddm-theme-breeze_4%3a5.7.4-1_amd64.deb I
still have, that file is indeed included. Maybe the package forgot to
delete this file on an upgrade?

Also, in more recent log entries I see similar messages for other
files as well:
Nov 14 22:28:23 samd sddm-greeter[3210]: 
file:///usr/share/sddm/themes/breeze/components/Battery.qml:39:18: Unable to 
assign [undefined] to int

And this file is clearly in the current package:
sddm-theme-breeze: /usr/share/sddm/themes/breeze/components/Battery.qml

Greetings

Helge


-- 
  Dr. Helge Kreutzmann deb...@helgefjell.de
   Dipl.-Phys.   http://www.helgefjell.de/debian.php
64bit GNU powered gpg signed mail preferred
   Help keep free software "libre": http://www.ffii.de/


signature.asc
Description: Digital signature


Bug#844194: InfoPane.qml:54:22: Unable to assign [undefined] to int

2016-11-15 Thread Maximiliano Curia

¡Hola Helge!

El 2016-11-13 a las 09:50 +0100, Helge Kreutzmann escribió:
Package: sddm 
Version: 0.13.0-1 
Severity: minor



Since Sep. 21 I repeatedly see the following error (?) message in my logs:
Sep 21 21:42:48 samd sddm-greeter[17492]: file:///usr/share/sddm/themes/breeze/components/InfoPane.qml:54:22: Unable to assign [undefined] to int 
Everything (except the PID and the date) is always the same.


That file (/usr/share/sddm/themes/breeze/components/InfoPane.qml) is not 
included in our packages, where did it come from?



I did not notice anything visually (but then I might have missed it).


Happy hacking,
--
"Seek simplicity, and distrust it." -- Whitehead's Rule
Saludos /\/\ /\ >< `/


signature.asc
Description: PGP signature


Bug#844194: InfoPane.qml:54:22: Unable to assign [undefined] to int

2016-11-13 Thread Helge Kreutzmann
Package: sddm
Version: 0.13.0-1
Severity: minor

Since Sep. 21 I repeatedly see the following error (?) message in my logs:
Sep 21 21:42:48 samd sddm-greeter[17492]: 
file:///usr/share/sddm/themes/breeze/components/InfoPane.qml:54:22: Unable to 
assign [undefined] to int
Everything (except the PID and the date) is always the same.

I did not notice anything visually (but then I might have missed it).


-- System Information:
Debian Release: stretch/sid
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 4.5.5samd.10-grsec (SMP w/4 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8) (ignored: LC_ALL 
set to de_DE.UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages sddm depends on:
ii  adduser 3.115
ii  debconf [debconf-2.0]   1.5.59
ii  libc6   2.24-5
ii  libgcc1 1:6.2.0-10
ii  libpam0g1.1.8-3.3
ii  libqt5core5a5.6.1+dfsg-3+b1
ii  libqt5dbus5 5.6.1+dfsg-3+b1
ii  libqt5gui5  5.6.1+dfsg-3+b1
ii  libqt5network5  5.6.1+dfsg-3+b1
ii  libqt5qml5  5.6.1-11
ii  libqt5quick55.6.1-11
ii  libstdc++6  6.2.0-10
ii  libsystemd0 231-9
ii  libxcb-xkb1 1.12-1
ii  libxcb1 1.12-1
ii  qml-module-qtquick2 5.6.1-11
ii  sddm-theme-breeze [sddm-theme]  4:5.8.2-1

Versions of packages sddm recommends:
ii  libpam-systemd  231-9

Versions of packages sddm suggests:
ii  libpam-kwallet5  5.8.2-1

-- debconf information:
* shared/default-x-display-manager: sddm
  sddm/daemon_name: /usr/bin/sddm

-- 
  Dr. Helge Kreutzmann deb...@helgefjell.de
   Dipl.-Phys.   http://www.helgefjell.de/debian.php
64bit GNU powered gpg signed mail preferred
   Help keep free software "libre": http://www.ffii.de/


signature.asc
Description: Digital signature