On Wed, Jul 20, 2005, Matthias Kurz wrote:

> 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):
>
> I have only one comment regarding the odoc support. I also released a
> package that _adds_ the support. Maybe you missed it.

No, I never miss any commit. I just don't comment everything in public ;-)
See my personal mail to you in your [EMAIL PROTECTED] mailbox...

> 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.

I can't add it because as far as I see it, just using %{l_odoc} doesn't
work. And the "bacula" package can be used without this, so I recommend
to leave "odoc" out from "bacula" for now until all issues with "odoc"
are resolved. Ok?

                                       Ralf S. Engelschall
                                       [EMAIL PROTECTED]
                                       www.engelschall.com

______________________________________________________________________
The OpenPKG Project                                    www.openpkg.org
Developer Communication List                   openpkg-dev@openpkg.org

Reply via email to