<div dir="ltr">Moving the conversation from the ticket.<br><div><br>On Sat, Jun 6, 2015 at 9:16 AM, GRASS GIS <<a href="mailto:trac@osgeo.org" target="_blank">trac@osgeo.org</a>> wrote:<br>><br>> #2349: CELL raster format: make ZLIB level 3 standard compression instead of RLE<br>> --------------------------+-------------------------------------------<br>>   Reporter:  neteler      |      Owner:  grass-dev@…<br>>       Type:  enhancement  |     Status:  new<br>>   Priority:  critical     |  Milestone:  7.1.0<br>>  Component:  Raster       |    Version:  svn-trunk<br>> Resolution:               |   Keywords:  compression, r.compress, null<br>>        CPU:  Unspecified  |   Platform:  All<br>> --------------------------+-------------------------------------------<br>><br>> Comment (by martinl):<br>><br>>  Replying to [comment:34 glynn]:<br>>  > Replying to [comment:33 neteler]:<br>><br>>  > I added some support for the improved r.null/r.compress functionality to<br>>  lib/raster, but it was amongst the things broken by r65348 (the tempfile<br>>  changes), so this needs to wait until that gets fixed or reverted.<br>><br>>  by "fix" you mean to ignore GRASS_TMPDIR_MAPSET when working with raster<br>>  data?<br><br></div><div>Hi, it seems to me that there is some confusion here. AFAIK r65348 has two issues.<br><br></div><div>First, it allows unorthodox behavior of GRASS but this behavior is (or at least should be) applied only when GRASS_TMPDIR_MAPSET is set to some specific value. I don't see much problem with it as long as the documentation properly discuss all the potential issues and the default is the standard and safe way. This was discussed in [GRASS-dev] r65348 (GRASS_TMPDIR_MAPSET).<br><br></div><div>Second, it breaks reading of header files (and perhaps other files) in some cases (issue reported in #2687). I really don't know how this is related to tmp files (seems unrelated to me), but tests are showing that it happened between r65346 (OK) and r65349 (broken). When I created #2687, I tried to compile different versions manually and r65348 was when it got broken. Also r65347 is backslash cosmetics in grass.py and r65349 is in grass-addons. In first comment for #2687, Glynn explains that the picking of a wrong Mapset by <a href="http://r.info">r.info</a> "is caused by r65348, which constructs the path before the mapset has been resolved." This renders trunk almost unusable.<br></div><div><br></div><div>I hope this overview helps,<br></div><div>Vaclav<br></div><div><br><br>r65346 <a href="https://trac.osgeo.org/grass/changeset/65346">https://trac.osgeo.org/grass/changeset/65346</a><br>r65347 <a href="https://trac.osgeo.org/grass/changeset/65347">https://trac.osgeo.org/grass/changeset/65347</a><br>r65348 <a href="https://trac.osgeo.org/grass/changeset/65348">https://trac.osgeo.org/grass/changeset/65348</a><br>r65349 <a href="https://trac.osgeo.org/grass/changeset/65349">https://trac.osgeo.org/grass/changeset/65349</a><br>#2687 <a href="https://trac.osgeo.org/grass/ticket/2687">https://trac.osgeo.org/grass/ticket/2687</a><br>#2687 <a href="https://trac.osgeo.org/grass/ticket/2687#comment:1">https://trac.osgeo.org/grass/ticket/2687#comment:1</a><br>#2349 <a href="https://trac.osgeo.org/grass/ticket/2349#comment:34">https://trac.osgeo.org/grass/ticket/2349#comment:34</a><br><br>[GRASS-dev] r65348 (GRASS_TMPDIR_MAPSET)<br><a href="http://lists.osgeo.org/pipermail/grass-dev/2015-June/075220.html">http://lists.osgeo.org/pipermail/grass-dev/2015-June/075220.html</a><br><a href="http://osgeo-org.1560.x6.nabble.com/r65348-GRASS-TMPDIR-MAPSET-td5208936.html">http://osgeo-org.1560.x6.nabble.com/r65348-GRASS-TMPDIR-MAPSET-td5208936.html</a><br><a href="http://comments.gmane.org/gmane.comp.gis.grass.devel/64556">http://comments.gmane.org/gmane.comp.gis.grass.devel/64556</a><br></div></div>