Can't compile topology or raster anymore under mingw

[robe]

Something happened recently and I don't know what it is. Might be the work strk and dustymugs have been doing with building uninstall scripts for raster and topology, but I can't compile

--with-raster --with-topology anymore.

I first tried --with-topology and that doesn't work either, but leaving those out seems to still work. I think the last time I tried might have been around 12/20 - 12/22, but I think dusty had put in some extra cleaning logic since he said not everything was being removed that should have been so this might be a longer standing problem that is only exhibiting itself now that the clean process is cleaning more.

I tried wiping out my trunk svn and redownloading and that did not help.

Attached is my build log just trying to build --with-topology. I tried with 9.1 and have same issue.

[robe]

Hmm on closer inspection. Looks like it can't find perl. I had this issue with PostGIS and have a hack in place for postgis to replace ${PERL} with perl.

I'll try the same hack with raster and topology and see if that fixes the issue. Would be nice if I didn't need all these hacks. Is there a reason we can't just say perl? Isn't perl always global?

[robe]

Okay employing this hack:

sed 's,$(PERL),perl,g' <topology/Makefile >topology/Makefile2
mv topology/Makefile2 topology/Makefile

for topology fixed my topology compile issue, however similar hack for raster

sed 's,$(PERL),perl,g' <raster/rt_pg/Makefile >raster/rt_pg/Makefile2
mv raster/rt_pg/Makefile2 raster/rt_pg/Makefile

Did not. It doesn't seem to be complaining about not being able to find perl though so something else must be wrong.

See attached build just trying to build --with-raster

[robe]

Okay I think I figured out my issue with raster. strk it's this line you added to rt_pg/Makefile.in

http://trac.osgeo.org/postgis/browser/trunk/raster/rt_pg/Makefile.in#L92

# SQL objects deps here
$(SQL_OBJS): ../../postgis/sqldefines.h

Wouldn't that overwrite this?

$(SQL_OBJS): %.in: %.in.c
	$(CPP) -I../../postgis/ -I../../ -traditional-cpp $< | grep -v '^#' > $@

Should that other part go in ${OBJS} instead.

I realized I don't know quite what I'm talking about but getting rid of that line you added fixes my compile issue.

[darkblueb]

I am not able to build on Debian Wheezy today...

[strk]

Theoretically no, there should be no override, the latter should be only encoding an additional dependency. I guess it depends on make version. It is meant to force rebuild of the SQL file when changing sqldefines, which in turn is changed by ./configure

[strk]

Peter Clark found out: white space. Please try r8599

[robe]

okay still seems to work so I guess it fixes it.

[darkblueb]

yes - fixed for me, too