[csw-maintainers] Updated findutils package (4.4.2) in experimental/, please test

[Dagobert Michelsen]

Hi Gordon,

Am 26.08.2010 um 19:07 schrieb Gordon Marler:
> findutils 4.4.2 is now being packaged under GAR, instead of manually  
> as it was before.
>
> http://mirror.opencsw.org/experimental.html#gmarler
> pkgutil -t http://mirror.opencsw.org/opencsw/experimental/gmarler -i  
> findutils
Great!

> At present, testing is disabled, since 12 tests in the findutils  
> suite, and it appears that they were never successful in the  
> manually built package either.
>
> The failed tests are explicitly documented in the Makefile, if they  
> are deemed important enough to fix.  At least 4 of them appear to be  
> simple permissioning errors in the testing area setup by findutils  
> own scripts.  The remaining failures have to do with the iregex  
> routines pulled in from the built-in regex library apparently  
> extracted from some release of GNU libc.

The recommended cause of action is:
1. File a bug upstream
2. Put a comment about the failing tests before "SKIPTEST ?= 1" together
    with a link to the upstream bug with something like "Tests are  
disabled
    until these bugs are fixed: <url>"

> The following minor changes were made:
> - Binaries are still 32-bit, but generated for extra ISAs sparcv8plus 
> +vis and pentium_pro

Do these really provide any advantage? Adding an extra ISA is  
recommended
only if benchmarking showed any difference.

> - Testing is currently disabled, but majority of tests pass
>
> Any testing would be appreciated.

I suppose your email in your .garrc is wrong: gmarelr at opencsw.org

I have two more issues about the postinstall:
1. You should use the cswcrontab CAS instead of rolling your own
       http://wiki.opencsw.org/cswclassutils-package#toc23
2. It should honour autoenable_daemons in csw.conf (same document)
    If it is not set IMHO there should be no crontab entry.


Best regards

   -- Dago