[Mageia-dev] Some more new rpmlint warning on upload

[Thierry Vignaud]

On 27 July 2011 13:37, Michael Scherer <misc at zarb.org> wrote:
> * non-standard-group
>
> This one cause problem in rpmdrake display ( among others ), as this add
> category with 1 single package. There is no false positives.

That's actually a conf regression on our side, I think most packagers are used
to such rejects for a couple years and nobody will complain

> * invalid-desktopfile
>
> rpmlint run desktop-file-validate, and any invalid file is detected.
> Invalid file will in the best case work fine, and in the worst case, not
> work. So better be safe and fix the file.
>
> There is also some more controversial :

I think this one got enabled at one stage on mdv

> * file-not-in-%lang
> this will result in more disk space used for file that are not needed
> ( like locales, etc ). However, this can be annoying to fix, and not a
> widespread errors.

This is borderline.
Not a real blocking issue IMGO

> * version-control-internal-file
> same as previous, but there is maybe some use case, that should then be
> added to exception

idem

> * standard-dir-owned-by-package
> some package ( like gromacs ) do created /usr/bin/. This is unclean and
> should be blocked. On the other hand, the problem are mostly theoric
> ( like "having a directory with 2 differents modes or owners ).

> I also found some stuff that would cause real problem :
> hunspell-ca.noarch: W:
> world-writable /usr/share/doc/hunspell-ca/LICENSES-en.txt 0666
>
> Yet, there is maybe some good case to have a file to be world writable ?

we can whitelist the good cases if any

> Or :
> hdf-java.i586: W: non-readable /usr/share/java/fits.jar 0700
>
> There is also some good case to have a file non-readable, but in /usr,
> there is none. ( and in this case, this seems like the package is broken
> )
>
> So, to summarize, the proposal is :
> - block upload on :
> * missing-lsb-keyword
> * malformed-line-in-lsb-comment-block
> * useless-provides
> * non-standard-group
> * empty-%postun
> * empty-%post
> * invalid-desktopfile
> * unknown-lsb-keyword
> * standard-dir-owned-by-package
>
> discuss for blocking :
> * backup-file-in-package
> * version-control-internal-file
>
> discuss and find a list of exceptions for :
> * world-writable
> * non-readable
> ( especially the non-readable part ).
>
> If no one protest, the 1st list will be added before I take my vacation,
> in a few days.

Let's go!