D. Malcom. I was wondering if you could help me test the patch included in the following bug report: http://bugzilla.gnome.org/show_bug.cgi?id=211058 The patch should be valid against the latest evolution CVS and possibly older version. I have never had much luck trying to compile CVS evolution. I may not be reading the code correctly, so if the patch doesn't allow you to catch bogus (future) dates properly, I may have to reread the code (much more slowly this time) and try again. If the patch works or doesn't please, if you wouldn't mind help me get a working version included in CVS evolution, I would appreciate it.
Well, I stink with glade. "Ago" is coded as a label in filter.glade. I have never quite figured out glade nor how to attach it to code. Is there any RedHat/Fedora Core person who understands glade. The changes needed are the label needs to become a drop down or some other selection method. The options need to be "in the future" and "ago" if I understand the code correctly (in that it creates a string that goes to the file I patched). Then whatever the glade parsing/handling/callback code changes need to be so that it fetches the value from the selection/dropdown/whatever-method-chosen. Of course, there are the maximum values in the code/patch I did. Somehow these need to be brought in and parsed correctly for "in the future" options. I am sorry, I just don't do a lot of gui programming and am still trying to learn. Thank you much for any help anyone can provide.
MY NOTES - CURRENTLY TRYING MYSELF Ok, my patch is completely bogus. Label8 in filter.glade needs to become a selection with "ago" and "in the future" or similar labels with appropriate translations. All the timespans with negative entries (i.e. mine) need to be removed. All ago/in the future references in timespans need to be removed and replaced with the selection mentioned above, with appropriate translations. set_values must be modified to handle negative values by setting the lable8 replacement (option_past_future) correctly and to not set the spin to a negative value as it is setup to ignore it. get_values needs to be updated to use option_past_future button_clicked needs to have stuff added for option_past_future I think that is my total notes. Going to try to fetch the latest srpm and try to build from that.
Matthew Barnes, I hope you don't mind that I am trying to add you as a cc here. Below are some patches I am trying that appear to work. They supercede my other patch (as it was completely broken).
Created attachment 131697 [details] Glade part of patch I am still working on the code part of this (having trouble compiling), but the glade part is working it seems with the current evolution. I hope to have the code diffed in about 40 minutes.
Does http://www.go-evolution.org/Compiling_Evolution_from_CVS help? I'd suggest taking this to one of these mailing lists: - evolution-hackers (intended for high-level discussion of development) - evolution-patches (intended for patch review, can be quite terse) - gnome-love (intended as a friendly place for people new to gnome development looking to get involved) Reassigning to Matt
Matt, I had several false starts. Lessons learned: rpmbuild's shortcircuit is not the friend it appears to be... too bad. This patch should be very clean other than one hack, but it is due to another long-standing-hack already in the code. Please, apply this to rawhide. I will work on getting it upstreamed. One and a half questions: Is this a big enough change for me to add my name as an author? Is it big enough to add my own copyright (under the GPL of course due to my desires and the fact it is already such)? This patch supercedes all others from me relating to this issue (that in 211058 that you included, and the glade one here).
Created attachment 131711 [details] Believed and tested good to add the check against future dates This patch obsoletes all others from me. It includes .h .c and .glade changes necessary. Please, allow people to test this in rawhide, I will try to get it upstreamed.
Evolution currently still has the old and broken patch. I am rebuilding today's rawhide evolution changing the old patch for my new one, the patch does work.
Mr. Barnes, Evolution in rawhide still has a broken version (the initial try) of my patch. Can you please remove it and replace it with the last one submitted here. It works perfectly. Thank you. (Still trying to get it upstreamed.)
Done - evolution-2.7.3-10 Leaving this bug open until upstream accepts the patch.
This patch seems to have fallen out of the latest evolution builds. Is there a reason for that?
Fedora Core 5 and Fedora Core 6 are, as we're sure you've noticed, no longer test releases. We're cleaning up the bug database and making sure important bug reports filed against these test releases don't get lost. It would be helpful if you could test this issue with a released version of Fedora or with the latest development / test release. Thanks for your help and for your patience. [This is a bulk message for all open FC5/FC6 test release bugs. I'm adding myself to the CC list for each bug, so I'll see any comments you make after this and do my best to make sure every issue gets proper attention.]
This patch appears to have fallen out some how or no longer works. I will try to verify which, but I believe it fell out. Still valid.
Thanks.
I believe my patch or something similar has been upstreamed. Either way, I am no longer maintaining the patch. Maybe this should be closed?
Yes, thanks for the reminder. I believe your patch was accepted upstream.