Opened 14 years ago

Closed 14 years ago

#3293 closed defect (fixed)

libgd < 2.0.30 compatibility (patch included)

Reported by: dpanech Owned by: aboudreault
Priority: normal Milestone: 5.6.2 release
Component: Build Problems Version: 5.4
Severity: normal Keywords: libgd
Cc: dmorissette

Description

The file mapagg.cpp uses the function "gdImageGetTrueColorPixel", which is missing in libgd 2.0.28 that I'm using. The attached patch fixes the issue.

See also: http://n2.nabble.com/MS-5-0-0-rc2-issue-tt1999432.html#a1999433

Attachments (1)

mapserver-5.4.2-gd_pre_2.0.30.patch (2.7 KB ) - added by dpanech 14 years ago.
Updated patch with "Index: ..." lines

Download all attachments as: .zip

Change History (8)

comment:1 by dpanech, 14 years ago

It seems this website is unable to display unified diffs properly. Please download the file to see all changes.

comment:2 by dpanech, 14 years ago

Summary: libgd < 2.0.30 compatibilitylibgd < 2.0.30 compatibility (patch included)

comment:3 by dmorissette, 14 years ago

Cc: dmorissette added
Milestone: 5.6.2 release
Owner: changed from dmorissette to aboudreault

Reassigned to Alan to review and apply the patch if appropriate.

(BTW, multi-file diffs normally work with this Trac instance. There must be something odd or missing in your patch files that confuses it.)

in reply to:  3 comment:4 by dpanech, 14 years ago

Replying to dmorissette:

Reassigned to Alan to review and apply the patch if appropriate.

(BTW, multi-file diffs normally work with this Trac instance. There must be something odd or missing in your patch files that confuses it.)

There are 2 files in the diff, it only shows the first one. Patch generated with GNU diff ("diff -u oldfile newfile"). The "patch" program works correctly with this patch. What am I doing wrong?

comment:5 by dmorissette, 14 years ago

My guess is that your patches are missing the multi-file header/separator that 'svn diff' would produce:

Index: path/to/file.c
===================================================================

Since Trac/diff issues are off-topic for this ticket, let's continue the discussion on the mapserver-dev list if necessary.

by dpanech, 14 years ago

Updated patch with "Index: ..." lines

comment:6 by aboudreault, 14 years ago

The patch doesn't seem to work when we specify the path where gd is installed. AC_CHECK_DECL seems to only look in the default include paths.

comment:7 by aboudreault, 14 years ago

Resolution: fixed
Status: newclosed

I've added a small hack with the CFLAGS/CXXFLAGS to fix the problem reported in the last comment. Thanks for the patch dpanech.

Fixed and committed in r10016.

Note: See TracTickets for help on using tickets.