[Shotwell] [future request] add some default verbosity

[Adam Dingle]

Alexey,

On 12/29/2010 02:50 AM, Alexey Fisher wrote:
> Hallo devs,
>
> i have some crashes with shotwell 0.8. It happens on import or on
> library update. At the same time i get "Error: Directory NikonPreview
> with 21313 entries considered invalid; not read." After some googling i
> found this mail: http://osdir.com/ml/digikam-users/2010-08/msg00086.html
> it says, this message come from lib exif2.

That's (almost) correct: this error message comes from the exiv2 
library.  We'd like to suppress it, but (at last until recently) the 
library has insisted on writing these messages to standard output: see 
http://trac.yorba.org/ticket/1872 .  It appears this may finally have 
been fixed in the latest exiv2 release according to one of the comments 
on that ticket.

> I still can't find if this error is the reason of the crash. Some times
> i get many of this errors and shotwell is working, some times it crashes
> after few of them.

This error message is benign, and is almost certainly not the cause of 
the crash you're observing.

> My [future request] is to add some default verbosity. For example on
> photo import shotwell should write to terminal the path of current
> photo. So if it crashes i can find where it was.
> Second: is to fix this message "Error: Directory NikonPreview with 21313
> entries considered invalid; not read." it says almost nothing. It should
> be some thing like this:
> shotwell: read image, path: /home/bla/Bilder/someimage.jpg
> libexiv2: can't parse exif of image: someimage.jpg
> libexiv2: 'NikonPreview' entry is too big (21313 byte). Skip it.

As Lucas mentioned in his earlier reply, Shotwell does have a logging 
facility which can write the path of each photo as it's imported.  See 
http://trac.yorba.org/wiki/Shotwell/FAQ .

adam