#3815: compression: libzstd should be a default config and a requirement if ZSTD is default compression method --------------------------+----------------------------------- Reporter: mlennert | Owner: grass-dev@… Type: defect | Status: new Priority: normal | Milestone: 7.8.0 Component: Raster | Version: svn-trunk Resolution: | Keywords: compression zstd zlib CPU: Unspecified | Platform: Unspecified --------------------------+-----------------------------------
Comment (by mlennert): Replying to [comment:1 mmetz]: > Replying to [ticket:3815 mlennert]: > > I just came upon the problem that in my grass installation ZSTD is configured and I thus worked with raster data which was automatically compressed using ZSTD, but when I transferred the data to our HPC system, I realized that grass there is not configure with ZSTD and the latter is currently not available on the system. > > > > As the r.compress man page says, ZSTD is the default compressor if available. However, when I check in current trunk's configure, it says: > > > > > > {{{ > > --with-zstd support Zstandard functionality (default: no)" > > }}} > > > > Does that mean that even if libzstd is available on a system, GRASS is not compiled with support for it, unless --with-zstd is set ? > > yes > > > > I find this situation a bit uncomfortable as it can lead to situations where the user creates data without actively thinking about the compression method and this data is the unusable elsewhere. > > > > I would plead, thus, for libzstd to become a compulsory requirement for GRASS GIS, and for the configure script to default to --with-zstd=yes. > > Done in trunk r74391. The default is now `--with-zstd=yes`, you can disable ZSTD support with `--with-zstd=no`. To be backported I guess. +1 for backporting. Should libzstd be added to REQUIREMENTS.html ? -- Ticket URL: <https://trac.osgeo.org/grass/ticket/3815#comment:2> GRASS GIS <https://grass.osgeo.org>
_______________________________________________ grass-dev mailing list grass-dev@lists.osgeo.org https://lists.osgeo.org/mailman/listinfo/grass-dev