Bug report #4111
Build with current external spatialindex lib
| 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 
(9.07 KB)
History
#1
Updated by Volker Fröhlich over 13 years ago
Where can I download the 10 Euro?
#2
Updated by Paolo Cavallini over 13 years ago
Please wait until it has been applied, then send me your details (PayPal or bank). Thanks.
#3
Updated by Volker Fröhlich about 13 years ago
The question was actually meant as a joke.
#4
Updated by Tim Sutton about 13 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 about 13 years ago
- Status changed from 7 to Fixed for bounty
#6
Updated by Jürgen Fischer about 13 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 about 13 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 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 over 12 years ago
- Resolution set to fixed