Re: [Evolution-hackers] Remove duplicate e-util-marshal.list in evolution/widgets/misc

2006-01-11 Thread Srinivasa Ragavan
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

2006-01-10 Thread Harish Krishnaswamy
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