#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.6.2
Component: Raster | Version: svn-trunk
Resolution: | Keywords: compression zstd zlib
CPU: Unspecified | Platform: Unspecified
--------------------------+-----------------------------------
Comment (by mmetz):
Replying to [comment:6 mmetz]:
> Replying to [comment:5 neteler]:
> > Replying to [comment:3 mmetz]:
> > > Replying to [comment:2 mlennert]:
> > > > +1 for backporting.
> > >
> > > Done in relbr76 r74398
> >
> > Just FYI:
> > This broke the manual-building cronjobs on the grass.osgeo.org server
but I fixed it by now actively disabling zstd in the configure step
(Debian Jessie doesn't have zstd yet and #3486 = compile in a docker
container is still open).
>
> A good reason to not make zstd a compulsory requirement. Debian Jessie
has long-term support until 30 June 2020, and GRASS 7.x should IMHO run on
this system (with appropriate configure options).
I think there was some misunderstanding from my side. For maximum
compatibility of GRASS raster data, the default compression should indeed
be ZLIB, even if ZSTD is available. I made ZSTD the default if available
in order to get it tested.
There was one issue reported where raster data compressed with ZSTD 1.3.6
could not be read with ZSTD 1.3.1, which remains a mystery.
What are the opinions on changing the default compression back to ZLIB
even if ZSTD is available?
+1 from Moritz Lennert as I understand
0 from me
--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3815#comment:7>
GRASS GIS <https://grass.osgeo.org>
_______________________________________________
grass-dev mailing list
grass-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/grass-dev
