[pkg-bacula-devel] [pkg-bacula-commits] [SCM] Bacula, a network backup, recovery and verification program branch, master, updated. debian/5.0.3-1-121-g2b82cc6

[Alexander Golovko]

On Fri, 01 Jun 2012 20:35:41 +0200, Luca Capello wrote:
> Hi Alexander!
>
> Please reply to pkg-bacula-devel@, -commits should be "read-only" ;-)

ok

>
> On Fri, 01 Jun 2012 13:31:20 +0200, Alexander Golovko wrote:
>> Upstream patch is incorrect.
>>
>> They add 'libtool/ltdl.m4' usage into aclocal.m4, but forget add
>> ltdl.m4'.
>
> I discovered that while testing the upstream patch: given that 
> upstream
> suggested that we need to regenerate aclocal.m4 anyway, I guessed 
> that
> this was correct and this is why I added libltdl-dev to 
> Build-Depends:.
> Thus, if I have understood correctly, if we add 'libtool/ltdl.m4' 
> then
> there is no more need for libltdl-dev.

Hmm, if i understand Marco post in upstream bugtracker, we need 
regenerate only configure script, not aclocal.m4
in upstream bug 1880 Marco write, that he add missed file, but it not 
push his changes, so i can't get his version.

Yes, it look like we can remove this build depends.


>
>> Yes, we can completelly regenerate aclocal.m4, as you do, but in 
>> this
>> case we need to remove aclocal.m4 changes from upstream patch, or 
>> dpkg
>> can't unapply patches after building. I think, that add missed file 
>> to
>> patch is better.
>>
>> Due to bug in upstream bugtracker already closed and i don't have
>> permissions to reopen it, was created new bug 1880.
>
> Thank you.
>
> Thx, bye,
> Gismo / Luca

-- 
with best regards,
Alexander Golovko
email: alexandro at ankalagon.ru
xmpp: alexandro at ankalagon.ru