Restarting unity shows two psensor icons in the unity bar

[Jonathan Almeida]

One icon is the normal indicator that functions normally as the indicator should (right in screenshot).

The second seems to be a system tray icon (left in screenshot). There is no right click options for it, but a single left click brings up the Psensor window.

To produce the bug, run "unity" in terminal. Another way to reproduce the bug is to plug in an external monitor, the second system tray icon appears in the main monitor only.

[Jean-Philippe Orsini (jfi)]

I don't reproduce with precise+psensor v0.6.2.16+unity(3d).

Could you please add information about your system (ubuntu and psensor + whether it is unity-2d or 'normal' unity) ?

Did you modify the default systray white-list? What is the output of 'gsettings get com.canonical.Unity.Panel systray-whitelist'? If psensor systray is 'white-listed' is may explain this report...

The code actually set to 'true' the visibility of the status widget during the initialization of psensor ui, so if unity decide to display systray widgets + application indicators. you got 2 icons. This behaviour is probably not a good idea and needs to be changed but I would like first to be sure about the scenario of this report.

Could you also delete ~/.psensor/log, run psensor with 'psensor -d 3' reproduce the issue, and finally attach the ~/.psensor/log file?

Thanks.

[Jean-Philippe Orsini (jfi)]

Was wrong, the systray widget is not initialized during startup if psensor is compiled with application indicator, but when the application indicator fallback function is called. The technical issue is that the unfallback function does not disable the systray widget. I guess that when unity is restarted fallback fct then unfallback fct are called. The ~/.psensor/log should allow to confirm it.

[Jonathan Almeida (jonalmeida)]

I'm running Ubuntu 11.10 (64 bit) running Unity 3D (normal Unity) and the systray-whitelist has not been modified at all. The output below shows this.
Output of 'gsettings get com.canonical.Unity.Panel systray-whitelist' : ['all', 'clementine']

The log file attached shows the unity_fallback starting on line 86 and 104.

I hope all this helps.

[Jean-Philippe Orsini (jfi)]

By default on a refresh oneiric installation, the systray-whitelist does not contain 'all' so the psensor systray icon is not visible. I guess the installation of a package has added 'all' in your case or you have forgot that you have set it manually.
Anyway, this usecase must be handled properly by psensor, a fix is on progress.

Thanks for the report and additionnal information.

[Jean-Philippe Orsini (jfi)]

Fix on mainstream trunk

[Ubuntu Foundations Team Bug Bot (crichton)]

The attachment "LP971098.diff" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

[Jean-Philippe Orsini (jfi)]

Hum hum, no need to review the attached patch as it is a mainstream trunk patch and cannot be added as-is to the older version of psensor used in precise.

@jonathan could you please install the daily trunk build of psensor and let me know if it fixes the issue?

If yes, I will attach the debdiff for precise.

[Jean-Philippe Orsini (jfi)]

The fix is included in Psensor since v0.7.0.1

[Jean-Philippe Orsini (jfi)]

For the log, a user has reported that this issue can be reproduced on XUbuntu 12.04 with the stock version of psensor (0.6.2.16) and that it is fixed with the 0.7.0.3 release.

[Launchpad Janitor (janitor)]

This bug was fixed in the package psensor - 0.7.0.3-1ubuntu1

---------------
psensor (0.7.0.3-1ubuntu1) raring; urgency=low

  * Merge from Debian unstable, remaining Ubuntu changes:
   - Build depend on nvidia-settings build-dep on i386 and amd64.
   - Build depend on libunity-dev and libappindicator3-dev for Unity support.

psensor (0.7.0.3-1) unstable; urgency=low

  * New upstream release
      + added gconf schema. (LP: #1026807)
      + fixed systray and application indicator visible at the
        same time. (LP: #971098)
      + fixed wrong background color, get the background color of
        the window and not the canvas widget. (LP: #973122)
      + removed hicolor/48x48/psensor_hot.png. (LP: #1029065)
      + added degree sign for celcius. (LP: #1021900)
      + fixed hide window on startup for gnome-shell. (Closes: #676361)
  * debian/control
      + added builddep to libatasmart-dev.
      + switched to debhelper 9.
  * debian/psensor.install
      + added gconf schema.
  * debian/rules
      + set GCONF_DISABLE_MAKEFILE_SCHEMA_INSTALL to 1 to avoid gconf
        schema installation during package build.
  * debian/compat
      + switched to debhelper 9, fixes hardening-no-relro and
        hardening-no-fortify-functions lintian warnings.
 -- Andrew Starr-Bochicchio <email address hidden> Sun, 27 Jan 2013 19:04:44 -0500