Bug 847513
| Summary: | Review Request: librasterlite - Support Raster Data Sources within a SpatiaLite DB | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Volker Fröhlich <volker27> |
| Component: | Package Review | Assignee: | Lucian Langa <lucilanga> |
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | lucilanga, notting, package-review |
| Target Milestone: | --- | Flags: | lucilanga:
fedora-review+
gwync: fedora-cvs+ |
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2012-12-20 16:25:35 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Volker Fröhlich
2012-08-12 13:09:56 UTC
I'm not entirely sure this package works properly. It seems to build fine, but when trying to run any CLI tool (without any parameters) I'm getting, besides the syntax help, strange warnings that doesn't look too good [1]. They seem to come and/or belong to a dependency package (spatialite) and of course it cannot create the required directories (because it doesn't have the necessary permissions. Fist of all you should investigate if this error comes from this package or not (and if other possibly packages that depend on spatialite are affected). I've also tried to run this as root and - warnings stopped of course - however an entirely new structure appeared on my filesystem /builddir/build/BUILD/.. etc This shouldn't have happened, should it ? [1] rasterlite_load did you forget to set the --db-path argument ? did you forget to set the --table-name argument ? did you forget to set the --dir-path OR --file-path argument ? usage: rasterlite_load ARGLIST ============================================================== -? or --help print this help message -t or --test test only - no actual action -v or --verbose verbose output -d or --db-path pathname the SpatiaLite db path -T or --table-name name DB table name -D or --dir dir_path the DIR path containing GeoTIFF files -f or --file file_name a single GeoTIFF file -s or --tile-size num [default = 512] -e or --epsg-code num [optional: EPSG code] -i or --image-type type [JPEG|PNG|GIF|TIFF] -q or --quality num [default = 75(JPEG)] profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/virtualtext/.libs/virtualtext.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/srsinit/.libs/srs_init.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/shapefiles/.libs/shapefiles.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/spatialite/.libs/virtualshape.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/spatialite/.libs/virtualnetwork.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/spatialite/.libs/virtualspatialindex.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/spatialite/.libs/virtualfdo.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/spatialite/.libs/virtualXL.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/spatialite/.libs/virtualdbf.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/spatialite/.libs/spatialite.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/spatialite/.libs/mbrcache.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_gml.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_kml.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_geoJSON.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_ewkt.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_vanuatu.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_wkt.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_wkb.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_transform.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_shape.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_relations.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_geoscvt.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_geometries.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_geodesic.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_endian.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiageo/.libs/gg_advanced.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiaexif/.libs/gaia_exif.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiaaux/.libs/gg_utf8.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/gaiaaux/.libs/gg_sqlaux.gcda:Skip profiling:/builddir:Cannot create directory profiling:/builddir/build/BUILD/libspatialite-3.1.0-RC2/src/.libs/version.gcda:Skip Thanks a lot for noticing this! I had libspatialite built with coverage profiling, which led to this problem. The current libspatialite build solves it. Thank you! Fixing libspatialite has fixed the warnings in this package and rasterlite cli programs appears to be working just fine now. I notice that package comes with a test suite. Unfortunately it seems non-functional (it segfaults now), with a little effort it can be made functional - missing files, error checking etc - perhaps you should contact upstream about fixing this and later add the appropriate section in the specfile. The lack of buildroot and cleaning up suggest you are not going to build this package for EL5, hence no pkgconfig requirement is necessary. This package is =========== APPROVED =========== Thank you for the review. I already contacted the authors about the failing test. Brad Hards, who wrote the tests, says, I can ignore the result of check_rastergen. It orginally segfaulted, because it couldn't find jpen50ref.jpg. It still fails afterwards, which is attributed to using libjpegturbo. I'll check in the package now. New Package SCM Request ======================= Package Name: librasterlite Short Description: Support Raster Data Sources within a SpatiaLite DB Owners: volter Branches: f17 f18 InitialCC: Git done (by process-git-requests). librasterlite-1.1c-2.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/librasterlite-1.1c-2.fc18 librasterlite-1.1c-2.fc18 has been pushed to the Fedora 18 testing repository. librasterlite-1.1c-2.fc18 has been pushed to the Fedora 18 stable repository. |