[poppler] Re: [Patch] Annotations

[Marco Pesenti Gritti]

On 9/14/05, Brad Hards <bradh at frogmouth.net> wrote:
> 
> On Wednesday 14 September 2005 17:57 pm, Marco Pesenti Gritti wrote:
> > Hi,
> Please don't top quote.
> > how long is that going to take? Can we integrate annotations changes 
> first?
> It takes as long as it takes - there is a lot to do, but we have to do it
> properly. If you want it to go faster, please help out.


I meant if we could integrate the 3.01 annotations changes first. I have no 
idea if that's possible and I dont know how are you doing the merge 
(gradually or all in one go). That's why I _asked_.
I dont really understand the harsh reaction... if you ask to delay someone 
else patches it seem natural to provide at least rough time informations.

I don't think it is appropriate for your changes to go in first - as I wrote
> below, the merge is already hard enough. You can put in the glib only 
> parts -
> that isn't going to complicate the merge.


As I said I'm fine with delaying this a bit. But I need to have a rough idea 
on when that's going to happen because I have work blocking on it. 

I've also got stuff waiting for the merge - AES encryption and embedded
> documents support. I don't want to sound harsh, but your change doesn't 
> seem
> so much more important than those.


That's subjective I guess. We are working on new features in evince that 
requires annotation changes, so it's higher priority for us.

Anyway, if you are going to oppose to any changes until the merge is done, 
please discuss it and provide more details of what is going on. If the team 
decision is to delay changes until the merge is completed I'm fine with it. 
But let's announce it, so that people don't waste time and can help out with 
the merge instead. If that was done before, we would have avoided this 
unpleasant discussion.

Thanks,
Marco
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.freedesktop.org/archives/poppler/attachments/20050914/ff182657/attachment.html