[Mapserver-users] GDAL / GIF misunderstanding?

[Frank Warmerdam]

This is a multi-part message in MIME format.
--------------090601060905000508070705
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit

Daniel Morissette wrote:
> Frank Warmerdam wrote:
> 
>>I could provide a patch for mapraster.c if you are interested.  Should this
>>be rolled into a future 3.6.x?  Note, I did already ensure that GDAL would
>>be used if TIFF, PNG or JPEG were not configured, but for some reason the
>>same fix was not applied for GIF.
>>
> 
> 
> Frank,
> 
> If the same logic is already applied and working for the JPEG and PNG
> formats in 3.6.x then I think it would be a good idea to fix the GIF
> case the same way in the 3.6.x branch so if we ever release 3.6.5 then
> this fix will be part of it.

Daniel / Doug,

OK, at this point I have to admit that I am not very CVS-branch competent.
I have checked out 3.6.4 just fine and built it, and prepared the patch.
Do I just commit, and it automatically goes into the right branch?

I have attached the very simple patch (untested).

Best regards,

-- 
---------------------------------------+--------------------------------------
I set the clouds in motion - turn up   | Frank Warmerdam, warmerdam at pobox.com
light and sound - activate the windows | http://pobox.com/~warmerdam
and watch the world go round - Rush    | Geospatial Programmer for Rent


--------------090601060905000508070705
Content-Type: text/plain;
 name="patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
 filename="patch"

Index: mapraster.c
===================================================================
RCS file: /data2/cvsroot/mapserver/mapraster.c,v
retrieving revision 1.54.2.2
diff -r1.54.2.2 mapraster.c
1774a1775
> #if !defined(USE_GDAL) || defined(USE_GD_GIF)
1830a1832
> #endif /* !defined(USE_GDAL) || defined(USE_GD_GIF) */

--------------090601060905000508070705--