Re: CVS commit: pkgsrc/multimedia/vlc

[Joerg Sonnenberger <joerg%britannica.bec.de@localhost>]

On Sat, Nov 12, 2011 at 02:01:57AM +0100, Alistair Crooks wrote:
> So please do NOT make changes based on the tinderbox "findings";
> rather, I'd much prefer it if you set PKG_DEVELOPER=yes (which, as wiz
> points out) does most of the work that tinderbox does, but for all
> of the pkgsrc platforms as a whole. 

To go a bit more into detail here, PKG_DEVELOPER includes:

- Missing files in $PREFIX
- Additional files in $PREFIX, unless explicitly marked so
- Missing shared library dependencies
- Missing interpreters in #! lines
- References to the $WRKDIR with different levels, relatively relaxed by
default
- Checks for sane permissions (e.g. no world writeable files and
direcotries)

Additionally, bulk builds will install and deinstall the package to test
INSTALL/DEINSTALL rules. Note that by default, only the package itself
will be deinstalled. For pbulk, there is an option to force a full
deinstall cycle, but no checks for left-over directories ATM. It is
quite expensive to do after all.

There is a reason why the tinderbox port wasn't received with standing
ovations. What it is providing over the existing infrastructure is quite
small. The only useful functionality is better guessing of what changed,
which is easier for Ports due to the much smaller number of .includes,
and better automated report handling as part of pointy hat. The sanity
checks for packages are generally done already.

Joerg