Opened 9 years ago

Closed 9 years ago

#8836 closed defect (fixed)

win32: check for target instead of Makefile for libexif

Reported by: jonathan@… Owned by: robertm
Priority: minor Milestone: unknown
Component: Ports - Windows Version: Unspecified
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Attached patch makes win32-packager.pl check for the target DLL instead of the Makefile analogue to all other libs.

Attachments (2)

win32-packager.pl-check-for-target-instead-of-Makefile.patch (643 bytes) - added by jonathan@… 9 years ago.
Check for libexif.dll instead of Makefile
win32-packager.pl-check-for-target-instead-of-Makefile.patch2 (646 bytes) - added by Jonathan Martens <jonathan@…> 9 years ago.
Updated patch, checking for libexif-12.dll instead of libexif.dll

Download all attachments as: .zip

Change History (6)

Changed 9 years ago by jonathan@…

Check for libexif.dll instead of Makefile

comment:1 Changed 9 years ago by robertm

Owner: changed from Nigel to robertm
Status: newassigned

comment:2 Changed 9 years ago by robertm

Resolution: fixed
Status: assignedclosed

(In [25950]) Windows Port: Check that libexif has actually been installed, rather than just seeing if configure has succeeded in libexif's source. Patch from Jonathan. Fixes #8836.

comment:3 in reply to:  2 Changed 9 years ago by Jonathan Martens <jonathan@…>

Resolution: fixed
Status: closednew

Replying to robertm:

(In [25950]) Windows Port: Check that libexif has actually been installed, rather than just seeing if configure has succeeded in libexif's source. Patch from Jonathan. Fixes #8836.

It seems my patch was not done properly (http://mythtv.org/pipermail/mythtv-dev/2010-August/069311.html)... it should test for libexif-12.dll as the symlink to libexif.dll is created in the next step. New build environments being setup will halt now, it must have slipped as mine was not created cleanly again and already had the symlink in place. :blush:

Changed 9 years ago by Jonathan Martens <jonathan@…>

Updated patch, checking for libexif-12.dll instead of libexif.dll

comment:4 Changed 9 years ago by robertm

Resolution: fixed
Status: newclosed

(In [25997]) Fix #8836, hopefully properly this time.

Note: See TracTickets for help on using tickets.