On Monday 24 June 2002 1:28 pm, Lars Gullik Bjønnes wrote: > Angus Leeming <[EMAIL PROTECTED]> writes: > | On Monday 24 June 2002 11:28 am, Lars Gullik Bjønnes wrote: > >> Angus Leeming <[EMAIL PROTECTED]> writes: > >> | All the forms/form_*.[Ch] files are now autogenerated. As such, you'll > >> | probably find them in your build tree, not your src tree. > >> > >> One note though... I do not agree to when this is generated, now you > >> generate upon compile, right? It should be done on autogen, so that a > >> source distribution does not require users to have fdesign installed. > | > | Can I convince you that you're wrong? Probably not ;-) Anyway here is why > | I prefer the current approach: > | > | 1. What is wrong with forcing the users to install fdesign? It's the > | appropriate compilation tool. (You insist on a C== compiler for .C > | code...) In effect, you are special-casing the xforms frontend. The qt > | frontend will require uic to be present. Ditto, gnome will require glade. > | > | 2. The makefile is used to rebuild object files when a src file is > | changed. In this case, the src file is the .fd file. When it's modified > | all is rebuilt correctly. If we adopt your proposal, then that is no > | longer the case. > > just make the form_about.c depend on form_about.fd then...
??? It does. Not only does all work perfectly for me, but the Makefile.am looks fine too. However, I'm no expert and you are. Could you have a look at Makefile.am and see if there are any glaring errors. FWIW, here's what happens for me: aleem@pneumon:lyx-> cd devel/src/frontends/xforms/forms aleem@pneumon:forms-> touch form_aboutlyx.fd aleem@pneumon:forms-> cd ../../../../../devel-build/src/frontends/xforms/forms aleem@pneumon:forms-> make /bin/sh ../../../../../devel/src/frontends/xforms/forms/fdfix.sh ../../../../../devel/src/frontends/xforms/forms/form_aboutlyx.fd source='form_aboutlyx.C' object='form_aboutlyx.lo' libtool=yes \ depfile='.deps/form_aboutlyx.Plo' tmpdepfile='.deps/form_aboutlyx.TPlo' \ depmode=tru64 /bin/sh ../../../../../devel/config/depcomp \ /bin/sh ../../../../libtool --mode=compile cxx -std strict_ansi -DHAVE_CONFIG_H -I. -I../../../../../devel/src/frontends/xforms/forms -I../../../../src -I../../../../../devel/src/frontends/xforms/forms/.. -I../../../../../devel/src -I/usr/local/include -nopure_cname -nocleanup -msg_display_number -w1 -ptr /usr/users/aleem/OTHERS_CODE/lyx/devel-build/lyx_cxx_repository -O2 -c -o form_aboutlyx.lo `test -f form_aboutlyx.C || echo '../../../../../devel/src/frontends/xforms/forms/'`form_aboutlyx.C cxx -std strict_ansi -DHAVE_CONFIG_H -I. -I../../../../../devel/src/frontends/xforms/forms -I../../../../src -I../../../../../devel/src/frontends/xforms/forms/.. -I../../../../../devel/src -I/usr/local/include -nopure_cname -nocleanup -msg_display_number -w1 -ptr /usr/users/aleem/OTHERS_CODE/lyx/devel-build/lyx_cxx_repository -O2 -c -MD form_aboutlyx.C echo timestamp > form_aboutlyx.lo
