Bug report #4111

Build with current external spatialindex lib

Added by Volker Fröhlich over 12 years ago. Updated almost 12 years ago.

Status:Fixed for bounty
Priority:Normal
Assignee:Martin Dobias
Category:Build/Install
Affected QGIS version:master Regression?:No
Operating System: Easy fix?:No
Pull Request or Patch supplied:Yes Resolution:fixed
Crashes QGIS or corrupts data:No Copied to github as #:14093

Description

I modified the build system, created a find script and adapted the wrapper to the lib's new namespace. Martin said, he wanted to include the current lib, so the wrapper would fit both. I'd be happier to see you excluding this lib from the source tree, along with others.

The patch also contains the current FSF address. But shipping a copy of the GPLv3 would be even better (see #3789).

qgis_external_spatialindex_1.6.1.patch Magnifier (9.07 KB) Volker Fröhlich, 2011-07-22 06:35 AM

History

#1 Updated by Volker Fröhlich over 12 years ago

Where can I download the 10 Euro?

#2 Updated by Paolo Cavallini over 12 years ago

Please wait until it has been applied, then send me your details (PayPal or bank). Thanks.

#3 Updated by Volker Fröhlich over 12 years ago

The question was actually meant as a joke.

#4 Updated by Tim Sutton over 12 years ago

  • Target version set to Version 2.0.0
  • Category set to Build/Install
  • Status changed from Open to 7

Hi

I've applied your patch excluding the namespace changes in 356c87b. The namespace changes caused build errors for me here on ubuntu 11.04 and it wasnt clear to me that it was required for the actual issue - separability of spatialindex from the source tree if desired.

Regards

Tim

#5 Updated by Tim Sutton over 12 years ago

  • Status changed from 7 to Fixed for bounty

#6 Updated by Jürgen Fischer over 12 years ago

volter - wrote:

The question was actually meant as a joke.

careful, bad things can happen with 10€ ;) http://www.youtube.com/watch?v=ritEgrBlxwk

#7 Updated by Jürgen Fischer over 12 years ago

Tim Sutton wrote:

I've applied your patch excluding the namespace changes in 356c87b. The namespace changes caused build errors for me here on ubuntu 11.04 and it wasnt clear to me that it was required for the actual issue - separability of spatialindex from the source tree if desired.

c02421a completes it.

#8 Updated by Giovanni Manghi almost 12 years ago

  • Tracker changed from 4 to Bug report
  • Affected QGIS version set to master
  • Crashes QGIS or corrupts data set to No

#9 Updated by Giovanni Manghi almost 12 years ago

  • Resolution set to fixed

Also available in: Atom PDF