Comment 20 for bug 337394
Revision history for this message
| Ted Gould (ted) wrote Re: [Dx-team] [Bug 337394] Re: Notification entries are not appended unless a replace is used | #20 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
demo site
(Get the code!)
On Wed, 2009-04-08 at 15:00 +0000, Mark Shuttleworth wrote:
> MPT, if we had done "append" as a new operation in its own right, we
> would not have the API issue you describe.
Doing that would have required changing libnotify, along with all the
other bindings that actually don't use libnotify but reimplement the
spec. I think that it was a strong selling point for us to be able to
say that we required no changes in libnotify, because we were just doing
the same thing, even if we did a twist on it.
Also, our application patches would have required conditional compiles
based on detecting our version of the libnotify library and would have
been much less likely to get upstream.
Even with the confusion, and I agree with mpt's analysis, I still think
we made the better choice between those two options.