Opened 9 years ago
Closed 7 years ago
#47837 closed update (fixed)
port:exiv2 upgrade (also addresses a current crash)
Reported by: | RJVB (René Bertin) | Owned by: | \ |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | haspatch | Cc: | dbevans (David B. Evans), mkae (Marko Käning) |
Port: | exiv2 |
Description
exiv2 can crash attempting to extract the metadata from certain QuickTime movies. This bug is still present in the current svn/head, but at least that version provides an option to disable support for video (which doesn't have EXIF data anyway, AFAIK).
I reported the bug: http://dev.exiv2.org/issues/1088 .
Attached is a patch to upgrade port:exiv2 to the latest version, introducing a +video variant to enable support for video metadata for those who can live with the risk of crashing.
This update also adds the cmake.out_of_source yes
command that was still missing.
Attachments (2)
Change History (11)
Changed 9 years ago by RJVB (René Bertin)
Attachment: | patch-deactivate-QT-userdata.diff added |
---|
Changed 9 years ago by RJVB (René Bertin)
Attachment: | exiv2.diff added |
---|
comment:1 Changed 9 years ago by RJVB (René Bertin)
comment:2 Changed 9 years ago by dbevans (David B. Evans)
Cc: | tcurdt@… removed |
---|---|
Owner: | changed from macports-tickets@… to tcurdt@… |
comment:3 Changed 9 years ago by dbevans (David B. Evans)
Cc: | devans@… added |
---|
I'm not sure what the good of upgrading to SVN head is (other than for testing) if the problem persists there as well. Normally one would stick to stable upstream releases with patches to fix the issue if possible. If it is desireable to provide the upstream development version, a *-devel version of the port is typically provided.
comment:5 Changed 9 years ago by larryv (Lawrence Velázquez)
Why is the subversion
port necessary? What’s wrong with Xcode’s Subversion?
And please remove the if {[file exists ${filespath}/${name}-svn/.svn]}
testing detritus.
comment:7 Changed 9 years ago by ryandesign (Ryan Carsten Schmidt)
You can Cc and un-Cc yourself by clicking the "CcMe!" button.
On this ticket, you are not Cc'd, you are the reporter, so if you are wanting to stop receiving mail about this ticket, we would have to change the ticket's reporter address to something other than your address.
comment:8 Changed 7 years ago by kurthindenburg (Kurt Hindenburg)
Owner: | changed from tcurdt@… to \ |
---|---|
Status: | new → assigned |
comment:9 Changed 7 years ago by mf2k (Frank Schima)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
This port has been updated to 0.26 sometime in the past.
I've "stubbified" the function containing the higher-level operation during which the crash occurs, so
+video
can be made a default variant in order to avoid feature loss w.r.t. the current port:exiv2 .