Opened 18 years ago

Closed 18 years ago

Last modified 18 years ago

#1560 closed defect (fixed)

Disable SDE raster support

Reported by: dmorissette Owned by: dmorissette
Priority: high Milestone:
Component: Input - Native SDE Support Version: unspecified
Severity: normal Keywords:
Cc:

Description

SDE raster support has been disabled in the 4.8 source, see email exchange below.

Last thing to do is removing DES raster stuff form the configure.in script.


-------- Original Message --------
Subject: Re: [UMN_MAPSERVER-DEV] Experimental SDE Raster support
Date: Thu, 8 Dec 2005 13:21:57 -0600
From: Howard Butler <hobu@IASTATE.EDU>
Reply-To: Howard Butler <hobu@IASTATE.EDU>
To: MAPSERVER-DEV@LISTS.UMN.EDU
References: <s398259d.046@co5.dnr.state.mn.us>

I just committed fixups of the layer virtualization issue I was 
having with SDE *and* switched off support for SDE rasters.  Could 
someone comfortable with configure.in magic remove the SDE raster 
configuration stuff?

Thanks

Howard

At 12:22 PM 12/8/2005, Steve Lime wrote:
>Off it is!
>
> >>> Howard Butler <hobu@IASTATE.EDU> 12/08/05 12:06 PM >>>
>My call is to turn it off for now unless someone else wants to be on
>the hook for it ;)
>
>Howard
>
>At 11:58 AM 12/8/2005, Frank Warmerdam wrote:
> >On 12/8/05, Howard Butler <hobu@iastate.edu> wrote:
> > > Folks,
> > >
> > > I would like to comment out the experimental SDE raster support that
> > > we have in mapsde.c for the time being.  As far as I know, I'm the
> > > only one to ever get it to work, it hasn't kept up with the
> > > connection pooling stuff we did, and its utility is quite limited in
> > > comparison to regular gdal-based raster support (projections,
> > > resampling, etc).
> >
> >Howard,
> >
> >Since you would be implicitly on the hook to support
> >it, I think it would be your call.
> >
> >Best regards,
> >--
> >---------------------------------------+--------------------------- 
> -----------
> >I set the clouds in motion - turn up   | Frank Warmerdam, 
> warmerdam@pobox.com
> >light and sound - activate the windows | http://pobox.com/~warmerdam
> >and watch the world go round - Rush    | Geospatial Programmer for Rent

Attachments (1)

bug1560-2.patch (2.9 KB ) - added by dmorissette 18 years ago.
Patch file showing the code that was taken out of configure.in and Makefile.in

Download all attachments as: .zip

Change History (4)

comment:1 by dmorissette, 18 years ago

Cc: hobu@… added
Owner: changed from hobu@… to dmorissette@…
I'll take care of the configure stuff

comment:2 by dmorissette, 18 years ago

Status: newassigned

comment:3 by dmorissette, 18 years ago

Resolution: fixed
Status: assignedclosed
All that the configure.in SDE Raster code did was adding -DUSE_SDERASTER. I am
not setup to test, but I think removing that code is safe. Howard, can please
confirm that normal SDE builds still work.

Committed in configure.in r1.133. I'll attach a patch file that contains the
code that I took out of configure.in and Makefile.in in case anyone ever wants
to re-enable that code.

by dmorissette, 18 years ago

Attachment: bug1560-2.patch added

Patch file showing the code that was taken out of configure.in and Makefile.in

Note: See TracTickets for help on using tickets.