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)
Change History (8)
comment:1 by , 14 years ago
comment:2 by , 14 years ago
Summary: | libgd < 2.0.30 compatibility → libgd < 2.0.30 compatibility (patch included) |
---|
follow-up: 4 comment:3 by , 14 years ago
Cc: | added |
---|---|
Milestone: | → 5.6.2 release |
Owner: | changed from | to
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.)
comment:4 by , 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 , 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 , 14 years ago
Attachment: | mapserver-5.4.2-gd_pre_2.0.30.patch added |
---|
Updated patch with "Index: ..." lines
comment:6 by , 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 , 14 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
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.
It seems this website is unable to display unified diffs properly. Please download the file to see all changes.