After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 579360 - Problem with vfolders in latest svn trunk version
Problem with vfolders in latest svn trunk version
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.26.x (obsolete)
Other All
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
evolution[vfolders]
: 579273 579547 581090 581720 583012 583031 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2009-04-17 23:28 UTC by Thomas
Modified: 2009-05-22 04:41 UTC
See Also:
GNOME target: ---
GNOME version: 2.23/2.24


Attachments
Avoid vfolder hang (2.62 KB, patch)
2009-04-20 05:19 UTC, Akhil Laddha
none Details | Review
corrected patch (2.42 KB, patch)
2009-04-20 22:28 UTC, Thomas
committed Details | Review

Description Thomas 2009-04-17 23:28:49 UTC
Please describe the problem:
Here is another problem, which happens in the latest svn trunk version and which I could reproduce, but doesn't happen in a version dated 2009-03-31. See also issue 578565.

Steps to reproduce:
1. I create a search folder named tags:bewerbung, where I define 2 conditions which both have to be true.
 a) label "bewerbung" has to be true
 b) label "erledigt" has to be false.
2. the vfolder gets created and all mails in question are shown as expected.
3. Then I go to the folder containing the mails and remove the label erledigt from 2 mails.
4. I go to the search folder and the 2 mails are not shown - as expected.
5. I go back to the actual folder and reset the label "erledigt".
6. Going back again to the vfolder, the status bar tells me that evolution tries to save the vfolder and appears to be hung in that thread.


Actual results:


Expected results:


Does this happen every time?
Again, the version dated from 2009-03-31 does not show the problem!!

Other information:

Comment 1 Thomas 2009-04-17 23:30:57 UTC
Correction:

3. Then I go to the folder containing the mails and set the label erledigt
for 2 mails.
5. I go back to the actual folder and remove the label "erledigt".
Comment 2 Thomas 2009-04-17 23:32:57 UTC
Also got the following assertions. Might be useful in finding the problem:

(evolution:9694): filter-CRITICAL **: filter_option_add: assertion `find_option(fo, value) == NULL' failed

(evolution:9694): filter-CRITICAL **: filter_option_add: assertion `find_option(fo, value) == NULL' failed
Comment 3 Akhil Laddha 2009-04-20 05:08:34 UTC
*** Bug 579547 has been marked as a duplicate of this bug. ***
Comment 4 Akhil Laddha 2009-04-20 05:19:20 UTC
Created attachment 132941 [details] [review]
Avoid vfolder hang 

I saw similar hang while trying out unmatched search folder. Srini has provided the patch which  seems to work for me. Attaching the patch on behalf of him.
Comment 5 Srinivasa Ragavan 2009-04-20 08:22:20 UTC
Not exactly avoid. But Fix :-)


The first hunk is duplicate, ignore it.
Comment 6 Thomas 2009-04-20 22:27:18 UTC
Got a few compile errors, but otherwise the patch works... so far.



n -MT camel-vee-folder.lo -MD -MP -MF .deps/camel-vee-folder.Tpo -c -o camel-vee-folder.lo ../../../evolution-data-server/camel/camel-vee-folder.c
../../../evolution-data-server/camel/camel-vee-folder.c: In Funktion »camel_vee_folder_add_folder«:
../../../evolution-data-server/camel/camel-vee-folder.c:180: Warnung: Variable »cache« wird nicht verwendet
../../../evolution-data-server/camel/camel-vee-folder.c: In Funktion »vee_sync«:
../../../evolution-data-server/camel/camel-vee-folder.c:621: Warnung: Initialisierung von inkompatiblem Zeigertyp
../../../evolution-data-server/camel/camel-vee-folder.c: In Funktion »vee_search_by_uids«:
../../../evolution-data-server/camel/camel-vee-folder.c:796: Warnung: Übergabe des Arguments 2 von »g_ptr_array_add« entfernt Kennzeichner von Zeiger-Ziel-Typ
../../../evolution-data-server/camel/camel-vee-folder.c: In Funktion »folder_changed_add_uid«:
../../../evolution-data-server/camel/camel-vee-folder.c:1344: Fehler: »CamelVeeFolder« hat kein Element namens »full_name«
../../../evolution-data-server/camel/camel-vee-folder.c:1344: Warnung: Übergabe des Arguments 3 von »camel_db_delete_uid_from_vfolder_transaction« entfernt Kennzeichner von Zeiger-Ziel-Typ
../../../evolution-data-server/camel/camel-vee-folder.c:1345: Fehler: »CamelVeeFolder« hat kein Element namens »summary«
../../../evolution-data-server/camel/camel-vee-folder.c: In Funktion »folder_changed_remove_uid«:
../../../evolution-data-server/camel/camel-vee-folder.c:1397: Fehler: »CamelVeeFolder« hat kein Element namens »full_name«
../../../evolution-data-server/camel/camel-vee-folder.c:1398: Fehler: »CamelVeeFolder« hat kein Element namens »summary«
Comment 7 Thomas 2009-04-20 22:28:28 UTC
Created attachment 132994 [details] [review]
corrected patch

because of compile errors
Comment 8 David Ronis 2009-04-27 03:35:01 UTC
This hasn't been committed in the git repositories
Comment 9 Srinivasa Ragavan 2009-04-27 06:01:53 UTC
Would be committed to stable/master
Comment 11 Akhil Laddha 2009-05-05 12:01:23 UTC
*** Bug 581090 has been marked as a duplicate of this bug. ***
Comment 12 Thomas M. 2009-05-07 14:03:42 UTC
*** Bug 579273 has been marked as a duplicate of this bug. ***
Comment 13 Stanislav Brabec 2009-05-12 11:33:24 UTC
*** Bug 581720 has been marked as a duplicate of this bug. ***
Comment 14 Akhil Laddha 2009-05-19 04:17:42 UTC
*** Bug 583031 has been marked as a duplicate of this bug. ***
Comment 15 Akhil Laddha 2009-05-20 04:00:37 UTC
*** Bug 583012 has been marked as a duplicate of this bug. ***
Comment 16 Marcus Zurhorst 2009-05-21 16:37:48 UTC
In 583012, you wrote that this one here is a duplicate of my bug report.
Any mentioned, that this one is fixed.


So, is this already fixed in 2.26.2, or will I have to wait until 2.6.28 to see this annoying thing resolved.
With this information, I can ask the Mandriva maintainers to update the Evolution package as soon as possible.


Thanks!
Comment 17 Akhil Laddha 2009-05-22 04:41:34 UTC
> 
> So, is this already fixed in 2.26.2, or will I have to wait until 2.6.28 to see
> this annoying thing resolved.

Yup, this fix is available in 2.26.2 which got released on May 20.