Re: [Evolution-hackers] Removal of implementation details from public API, any breakages?

2007-05-21 Thread Srinivasa Ragavan
On Mon, 2007-05-21 at 12:38 +0100, Ross Burton wrote:
> On Mon, 2007-05-21 at 11:33 +, Srinivasa Ragavan wrote:
> > >From the current discussion, it looks like we are safe. Can we do
> > something like this for this release before we dung them out?
> > 
> > #ifdef E_D_S_DEPRECATED
> > #include 
> > #endif
> 
> The patches consist of removing functions or headers from the install,
> these cannot be deprecated because they are still used by EDS itself.

Hmm. Fine. Just go ahead then :)  

-Srini.
> 
> I don't think there needs to be any notice: the headers and functions
> are implementation details of libebook and libecal, and are not possible
> to use outside of the implementation of libebook/libecal.
> 
> Ross

___
Evolution-hackers mailing list
Evolution-hackers@gnome.org
http://mail.gnome.org/mailman/listinfo/evolution-hackers


Re: [Evolution-hackers] Removal of implementation details from public API, any breakages?

2007-05-21 Thread Matthew Barnes
On Mon, 2007-05-21 at 11:33 +, Srinivasa Ragavan wrote:
> #ifdef E_D_S_DEPRECATED
> #include 
> #endif

Just FYI, EDS_DISABLE_DEPRECATED is what Gtk-Doc looks for.

Matthew Barnes

___
Evolution-hackers mailing list
Evolution-hackers@gnome.org
http://mail.gnome.org/mailman/listinfo/evolution-hackers


Re: [Evolution-hackers] Removal of implementation details from public API, any breakages?

2007-05-21 Thread Ross Burton
On Mon, 2007-05-21 at 11:33 +, Srinivasa Ragavan wrote:
> >From the current discussion, it looks like we are safe. Can we do
> something like this for this release before we dung them out?
> 
> #ifdef E_D_S_DEPRECATED
> #include 
> #endif

The patches consist of removing functions or headers from the install,
these cannot be deprecated because they are still used by EDS itself.

I don't think there needs to be any notice: the headers and functions
are implementation details of libebook and libecal, and are not possible
to use outside of the implementation of libebook/libecal.

Ross
-- 
Ross Burton mail: [EMAIL PROTECTED]
  jabber: [EMAIL PROTECTED]
 www: http://www.burtonini.com./
 PGP Fingerprint: 1A21 F5B0 D8D0 CFE3 81D4 E25A 2D09 E447 D0B4 33DF



signature.asc
Description: This is a digitally signed message part
___
Evolution-hackers mailing list
Evolution-hackers@gnome.org
http://mail.gnome.org/mailman/listinfo/evolution-hackers


Re: [Evolution-hackers] Removal of implementation details from public API, any breakages?

2007-05-21 Thread Srinivasa Ragavan
Ross,

>From the current discussion, it looks like we are safe. Can we do
something like this for this release before we dung them out?

#ifdef E_D_S_DEPRECATED
#include 
#endif

-Srini.

On Mon, 2007-05-21 at 10:32 +0100, Ross Burton wrote:
> On Mon, 2007-05-21 at 12:15 +0530, Srinivasa Ragavan wrote:
> > It will be great if you can mail the details on the address book stuff
> > as well. I would like the libebook clients like OOo, etc to comment on
> > this. 
> 
> The addressbook changes are very similar:
> 
> - e_book_view_new() is not public
> - EBookListener and EBookViewListener are not public
> 
> As before, these are not usable outside of libedata-book, so clients
> should not be aware of their existence.
> 
> I've had a quick look at the Zimbra Evolution code and it appears to not
> use these either.
> 
> Ross

___
Evolution-hackers mailing list
Evolution-hackers@gnome.org
http://mail.gnome.org/mailman/listinfo/evolution-hackers


Re: [Evolution-hackers] Removal of implementation details from public API, any breakages?

2007-05-21 Thread Ross Burton
On Mon, 2007-05-21 at 12:15 +0530, Srinivasa Ragavan wrote:
> It will be great if you can mail the details on the address book stuff
> as well. I would like the libebook clients like OOo, etc to comment on
> this. 

The addressbook changes are very similar:

- e_book_view_new() is not public
- EBookListener and EBookViewListener are not public

As before, these are not usable outside of libedata-book, so clients
should not be aware of their existence.

I've had a quick look at the Zimbra Evolution code and it appears to not
use these either.

Ross
-- 
Ross Burton mail: [EMAIL PROTECTED]
  jabber: [EMAIL PROTECTED]
 www: http://www.burtonini.com./
 PGP Fingerprint: 1A21 F5B0 D8D0 CFE3 81D4 E25A 2D09 E447 D0B4 33DF



signature.asc
Description: This is a digitally signed message part
___
Evolution-hackers mailing list
Evolution-hackers@gnome.org
http://mail.gnome.org/mailman/listinfo/evolution-hackers


Re: [Evolution-hackers] Removal of implementation details from public API, any breakages?

2007-05-21 Thread Jules Colding
On Sun, 2007-05-20 at 11:29 +0100, Ross Burton wrote:
> Hi,
> 
> Last week I committed a patch to libebook, and want to commit a patch to
> libecal[1], which removes private functions and types from the installed
> headers.  This has several consequences:
> 
> - e_cal_view_new() is removed
> - ECalListener is removed
> - ECalViewListener is removed
> 
> I believe that nobody is using these functions apart from libecal
> itself, so this removal is safe.  However, I'd appreciate it if anyone
> writing advanced clients to EDS (like Zimbra or Brutas) remove their
> currently installed headers, apply the patch, and rebuild.

Brutus is safe.

Thanks for the notice,
  jules


___
Evolution-hackers mailing list
Evolution-hackers@gnome.org
http://mail.gnome.org/mailman/listinfo/evolution-hackers


Re: [Evolution-hackers] Removal of implementation details from public API, any breakages?

2007-05-20 Thread Srinivasa Ragavan
Ross,

It will be great if you can mail the details on the address book stuff
as well. I would like the libebook clients like OOo, etc to comment on
this. 

-Srini.

On Sun, 2007-05-20 at 11:29 +0100, Ross Burton wrote:
> Hi,
> 
> Last week I committed a patch to libebook, and want to commit a patch to
> libecal[1], which removes private functions and types from the installed
> headers.  This has several consequences:
> 
> - e_cal_view_new() is removed
> - ECalListener is removed
> - ECalViewListener is removed
> 
> I believe that nobody is using these functions apart from libecal
> itself, so this removal is safe.  However, I'd appreciate it if anyone
> writing advanced clients to EDS (like Zimbra or Brutas) remove their
> currently installed headers, apply the patch, and rebuild.
> 
> Thanks,
> Ross
> 
> [1] http://bugzilla.gnome.org/show_bug.cgi?id=438727
> ___
> 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] Removal of implementation details from public API, any breakages?

2007-05-20 Thread Ross Burton
Hi,

Last week I committed a patch to libebook, and want to commit a patch to
libecal[1], which removes private functions and types from the installed
headers.  This has several consequences:

- e_cal_view_new() is removed
- ECalListener is removed
- ECalViewListener is removed

I believe that nobody is using these functions apart from libecal
itself, so this removal is safe.  However, I'd appreciate it if anyone
writing advanced clients to EDS (like Zimbra or Brutas) remove their
currently installed headers, apply the patch, and rebuild.

Thanks,
Ross

[1] http://bugzilla.gnome.org/show_bug.cgi?id=438727
-- 
Ross Burton mail: [EMAIL PROTECTED]
  jabber: [EMAIL PROTECTED]
 www: http://www.burtonini.com./
 PGP Fingerprint: 1A21 F5B0 D8D0 CFE3 81D4 E25A 2D09 E447 D0B4 33DF



signature.asc
Description: This is a digitally signed message part
___
Evolution-hackers mailing list
Evolution-hackers@gnome.org
http://mail.gnome.org/mailman/listinfo/evolution-hackers