Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 448366 - app-admin/conky - IMAP handling does not update message count properly
Summary: app-admin/conky - IMAP handling does not update message count properly
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Daniel Pielmeier
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2012-12-24 07:09 UTC by Georg Hopp
Modified: 2013-01-11 18:11 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
conky-1.9.0-update-when-message-count-decreases.patch (conky-1.9.0-update-when-message-count-decreases.patch,2.71 KB, patch)
2012-12-24 07:09 UTC, Georg Hopp
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Georg Hopp 2012-12-24 07:09:13 UTC
Created attachment 333184 [details, diff]
conky-1.9.0-update-when-message-count-decreases.patch

Hi,

i used the hollidays to play around with the conky imap command.
First thing i noticed was that i got no changes with imap_unseen
command either when i read or delete a mail.
After checking that my arguments where ok i looked at the code.
Well, what a mess.... :D in my eyes this needs to be totally rewritten.
And it seems that the conky authors came to the same point because
in current trunk the mail.c is replaced by a totally new mail.cc.

The mail.c is so messed up that i soon stopped trying to fix the issue but i
still wanted this to work and tried a workaround that is not very nice
but works for me.
I changed the condition that mades updates occur and forced conky
to update on any event that occures in IDLE mode. My imap server (dovecot)
accepts this but always complains that my client is broken because i
trigger a STATUS command on a folder i have already SELECTed.
OK, this was the same with the old code and now it seems to work at
least.

As the current conky trunk is to be completely rewritten, at least regarding
the mail handling i send my patch for 1.9.0 to you.
Can you please review it an consider adding it to the ebuild.

Thank you and merry christmas
  Georg Hopp
Comment 1 Markos Chandras (RETIRED) gentoo-dev 2012-12-24 17:16:57 UTC
Hi, 

changes in the source code should be reported to the upstream developers

http://conky.sourceforge.net/
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2012-12-25 15:56:21 UTC
(In reply to comment #1)
> Hi, 
> 
> changes in the source code should be reported to the upstream developers
> 
> http://conky.sourceforge.net/

metadata.xml says otherwise.
Comment 3 Daniel Pielmeier gentoo-dev 2013-01-02 18:02:56 UTC
I can not test this as I do not use the imap functionality. This patch should go upstream, preferably as well for the rewritten new version.
Comment 4 Georg Hopp 2013-01-03 09:45:59 UTC
OK, i will send the patch upstream and ask if it is still relevant for the new version. I guess it's not.
Comment 5 Georg Hopp 2013-01-10 08:18:58 UTC
OK, the patch is merge on the official conky 1.9.0 github branch.

https://github.com/brndnmtthws/conky/blob/1.9.0/src/mail.c

anything else i can do?
Comment 6 Daniel Pielmeier gentoo-dev 2013-01-10 08:41:22 UTC
(In reply to comment #5)
> OK, the patch is merge on the official conky 1.9.0 github branch.
> 
> https://github.com/brndnmtthws/conky/blob/1.9.0/src/mail.c
> 
> anything else i can do?

There is a 1.9.1 branch with more recent commits. I guess it should go there.
Comment 7 Daniel Pielmeier gentoo-dev 2013-01-11 18:11:21 UTC
+*conky-1.9.0-r2 (11 Jan 2013)
+
+  11 Jan 2013; Daniel Pielmeier <billie@gentoo.org> +conky-1.9.0-r2.ebuild:
+  Fix bug #448366. Thanks to Georg Hopp for the patch.

Should be fixed now. I have created the patch from the commit to the 1.9.1 branch. If anything went wrong please re-open the bug and report here.