Subject: Re: CVS commit: pkgsrc/devel/libgphoto2
To: Bernd Ernesti <veego@NetBSD.org>
From: Stephen Borrill <sborrill@NetBSD.org>
List: pkgsrc-changes
Date: 03/31/2007 19:14:23
Bernd Ernesti wrote:
>>>I also tracked it down to
>>>
>>>https://sourceforge.net/tracker/?func=detail&atid=108874&aid=1648398&group_id=8874
>>>
>>>which uses a different patch which was allready commited to the libgphoto2
>>>source code.
>>
>>Yes, I'd looked at using UCS-2LE and BE.
>>
[snip]
> It would be better to use the commited patch and not something which isn't
> documented (in the patch file) and differes from there version.

Agreed.

>>>See attached patch (which fixed two files in one patch). I didn't had the 
>>>time to commit this patch.
>>
>>pkgsrc is frozen now for 2007Q1 anyway; I'd need to get permission to 
>>commit this (plus I'd like to test it on a BE and LE machine first).
> 
> It wasn't frozen when you wrote that mail.

Yes, there seems to have been some confusion over the exact start time (was 
it pushed back by 24 hours?). I was on ICB and messages were flying around 
saying freeze in 1 hour, yet next day it still wasn't frozen.

> Right now it is broken because doxygen isn't disabled, despite the
> DOXYGEN=no in the Makefile and therefor the PLIST isn't correct if
> doxygen is installed.
> 
> See the attached patch which fixes this problem.

OK, that's a different, but more serious issue IMHO. My patch fixed the 
iconv problem, but wasn't the 'official' fix.  However, this is a build 
problem and so should be higher priority.

Can this be committed in the freeze period? N.B. libgphoto2 is not a leaf 
package (though does anything depend on it besides gphoto2?). This commit 
with fix the doxygen problem as well as pull in a rework of their iconv code:

https://sourceforge.net/tracker/?func=detail&atid=108874&aid=1648398&group_id=8874

-- 
Stephen