Opened 4 years ago

Closed 4 years ago

Last modified 3 years ago

#2712 closed defect (fixed)

WARNING: Unable to rename null file

Reported by: hellik Owned by: grass-dev@…
Priority: blocker Milestone: 7.2.0
Component: LibRaster Version: svn-trunk
Keywords: raster Cc:
CPU: x86-32 Platform: MSWindows Vista

Description

running some raster modules, I get following warning message:

e.g.

(Sun Jul 19 15:14:36 2015)                                                      
r.slope.aspect --verbose elevation=elevation@PERMANENT slope=myslope            
Prozent fertig...
Höhen-Ergebnisse für Mapset <user1> in <nc_spm_08_grass7>.
WARNING: Unable to rename null file 'J:\gisdata/nc_spm_08_grass7/user1/.tmp/unknown/4828.1' to 'J:\gisdata/nc_spm_08_grass7/user1/cell_misc/myslope/null': Permission denied
minimale berechnete Hangneigung 0.0000, maximale berechnete Hangneigung 38.6894.
Neigungs-Rasterkarte <myslope> komplett.
(Sun Jul 19 15:14:38 2015) Befehl ausgeführt (1 Sek)                            
(Sun Jul 19 15:15:24 2015)                                                      
r.param.scale --verbose input=elevation@PERMANENT output=mymorpho method=feature
WARNING: Unable to rename null file 'J:\gisdata/nc_spm_08_grass7/user1/.tmp/unknown/3040.1' to 'J:\gisdata/nc_spm_08_grass7/user1/cell_misc/mymorpho/null': Permission denied
(Sun Jul 19 15:15:26 2015) Befehl ausgeführt (1 Sek)   

Change History (17)

comment:1 Changed 4 years ago by hellik

Keywords: raster added

comment:2 in reply to:  description ; Changed 4 years ago by hellik

Replying to hellik:

running some raster modules, I get following warning message:

e.g. also in r.watershed

(Sun Jul 19 15:20:04 2015)                                                      
r.watershed --verbose elevation=elevation@PERMANENT threshold=100 accumulation=myaccum tci=mytopoi drainage=mydraindir basin=myabasin
ABSCHNITT 1a (von 5): Initiiere den Speicher.
ABSCHNITT 1b (of 5): Ermittle den Fluß aus der Karte heraus.
SECTION 2: A* Search.
SECTION 3a: Accumulating Surface Flow with MFD.
SECTION 3b: Adjusting drainage directions.
ABSCHNITT 4: Bestimme Wassereinzugsgebiet.
ABSCHNITT 5: Schließe Karten.
WARNING: Unable to rename null file 'J:\gisdata/nc_spm_08_grass7/user1/.tmp/unknown/4520.1' to 'J:\gisdata/nc_spm_08_grass7/user1/cell_misc/myabasin/null': Permission denied
WARNING: Unable to rename null file 'J:\gisdata/nc_spm_08_grass7/user1/.tmp/unknown/4520.3' to 'J:\gisdata/nc_spm_08_grass7/user1/cell_misc/myaccum/null': Permission denied
WARNING: Unable to rename null file 'J:\gisdata/nc_spm_08_grass7/user1/.tmp/unknown/4520.5' to 'J:\gisdata/nc_spm_08_grass7/user1/cell_misc/mytopoi/null': Permission denied
WARNING: Unable to rename null file 'J:\gisdata/nc_spm_08_grass7/user1/.tmp/unknown/4520.7' to 'J:\gisdata/nc_spm_08_grass7/user1/cell_misc/mydraindir/null': Permission denied
(Sun Jul 19 15:20:14 2015) Befehl ausgeführt (9 Sek)  

comment:3 in reply to:  2 ; Changed 4 years ago by hellik

Replying to hellik:

Replying to hellik:

running some raster modules, I get following warning message:

e.g. also in r.watershed

see also https://trac.osgeo.org/grass/ticket/2689

in contrary to #2689, the null file isn't there.

comment:4 in reply to:  description Changed 4 years ago by neteler

Priority: normalblocker

Replying to hellik:

running some raster modules, I get following warning message:

The file in question is

lib/raster/close.c

...
WARNING: Unable to rename null file 'J:\gisdata/nc_spm_08_grass7/user1/.tmp/unknown/4828.1' to 'J:\gisdata/nc_spm_08_grass7/user1/cell_misc/myslope/null': Permission denied

The directory separators are likely wrong. Since it is in libraster it affects several modules.

comment:5 in reply to:  3 Changed 4 years ago by hellik

Replying to hellik:

Replying to hellik:

Replying to hellik:

running some raster modules, I get following warning message:

e.g. also in r.watershed

see also https://trac.osgeo.org/grass/ticket/2689

and also here: https://trac.osgeo.org/grass/ticket/2434

comment:6 in reply to:  2 ; Changed 4 years ago by hellik

Replying to hellik:

Replying to hellik:

running some raster modules, I get following warning message:

e.g. also in r.watershed

tested with the elevation data of the nc sample data set

comment:7 in reply to:  6 Changed 4 years ago by annakrat

Component: DefaultLibRaster
Milestone: 7.0.1
Version: svn-releasebranch70svn-trunk

Replying to hellik:

Replying to hellik:

Replying to hellik:

running some raster modules, I get following warning message:

e.g. also in r.watershed

tested with the elevation data of the nc sample data set

Problem seems to be only in trunk.

comment:8 Changed 4 years ago by martinl

Milestone: 7.1.0

comment:9 Changed 4 years ago by martinl

The bug seems to be related to the recent changes in null data support done in trunk.

comment:10 in reply to:  9 ; Changed 4 years ago by glynn

Replying to martinl:

The bug seems to be related to the recent changes in null data support done in trunk.

I think that it's because r65323 deferred closing the null file (because a compressed null file needs to have the row pointers written out), meaning that the file is still open at the time of the rename() call. This is valid on Unix but not on Windows (although the MSDN documentation neglects to mention this in the description of the EACCESS error for rename()).

So, the order needs to be:

  • Write out row pointers
  • Close file
  • Rename file

comment:11 in reply to:  10 ; Changed 4 years ago by glynn

Replying to glynn:

So, the order needs to be:

  • Write out row pointers
  • Close file
  • Rename file

Try r65775.

comment:12 in reply to:  11 Changed 4 years ago by hellik

Replying to glynn:

Replying to glynn:

So, the order needs to be:

  • Write out row pointers
  • Close file
  • Rename file

Try r65775.

tested with

GRASS Version: 7.1.svn                                                          
GRASS SVN revision: 65777                                                       
Build date: 2015-07-25                                                          
Build platform: i686-pc-mingw32                                                 
GDAL: 1.11.2                                                                    
PROJ.4: 4.8.0                                                                   
GEOS: 3.4.2                                                                     
SQLite: 3.7.17                                                                  
Python: 2.7.4                                                                   
wxPython: 2.8.12.1                                                              
Platform: Windows-Vista-6.0.6002-SP2 (OSGeo4W) 
r.slope.aspect --verbose elevation=elevation@PERMANENT slope=myslope            
Prozent fertig...
Höhen-Ergebnisse für Mapset <user1> in <nc_spm_08_grass7>.
minimale berechnete Hangneigung 0.0000, maximale berechnete Hangneigung 38.6894.
Neigungs-Rasterkarte <myslope> komplett.
(Sun Jul 26 10:12:35 2015) Befehl ausgeführt (2 Sek)  
r.param.scale --verbose input=elevation@PERMANENT output=myfeature method=feature
(Sun Jul 26 10:14:01 2015) Befehl ausgeführt (1 Sek) 
r.watershed --verbose elevation=elevation@PERMANENT threshold=100 accumulation=myaccum
ABSCHNITT 1a (von 5): Initiiere den Speicher.
ABSCHNITT 1b (of 5): Ermittle den Fluß aus der Karte heraus.
SECTION 2: A* Search.
SECTION 3a: Accumulating Surface Flow with MFD.
SECTION 3b: Adjusting drainage directions.
ABSCHNITT 4: Bestimme Wassereinzugsgebiet.
ABSCHNITT 5: Schließe Karten.
(Sun Jul 26 10:15:31 2015) Befehl ausgeführt (7 Sek) 

seems to be fixed now, closing ticket

comment:13 Changed 4 years ago by hellik

Resolution: fixed
Status: newclosed

comment:14 in reply to:  11 ; Changed 4 years ago by hellik

Replying to glynn:

Replying to glynn:

So, the order needs to be:

  • Write out row pointers
  • Close file
  • Rename file

Try r65775.

#2689 seems to be solved, #2434 is still valid

comment:15 in reply to:  14 ; Changed 4 years ago by glynn

Replying to hellik:

#2434 is still valid

#2434 was opened long before any of these changes were made.

comment:16 in reply to:  15 Changed 4 years ago by hellik

Replying to glynn:

Replying to hellik:

#2434 is still valid

#2434 was opened long before any of these changes were made.

right; but there seems to be similar problems with raster:

see e.g. 2434#comment:14

[...]
WARNING: Unable to rename null file 'J:\gisdata\nc_spm_08_grass7\user1\.tmp/unknown\5848.1' to 'J:\gisdata\nc_spm_08_grass7\user1\cell_misc\elev_lid792_rast\null': Permission denied
WARNING: Unable to rename cell file 'J:\gisdata\nc_spm_08_grass7\user1\.tmp/unknown\5848.0' to 'J:\gisdata\nc_spm_08_grass7\user1\fcell\elev_lid792_rast': Permission denied
WARNING: Unable to write quant rules: raster map <elev_lid792_rast> is integer

comment:17 Changed 3 years ago by neteler

Milestone: 7.1.07.2.0

Milestone renamed

Note: See TracTickets for help on using tickets.