Opened 17 years ago
Closed 12 years ago
#91 closed defect (fixed)
ps.map doesn't remove .tmp files when it is done
Reported by: | hamish | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 6.4.0 |
Component: | Ps.map | Version: | unspecified |
Keywords: | ps.map | Cc: | |
CPU: | Unspecified | Platform: | Unspecified |
Description
this was bug # 492 in the old gforge system. moving it here. http://wald.intevation.org/tracker/index.php?func=detail&aid=492&group_id=21&atid=204
Hamish
Hi,
ps.map creates some .tmp files, e.g. when the 'text' or 'rectangle' instructions are used but they aren't removed when the module finishes. For scripted ps.map jobs / trial+error iterations you end up with hundreds of tiny files in the .tmp dir until you exit the GRASS session.
They are human readable and useful for debugging, so it would be nice to only remove them if the g.gisenv DEBUG level was below some threshold.
Hamish
Attachments (1)
Change History (8)
by , 17 years ago
Attachment: | psmap_unlink.diff added |
---|
comment:3 by , 16 years ago
Keywords: | ps.map added |
---|
Replying to neteler:
Any objections that I apply attached patch?
There is a problem with the patch-
If the ps.map instructions include both masked and unmasked plotting commands, the plot command file will be removed after the first pass and the unmasked items will not be drawn because the file is missing.
Better to move the unlink to a cleanup section at the very end of ps_map.c? or main.c, it already does unlink (ps_mask_file);
just before exit().
also it would be nice to observe this part of the wish: "They are human readable and useful for debugging, so it would be nice to only remove them if the g.gisenv DEBUG level was below some threshold."
Hamish
comment:4 by , 16 years ago
comment:5 by , 16 years ago
Component: | default → ps.map |
---|---|
CPU: | → Unspecified |
Platform: | → Unspecified |
comment:6 by , 12 years ago
Milestone: | 6.4.0 → 6.4.4 |
---|
Hamish, please backport to relbranch if appropriate.
comment:7 by , 12 years ago
Milestone: | 6.4.4 → 6.4.0 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
it didn't need a backport as it was done before relbr64 was split off.
closing bug.
fix proposal