db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Army <qoz...@gmail.com>
Subject Re: [jira] Updated: (DERBY-1758) Enable xmlSuite to run as part of derbyall for JVMs that embed the required external jars.
Date Fri, 27 Oct 2006 15:04:40 GMT
Bryan Pendleton wrote:

> Hi Army,
> I put a few notes into the Jira issue, but basically I didn't
> encounter any problems with your second patch.

Thank you for reviewing the patch, Bryan, and for the comments.  I noticed that 
you wrote "basically" here and that you said "in general" in your comments for 
the Jira.  Is there anything in particular about the patch that is prompting 
these qualifiers?  Feel free to let me know if there's something you'd like to 
see changed--I'm open to making the patch better where possible.

> Is this patch ready for commit? If so, I'll go ahead and commit it.

I haven't heard anything from anyone else about the patch, and so far as I know 
no one else is reviewing it.  So yes, I think this patch is ready for commit 
(thanks for offering!).

Note that if you do have some suggestions about how to make the patch better, we 
could still commit the current (_v2) patch and then I can post a follow-up to 
address your suggestions (similar to what was done for DERBY-1976).

Thanks, as always, for being so willing to review and commit.  I appreciate it!


View raw message