On Thu, Aug 6, 2009 at 12:23 PM, Jamu Kakar <[email protected]> wrote: > Hi James, > > On Tue, Aug 4, 2009 at 11:30 PM, James Henstridge<[email protected]> > wrote: > > It would be nice to get lp:~jamesh/storm/bug-268151 merged. The C > > extension doesn't currently build under Windows, and this branch has > > been reported to fix that problem. > > I think this is important enough to hold up the release. I've added > the bug to the 0.15 milestone. Have you (or anybody) else tested > the patch on Windows? I've reviewed the code and tested it on > Ubuntu and it at least doesn't break anything here. I don't have a > Windows environment to test it in, or I'd do it myself. >
If you're holding up the release a few days, maybe it's worth considering getting the Oracle backend included. Jason Baker has gotten all the unit tests to pass with a branch taken from trunk a couple of months ago, and we are getting closer to putting it into production use at ZeOmega. We just need to take a new branch from trunk, add the Oracle backend (along with some additional tests and test modifications) and see if tests are still passing. Alternately, if you don't want to hold up the release of 0.15, hopefully we can still have it added to the trunk soon after. -- ZeOmega Open Minds' Open Solutions 3010 Gaylord Parkway, Ste. 210 Frisco TX, 75034 http://www.zeomega.com Brad Allen 214-618-9880 (ext. 8006)
-- storm mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/storm
