Opened 10 years ago
Closed 9 years ago
#535 closed defect (fixed)
Some files cause a mystery error - MakerNote Tag 0x0001
Reported by: | Mark Holmquist | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | |
Component: | programming | Keywords: | |
Cc: | Parent Tickets: |
Description
This file was troublesome, for some reason. It is pretty big, but I'm not sure that's the problem.
Subtickets
Attachments (2)
Change History (9)
Changed 10 years ago by
comment:1 Changed 10 years ago by
Interesting. It looks like EXIF.py (which we don't maintain) has some special code here relative to makertag stuff, and that it's broken somehow.
We also haven't pulled down a more recent EXIF.py in some time; I wonder if it's been fixed recently?
comment:2 Changed 10 years ago by
If someone wants to tackle this bug, it would be good if they could try pulling in a more recent version of EXIF.py and see if that fixes it. If not, we should file a bug and possibly a fix upstream.
comment:3 Changed 10 years ago by
Tested the image with a newer version of EXIF.py (available at https://github.com/ianare/exif-py). The scripts returns all the exif infos with no error compared to the actual version in mediagoblin. Tested with mediagoblin by replacing the EXIF.py in /mediagoblin/extlib/exif/ and it works flawlessly, the image shows up on the website and no error appear. So apparently this new version fixes this bug.
comment:4 Changed 10 years ago by
Owner: | set to Deb |
---|---|
Status: | new → accepted |
comment:5 Changed 10 years ago by
Status: | accepted → in_progress |
---|
Deb, are you working on this bug? I've set it in_progress and to you, not sure what's going on. Feel free to unclaim if you aren't presently working on it. Thanks!
comment:6 Changed 9 years ago by
Hey there. 1) I have not done any work on this and 2) it seems like it is fixed once you update exif.py
I didn't look into this in any detail and so don't feel qualified to close this bug.
comment:7 Changed 9 years ago by
Owner: | Deb deleted |
---|---|
Resolution: | → fixed |
Status: | in_progress → closed |
We are indeed using a newer version now (via a library) and everything seems fine, so closing this ticket.
The debug output, for your reference.