Bug report #4111

Build with current external spatialindex lib

Added by Volker Fröhlich about 8 years ago. Updated over 7 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 about 8 years ago

Where can I download the 10 Euro?

#2 Updated by Paolo Cavallini about 8 years ago

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

#3 Updated by Volker Fröhlich almost 8 years ago

The question was actually meant as a joke.

#4 Updated by Tim Sutton almost 8 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 almost 8 years ago

  • Status changed from 7 to Fixed for bounty

#6 Updated by Jürgen Fischer almost 8 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 almost 8 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 over 7 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 over 7 years ago

  • Resolution set to fixed

Also available in: Atom PDF