Re: [Evolution-hackers] Bug 201167: Complete implementation of category syncing
- From: Nathan Owens <pianocomp81 yahoo com>
- To: Not Zed <notzed ximian com>
- Cc: evolution-hackers lists ximian com
- Subject: Re: [Evolution-hackers] Bug 201167: Complete implementation of category syncing
- Date: Sat, 18 Jun 2005 22:12:16 -0700 (PDT)
On Sun, 2005-06-19 at 12:55:24 +0800, Not Zed wrote:
> On Fri, 2005-06-17 at 16:42 -0700, Nathan Owens wrote:
> > This is in regard to bug 201167
> > (http://bugzilla.gnome.org/show_bug.cgi?id=201167)
> > I submitted a patch for people to look at regarding todo-conduit category
> > syncing, and haven't heard anything. Should I assume people like it?
> Where did you submit? The bug system only? They should normally be
> sent to evolution-patches (with a link to the bug at least, dont need to
> include the attachment). Although having said that, nobody ever seems
> to care to review anything other than mail patches in a very timely
> manner anyway.
Actually, I posted it to evolution-hackers - at the time I didn't think it was
ready for submittal and wanted people to take a look at it for comments. It
must've gotten missed. After having used it for a week or so now, it seems
stable as-is (except for the NULL dereferencing which was fixed in the update).
> > Anyway, I found a bug in the code: it was possible for a NULL pointer to be
> > dereferenced. Attached is an updated patch against the latest CVS version.
> > Should I send this to evolution-patches as well?
> No need to this time, i think this should suffice for now, updates
> should go there though,
> It is quite a large patch - you will need copyright assignment (see the
> gnome evolution development pages i guess? sigh i wish we didn't need
> this red tape). And please read the HACKING file, the style of the new
> code is also incorrect, missing changelogs, etc.
I wondered about that. I'll send it in.
> > The category listeners still aren't working, but it's not my code causing
> > as none of the other category listeners employed in the tasks component
> > work either.
> Ok, hopefully someone who knows can review the logic. Otherwise give it
> a week and assuming other things above are done, poke again, and i'll
> give it a once over and probably accept it.
Ok. Thanks for responding.
Rekindle the Rivalries. Sign up for Fantasy Football
] [Thread Prev