[mythtv-commits] Ticket #7745: Support for raw photos in MythGallery

[MythTV]

#7745: Support for raw photos in MythGallery
----------------------------------------------+-----------------------------
 Reporter:  Chad Parry <spam@…>               |       Owner:  ijr       
     Type:  patch                             |      Status:  new       
 Priority:  minor                             |   Milestone:  unknown   
Component:  Plugin - MythGallery              |     Version:  0.22-fixes
 Severity:  medium                            |     Mlocked:  0         
----------------------------------------------+-----------------------------

Comment(by Chad Parry <spam@…>):

 Paul, I'm the first to admit that invoking an external executable is a
 crude way to call a library. It does provide a few advantages,
 however:[[BR]]
 1) The only supported distribution of the dcraw code is in a standalone
 executable. Right or wrong, the author feels very strongly that this added
 layer of isolation improves the program's reliability.[[BR]]
 2) The dcraw executable gets updated with new camera types and MythGallery
 will automatically benefit from that. Any other repackaging of the code
 ends up requiring extra maintenance and drifts out of sync with the
 original.[[BR]]
 3) This method doesn't require us to import dcraw sources or link to dcraw
 binaries, so we are able to avoid the licensing issues raised by Colin.

 So I would be happy to use a dcraw wrapper instead, but I wouldn't want to
 make the change unless there would be benefits to MythGallery. Since the
 current patch already works, I don't know of any potential benefits. Even
 the overhead of starting a new process is negligible, since unfortunately
 it takes several seconds to decode each image.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/7745#comment:6>
MythTV <http://www.mythtv.org/>
MythTV