[csw-maintainers] Preliminary package reviews

[Peter FELECAN]

"Maciej (Matchek) Blizinski" <maciej at opencsw.org> writes:

> It's quite common that one of maintainers sends e-mail to the mailing
> list asking others to review / test a package, but gets no feedback.
> I've got an idea that for preliminary reviews, a HTML or a text view
> of a package could be useful.  It would show things like pkginfo and
> pkgmap, with the intention of reviewing the way one software project
> is split into subpackages, where and what are the binaries, etc.  The
> idea is that it's much quicker to click a link and view a page than
> dissect a file in a terminal session.  Some maintainers wouldn't have
> time to dissect packages by hand, but they could quickly review a HTML
> report and offer feedback.
>
> I've prepared a prototype view:
>
> http://bender.opencsw.org/~maciej/pkg-review-prototype.html
>
> Thoughts?

As said during the discussion of the subject at the summer camp, this
could enhance the peer review of the packages when a maintainers submits
voluntarily to QA.

I have 2 suggestions:

1. get rid of the pytonish data representation in the output and choose
   a text formatting one (the reader is a human, not a machine), e.g.:

   binaries_dump_info

    * opt/csw/mysql51/bin/amd64/innochecksum
          o runpath:
            /opt/csw/lib/$ISALIST
            /opt/csw/lib/64
            /opt/csw/mysql51/lib/$ISALIST
            /opt/csw/mysql51/lib/64
            /opt/csw/mysql51/lib/64/$ISALIST/mysql
          o needed sonames:
            libz.so.1
            libpthread.so.1
            librt.so.1 
            libc.so.1
            libsocket.so.1
            libnsl.so.1
            libm.so.2
            libthread.so.1

   It will make for a longer presentation but with a higher readability
   coefficient

2. a mechanism of submitting of the package to this presentation need to
   be defined, implemented and documented, viz. use case and
   walk-through.

-- 
Peter