[pkg-bacula-devel] [pkg-bacula-commits] [SCM] Bacula, a network backup, recovery and verification program branch, alexandro-dev, created. debian/5.0.3-1-13-g68d9dca

[Alexander Golovko]

On Mon, 14 May 2012 15:08:36 +0200, Luca Capello wrote:
> Hi Alexander!
>
> On Sun, 13 May 2012 21:40:45 +0200, Alexander Golovko wrote:
>> The branch, alexandro-dev has been created
>>         at  68d9dca43507c3b9dc324133b7a3a2845fe8ecef (commit)
>>
>> - Shortlog 
>> ------------------------------------------------------------
>> commit 68d9dca43507c3b9dc324133b7a3a2845fe8ecef
>> Author: Alexander Golovko <alexandro at ankalagon.ru>
>> Date:   Sun May 13 13:22:30 2012 +0400
>>
>>     Switch from upstream nonfree SHA1 implementation to openssl
>>
>>     Closes: #658326
>
> This answers my question at:
>
>   <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=658326#31>
>
> Sorry, I did not read the full pkg-bacula-*@ backlog before replying, 
> so
> I think you understand that whenever you commit something WRT a bug 
> you
> should send a notice to the BTS ;-)
>
>> commit 97da252d67c40912b11682462d9db835dee138bf
>> Author: Alexander Golovko <alexandro at ankalagon.ru>
>> Date:   Sun May 13 13:00:47 2012 +0400
>>
>>     Fix linking with binutils-gold
>>
>>     Closes: #553956
>
> This was already committed by Hauke:
>
>
> 
> <http://anonscm.debian.org/gitweb/?p=pkg-bacula/bacula.git;a=commitdiff;h=df0a0e8ba55cbc01b36d9d5569775ede907c8b62>
>

Hauke patch (as i can see it based on patch in bug) fix linking with 
"libbac", but problem in "libbac" dependencies list.
My patch fix libbac dependencies. so it more simple.


>> commit ac1d79010e4fe6962badf526ea4567568d48b051
>> Author: Alexander Golovko <alexandro at ankalagon.ru>
>> Date:   Sun May 13 12:44:54 2012 +0400
>>
>>     Fix dbconfig-common usage in bacula-director-sqlite3 prerm and 
>> postrm scripts
>>
>>     Closes: #670427
>>     Closes: #672285
>
> As I wrote on the BTS, we should check if removing the database is 
> the
> correct thing to do, since it is not a configuration files:
>
>   <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=672285#15>
>

Yes, this is absolutelly correctly to allow purge database on package 
purging.
This is what to do bacula-director-mysql and bacula-director-pgsql 
packages

But this doesn't matter, because Hauk rework bacula-director-* 
prerm/postrm scripts and they generates from one template.
So this patch is not actual,


> As a general note, please use the upstream Git convention for commit
> messages, i.e. the first line should be withing 65 characters.  For 
> the
> git-buildpackage convention, see my comments at:
>
>   <mid:87y5p1u15p.fsf at gismo.pca.it>
>
> 
> <http://lists.alioth.debian.org/pipermail/pkg-bacula-devel/2012-May/000177.html>
>
> Thx, bye,
> Gismo / Luca

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