Re: [evolution-patches] Re: [Evolution-hackers] Remove duplicate e-util-marshal.list in evolution/widgets/misc
Patch committed into CVS HEAD. Both evolution/widgets/misc/e-util-marshal.list and evolution/e-util/e-marshal.list are removed. Harry On Wed, 2006-01-11 at 15:41 +0530, Srinivasa Ragavan wrote: > Looks fine to me. > > -Srini > On Tue, 2006-01-10 at 23:31 +0530, Harish Krishnaswamy wrote: > > This change looks fine to me. Actually, it must have been just oversight > > not having removed e-util-marshal.list from evolution/widgets/misc. > > > > Srini has been working on widgets/* more than any of us lately, though. > > Thoughts, Srini ? > > > > Thanks, > > Harish > > On Tue, 2006-01-10 at 18:47 +0800, simon.zheng wrote: > > > Hi all, > > > > > > Here is bug information. > > > http://bugzilla.gnome.org/show_bug.cgi?id=323529 > > > > > > We found another duplicate file "e-util-marshal.list". There's two > > > copies of e-util-marshal.list in and "evo/e-util" and > > > "evo/widgets/misc". They're 100% identical. What's more, we noticed the > > > other modules in evo/widgets, such as evo/widgets/table and > > > evo/widgets/text, use the copy in evo/e-util rather than their own > > > built-in copies. We think the one in evo/widgets/misc might be dropped. > > > > > > Attached the patch, pls review and comment. > > > > > > Thanks, > > > -Simon > > > ___ > > > Evolution-hackers mailing list > > > Evolution-hackers@gnome.org > > > http://mail.gnome.org/mailman/listinfo/evolution-hackers > > > > ___ > > Evolution-hackers mailing list > > Evolution-hackers@gnome.org > > http://mail.gnome.org/mailman/listinfo/evolution-hackers > > ___ > Evolution-patches mailing list > [EMAIL PROTECTED] > http://mail.gnome.org/mailman/listinfo/evolution-patches ___ Evolution-hackers mailing list Evolution-hackers@gnome.org http://mail.gnome.org/mailman/listinfo/evolution-hackers
Re: [Evolution-hackers] Remove duplicate e-util-marshal.list in evolution/widgets/misc
Looks fine to me. -Srini On Tue, 2006-01-10 at 23:31 +0530, Harish Krishnaswamy wrote: > This change looks fine to me. Actually, it must have been just oversight > not having removed e-util-marshal.list from evolution/widgets/misc. > > Srini has been working on widgets/* more than any of us lately, though. > Thoughts, Srini ? > > Thanks, > Harish > On Tue, 2006-01-10 at 18:47 +0800, simon.zheng wrote: > > Hi all, > > > > Here is bug information. > > http://bugzilla.gnome.org/show_bug.cgi?id=323529 > > > > We found another duplicate file "e-util-marshal.list". There's two > > copies of e-util-marshal.list in and "evo/e-util" and > > "evo/widgets/misc". They're 100% identical. What's more, we noticed the > > other modules in evo/widgets, such as evo/widgets/table and > > evo/widgets/text, use the copy in evo/e-util rather than their own > > built-in copies. We think the one in evo/widgets/misc might be dropped. > > > > Attached the patch, pls review and comment. > > > > Thanks, > > -Simon > > ___ > > Evolution-hackers mailing list > > Evolution-hackers@gnome.org > > http://mail.gnome.org/mailman/listinfo/evolution-hackers > > ___ > Evolution-hackers mailing list > Evolution-hackers@gnome.org > http://mail.gnome.org/mailman/listinfo/evolution-hackers ___ Evolution-hackers mailing list Evolution-hackers@gnome.org http://mail.gnome.org/mailman/listinfo/evolution-hackers
Re: [Evolution-hackers] Remove duplicate e-util-marshal.list in evolution/widgets/misc
This change looks fine to me. Actually, it must have been just oversight not having removed e-util-marshal.list from evolution/widgets/misc. Srini has been working on widgets/* more than any of us lately, though. Thoughts, Srini ? Thanks, Harish On Tue, 2006-01-10 at 18:47 +0800, simon.zheng wrote: > Hi all, > > Here is bug information. > http://bugzilla.gnome.org/show_bug.cgi?id=323529 > > We found another duplicate file "e-util-marshal.list". There's two > copies of e-util-marshal.list in and "evo/e-util" and > "evo/widgets/misc". They're 100% identical. What's more, we noticed the > other modules in evo/widgets, such as evo/widgets/table and > evo/widgets/text, use the copy in evo/e-util rather than their own > built-in copies. We think the one in evo/widgets/misc might be dropped. > > Attached the patch, pls review and comment. > > Thanks, > -Simon > ___ > Evolution-hackers mailing list > Evolution-hackers@gnome.org > http://mail.gnome.org/mailman/listinfo/evolution-hackers ___ Evolution-hackers mailing list Evolution-hackers@gnome.org http://mail.gnome.org/mailman/listinfo/evolution-hackers