[ardour-dev] ardour2: build it

[Paul Davis]

On Tue, 2005-11-08 at 16:00 +0100, Karsten Wiese wrote:
> Am Montag, 7. November 2005 14:15 schrieb Karsten Wiese:
> > this makes it build !
> > At least on FC4 64Bit.
> > didn't run it yet though ;-)
> > as the patch before, this is intended as
> > porting-accelerator for 'code-monkeys'
> > to get usefull parts of and/or to test in an
> > executing environment.
> 
> With attached patch applied against ardour2 cvs,
> ardour.bin displays its first words on the console and then
> crashes somewhere in Editor::Editor.

Karsten, first let me say how much i appreciate the really significant
effort you put into this (series of) patches. however, as it stands i do
not want to apply this patch. 

the main reason for this is that the approach you took to handling
canvas events (which compromises a substantial fraction of the overall
patch) is the opposite of what i wanted to do (and have started doing).
because we are using a C++ canvas object now, there is no need for any
canvas event handlers that are directly connected to the C-level
signals, and then proxy to the "real" ones: Gnome::Canvas and the
compiler will take care of those for us.

we also have a new implementation of the new session dialog, making the
work on that redundant, and doug mcclain is working on the plugin
selector, with various improvements. applying this patch will make it
harder to integrate both of those efforts.

i will try to go through your patch and extract the small "individual"
changes that are not covered by my comments above. i think we've taken
somewhat different approaches to the gtk2 stuff - i want to do as much
as possible in the "right" way at the source code level, then work on
fixing the inevitable run time issues. i do understand your desire to
create something that just runs and then improve things from there.
thanks again for all the work, and i hope you will keep a steady flow of
other patches coming. the size_allocate() patch, for example, looks
perfectly fine.

--p