Ticket #217 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Compile problem on non-GNU systems (__va_copy)

Reported by: pramsey Owned by: mcayland
Priority: medium Milestone: PostGIS 1.4.0
Component: postgis Version: 1.4
Keywords: Cc: stefano.bonnin@…

Description

gcc -g -O2  -fPIC -DPIC  -Wall -Wmissing-prototypes shpopen.o dbfopen.o getopt.o shp2pgsql.o ../liblwgeom/liblwgeom.a  -lm -o shp2pgsql
../liblwgeom/liblwgeom.a(vsprintf.o)(.text+0x1c): In function `lw_vasprintf':
/usr/src/smit/postgis-1.4.0rc1/liblwgeom/vsprintf.c:150: undefined reference to `__va_copy'
gmake[1]: *** [shp2pgsql] Error 1
gmake[1]: Leaving directory `/usr/src/smit/postgis-1.4.0rc1/loader'
gmake: *** [loaderdumper] Error 2
gmake -C liblwgeom

Attachments

postgis-vsprintf.patch Download (0.8 KB) - added by pramsey 3 years ago.
Interim patch from mcayland

Change History

Changed 3 years ago by pramsey

Interim patch from mcayland

Changed 3 years ago by pramsey

OS/X does not complain about this patch.

Changed 3 years ago by pramsey

OpenSolaris? does not complain about this patch.

Changed 3 years ago by mcayland

Okay great. I seem to remember something in the back of my mind about some of the va_args stuff being different between 32-bit/64-bit platforms? Can someone check on a 32-bit system and make sure that print output is not garbled? (building --with-debug and running a few simple queries would be a good test, as I seem to remember that just building was not a good enough test to make sure it actually worked).

ATB,

Mark.

Changed 3 years ago by robe

  • status changed from new to closed
  • resolution set to fixed

Tried under 32-bit and compiled fine. Did a couple of queries as well and compared my results to pre-version results and seemed to come out the same.

tested new loader changes and seemed to work fine too.

Changed 3 years ago by robe

Forgot to mention my 32-bit tests were with binaries I compiled with mingW.

Changed 3 years ago by robe

  • status changed from closed to reopened
  • resolution fixed deleted

Oops forgot Mark hasn't committed this yet so can't close out until he commits to SVN. Mark can you commit and close out.

Changed 3 years ago by mcayland

  • status changed from reopened to closed
  • resolution set to fixed

Sorry - was out in meetings all day yesterday.

Applied to 1.4 branch and trunk as r4263 and r4264 respectively.

ATB,

Mark.

Note: See TracTickets for help on using tickets.