While we are on the topic of makefiles, I'd like to suggest a change.
Currently, the noweb stuff is run through indent after notangle
generates the code. This changes the line numbering and thus totally
screws up GDB (or any other debugger). I didn't notice any way to
switch this off short of hacking the makefile.
Also, I've been looking into fixing the nasty problems which occur when
zooming while performing an action. It seems like a lot of the stuff in
x_event.c and a few other places needs to be changed to use world
coordinates instead of using screen coordinates like they do now. I'd
love to fix the problems and send in a patch, but I would like to check
if I'm going down the right path before I start changing stuff. Is my
understanding correct, or am I missing something? Any other ideas on
how this should be done?
-- Igor
Ales Hvezda wrote:
Hi Matthias,
[snip]
Here's a one-byte change I'd love to have in the toplevel Makefile,
which allows me to move the installdir (aka "prefix") around:
This one byte change has been accepted. Thanks.
Another small thing: currently gschem build doesn't use $prefix/include
but some system-wide includes (where in my case it finds outdated
geda-installs from my distro). In geda/devel/gschem/src/Makefile.am the
include line should read:
INCLUDES = -I$(prefix)/include -I$(top_srcdir)/intl
-I$(top_srcdir)/include @GSCHEM_CFLAGS@
same for linking:
gschem_LDADD = -L$(prefix)/lib @GSCHEM_LDFLAGS@ @LIBINTL@
Also accepted. I wonder if this should be propagated to the
other programs. I didn't do that for now, but it is something I
need to consider.
Then gEDA builds propperly. Patch attached.
Thanks for the patch!
-Ales
