Bug 90768 fix

[Derek Atkins]

Ok, this is now committed to head.  Any chance you could write
up the tests as an automated test suite that we could
install into CVS?  That would be "really cool"  ;)

-derek

Matthew Vanecek <mevanecek at yahoo.com> writes:

> Here's the HEAD patch we talked about on #gnucash.  Fixes the
> transaction handling for upgrades.  Also made some changes to
> newtables.h.  I fixed the multibook_support() upgrade too, as it was
> broken.  The only upgrade currently that does not work is the 1.0.0 -->
> 1.1.1 upgrade.  That one doesn't work due to missing SQL for the
> gncGUIDcache table (which I asked about in a different message). 
> Perhaps we can remove that upgrade? 1.1.1 -> 1.2.1 -> 1.3.1 -> 1.4.1 ->
> 1.5.1 all work normally now, though, with this patch.  This patch is
> against HEAD.  I'll send the backport tomorrow after i've run the same
> tests on it.
> 
> On Tue, 2003-08-12 at 14:09, Derek Atkins wrote:
> > FYI, you left out the "newtables.h" in this patch.  I copied the
> > file from HEAD.
> > 
> 
> *sigh* Just getting over a fever from last week, really.  I probably
> assumed it would already be tagged or something silly like that...
> 
> > -derek
> > 
> > Matthew Vanecek <mevanecek at yahoo.com> writes:
> > 
> > > Here's the 1.8.x branch patch.  The internationalization is removed;
> > > I'll put it back in after 1.8.5 is released.
> > > 
> > > 
> > > On Sun, 2003-08-10 at 21:48, Derek Atkins wrote:
> > > > Matthew Vanecek <mevanecek at yahoo.com> writes:
> > > > 
> > > > > On Sun, 2003-08-10 at 15:21, Derek Atkins wrote:
> > > > > > Ok, I've committed this to HEAD.  I'd like to commit to 1.8, too..
> > > > > > I can try to backport to 1.8 myself without the translated string
> > > > > > changes, but I have no way to test it.  So, would you like to do
> > > > > > that backport or would you like me to try it?  I suppose I could
> > > > > > try the backport and send you a patch to test before I commit?
> > > > > > 
> > > > > > -derek
> > > > > 
> > > > > Either way would work.  I've tried modifying the patch to apply against
> > > > > 1.8, and there were some failures and offsets I haven't worked through
> > > > > yet.  Since I've already made a start, I can just go ahead and finish, I
> > > > > reckon.  Get something out by Tuesday night if all goes well.
> > > > 
> > > > Sounds fine to me.  Just leave the translated strings out of the 1.8
> > > > patch, which should be pretty easy to do.  I look forward to the
> > > > patch ;)
> > > > 
> > > > -derek
> > > -- 
> > > Matthew Vanecek
> > > perl -e 'print $i=pack(c5,(41*2),sqrt(7056),(unpack(c,H)-2),oct(115),10);'
> > > ********************************************************************************
> > > For 93 million miles, there is nothing between the sun and my shadow except me.
> > > I'm always getting in the way of something...
> > > 
> -- 
> Matthew Vanecek
> perl -e 'print $i=pack(c5,(41*2),sqrt(7056),(unpack(c,H)-2),oct(115),10);'
> ********************************************************************************
> For 93 million miles, there is nothing between the sun and my shadow except me.
> I'm always getting in the way of something...
> 

-- 
       Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
       Member, MIT Student Information Processing Board  (SIPB)
       URL: http://web.mit.edu/warlord/    PP-ASEL-IA     N1NWH
       warlord at MIT.EDU                        PGP key available