-----Original Message----- From: Bryce Harrington [mailto:bryce@...961...]
On Wed, Nov 21, 2007 at 11:43:57AM -0800, Ted Gould wrote:
On Wed, 2007-11-21 at 20:32 +0100,
J.B.C.Engelen@...1578... wrote:
I don't think we want a release now, so we can quickly address issues that surface. I think it is best to keep 2geom inside Inkscape code for 0.46, and have a good and nice 2geom
release for
0.47. This way we can profit from bleeding edge 2geom
which atm is
nicer than a release will be imho. I am fine by copying 2geom code to Inkscape for now :-)
We can for development, but I'd prefer not to for release.
Unless the
lib2geom folks are unwilling to release it, I'd prefer to
go that route.
The biggest advantage this gives us is that they can fix bugs independent of Inkscape. So, if the intersection routine
turns out to
be numerically unstable in a certain situation a spin of
lib2geom can
be done without having to do a release of Inkscape. And Inkscape users still get the benefit.
I definitely agree with Ted. There are several libraries copied into the tree that probably ought to be broken out where possible. Other projects like Xorg, OpenOffice, etc. demonstrate the many downsides to monolithic applications, and the benefits to both users and developers of proper modularization.
I also definitely agree that if there's bugs in lib2geom, rolling a new release of it would be much, much preferable to having to roll a new Inkscape.
Perhaps I thought too much from the "who is going to do it" perspective. I think you have convinced me that is it better to dynamically link 2geom. This has not been tested though, so probably needs work. Static linking was what I started out with in LPE development, but when LPE moved to trunk, we chose for having our own copy of 2geom because of frequent 2geom updates caused by progressing LPE dev. Now the situation is different. (static linking would make new lib2geom releases useless for released Inkscape right?)
However, I think there are still times when 2geom changes are so big to require some rewrite of Inkscape's 2geom-using code.
Regards, Johan