search

jwilk / scanhelper

command-line tool for batch scanning
https://jwilk.net/software/scanhelper
GNU General Public License v2.0
3 stars 0 forks source link

misleading names of temporary files #1

Closed jwilk closed 6 years ago

jwilk commented 11 years ago

Issue reported by @jsbien:

The disk was full during a scanning session. There seem to be no way to restart processing after freeing the disk space and I was left with TIFF files named e.g p0316.png.tmp.scanhelper.

jwilk commented 10 years ago

The filename was meant to indicate that it's a temporary scanhelper file that will be eventually converted to p0316.png. But I see that the name might be misleading.

Would p0316.png.tiff.tmp.scanhelper be okay with you? Or is it too convoluted? :-)

jwilk commented 10 years ago

Comment submitted by @jsbien:

This doesn't have now a practical importance for me, but I think a TIFF file should have the extention tiff, I don't care about the rest of the name. Just an idea: tmp.p0316.png.tiff or p0316.png.tmp.scanhelper.tiff.

jwilk commented 6 years ago

The good news is that in SANE ≥ 1.0.25, scanimage (which is what scanhelper uses under the hood) supports the PNG format natively. My plan is to drop the use of temporary files completely, and instead require new enough SANE when using the PNG output format.

jsbien commented 6 years ago

It's OK with me.

jwilk commented 6 years ago

SANE ≥ 1.0.25

To clarify, I meant sane-backends here. (sane-frontends in versioned independently.) This version is available since Debian 9 (stretch, currently stable) and Ubuntu 16.04 (xenial).

jwilk commented 6 years ago

My plan is to drop the use of temporary files completely, and instead require new enough SANE when using the PNG output format.

This was implemented in 4d69ee547dee4a64d3b177a225043b1ece6d2021.

jwilk commented 6 years ago

Fixed in scanhelper 0.7.