multi-notification plugin indicator support disabled and broken

[Hakan Erduman]

Binary package hint: claws-mail-extra-plugins

The Notification plugin in package claws-mail-multi-notifier (dependency of claws-mail-extra-plugins) has the libindicate-support disabled.

When enabling this via configure, it will register but crash claws-mail when clicked on inbox etc.

Where:
Maverick, amd64, under Xfce 4.8 from PPA with indicator panel plugin.

[Simon Steinbeiß (ochosi)]

has this bug been reported upstream already?

[Hakan Erduman (aquaherd)]

I think the libindicate modification in ubuntu caused this bug but I might be wrong.

[Simon Steinbeiß (ochosi)]

yes, but even if that is the case the bug would have to be fixed upstream i guess

[Hakan Erduman (aquaherd)]

now in natty, indicate-0.5 is installable, upstream references indicate version 0.3.

Where is upstream so that I can submit a patch there?

[Chad Wallace (cmdrwalrus)]

A bug just has been submitted to the claws-mail team here:

http://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=2423

[Simon Steinbeiß (ochosi)]

i linked to the upstream-bug in "assigned to" on this page before (just for the future).

anyway, it seems to be a duplicate of this bug (http://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=2180) and fixed in cvs since a week.

can anyone test it?

[Hakan Erduman (aquaherd)]

Indeed fixed upstream, but for it to compile in natty, you will need an extra patch that is not in bzr (attached);
Reason being rename of indicate to indicate-0.5; Don't know about oneiric, though.

[Simon Steinbeiß (ochosi)]

wondering who's taking care of claws in ubuntu, would be great if this patch would make it into a natty backport... (cause i assume that in oneiric things will be completely different again, at least until it gets released)

[Charlie Kravetz (cjkgeek)]

Due to the procedures defined for bug triage and work in Ubuntu, we do not have to go directly to the maintainers to have this worked. The bug has been tagged with a patch, as soon as possible a developer will check to insure the patch is correct. Then this can be applied to the distribution.

[Holger Berndt (berndth)]

It's not necessary anymore in this case, but for future reference: If it's not a distribution-issue but an upstream change in the library that causes these kinds of problems (as in this case with the .pc file rename), it would be nice to notify upstream of the library user (i.e. Claws Mail) instead of or in addition to fixing it in the distribution. This affects every consumer, not just Ubuntu.

On a related note, it would be nice if libindicate would provide Release Notes when doing backwards-incompatible changes. Not being a user of indicators myself, all I can do now is either review all changes in the 0.5 branch to see if I can safely support the new version (that's not gonna happen), or just support it and wait for bug reports in case it turns out to be incompatible.

[Simon Steinbeiß (ochosi)]

thanks for your response holger.
i completely agree with your points. unfortunately i can only speak for myself and try to improve this in my personal bugreports...

so this will be fixed in the next stable release of claws?

[Holger Berndt (berndth)]

I can't install libindicate 0.5 right now, but if somebody can confirm that it is compatible to 0.4 I'll add support for it to the build system.

[Holger Berndt (berndth)]

I had positive reports from users that were using libindicate 0.5, so it's now picked up upstream.

[Simon Steinbeiß (ochosi)]

thanks again! looking forward to seeing/testing that in the next claws-release.

[Ricardo Mones (mones)]

FWIW, Debian version 3.8.0-2 now enables libindicate, so syncing from there should fix this.

[Launchpad Janitor (janitor)]

This bug was fixed in the package claws-mail-extra-plugins - 3.8.0-2

---------------
claws-mail-extra-plugins (3.8.0-2) unstable; urgency=low

  * debian/rules
  - Simplify generation of manpages
  * debian/patches/multi-notifier00allow-libindicate-0.6.patch
  - Patch to allow detection of libindicate greater than 0.6
  * debian/control
  - Added build dependencies to:
    . Enable indicators in notification plugin (LP: #715257)
    . Enable canberra sound notifications in notification plugin
  - Update Standards-Version to 3.9.3 (no other changes)
  - Set c-m-e-p binary Section to metapackages
  - Fix Homepage to be a working URL
  * debian/copyright, debian/*.copyright
  - Update to Copyright Format 1.0
  - Fix Source fields to link plugins page

 -- Ricardo Mones <email address hidden> Fri, 04 May 2012 20:05:33 +0200