On Wed, Jul 20, 2005, Ralf S. Engelschall wrote:
> On Wed, Jul 20, 2005, Matthias Kurz wrote:
>
> > OpenPKG CVS Repository
> > http://cvs.openpkg.org/
> >
> > ____________________________________________________________________________
> >
> > Server: cvs.openpkg.org Name: Matthias Kurz
> > Root: /v/openpkg/cvs Email: [EMAIL PROTECTED]
> > Module: openpkg-src Date: 20-Jul-2005 13:36:10
> > Branch: HEAD Handle: 2005072012360901
> >
> > Added files:
> > openpkg-src/bacula bacula.patch bacula.spec bconsole.sh rc.bacula
> >
> > Log:
> > new package: bacula 1.37.30 (Network backup tool)
> > [...]
>
> Cool. Thanks for this contribution, Matthias. In my role as the
> OpenPKG Principal Architect I've reviewed this new package and here
> is my feedback (already in the form of a quoted patch with inserted
> annotations):
Hi, Ralf.
I have only one comment regarding the odoc support. I also released a
package that _adds_ the support. Maybe you missed it. And i'm sorry
that i have to insist in using it :) The bacula package is still work
in progress and i actually want to _use_ it. That means i do not want
to maintain two different versions of the package.
The %docdir clause does not work in the files list. Alternatively you
could change your rpm patch that removes the %{_docdir} semantics from
rpm. I asked you several times to do so, but you did not even comment.
I thought you where just too busy with your AIX porting, so i decided
to not insist too much :)
Apart from this i thank you for your comments. I did not see other
critical changes. Add the odoc stuff and commit it. I'm going to check,
whether it works for me.
(mk)
--
Matthias Kurz; Fuldastr. 3; D-28199 Bremen; VOICE +49 421 53 600 47
>> Im prämotorischen Cortex kann jeder ein Held sein. (bdw) <<
______________________________________________________________________
The OpenPKG Project www.openpkg.org
Developer Communication List openpkg-dev@openpkg.org
