Simplify rpmInstallLoadMacros()
- fi argument unused, remove
- no relevant return value, make it void
- handle any proper data type for per-header macros

Avoid reconstructing half of rpmfi manually in rpmInstallSourcePackage()
- similarly to commit 53ee682a491ceee4392815075cabbe58d0cc0c22, arrange
  the header to contain necessary paths before calling rpmfiNew()
- avoid unnecessary accesses to rpmfi internals otherwise too

Unnecessary local variable, result only needed once

Assume failure in rpmInstallSourcePackage()
- avoids bunch of redundant assignments

Calculate total file size correctly in build (mdvbz#45820, rhbz#247374)
- delay total size calculation until the real file list is known, ie
  in genCpioListAndHeaders() where duplicates and excludes have been
  weeded out

Make depflagsFormat() accept any numeric type

Simplify xmlFormat() by taking advantage of rpmtdClass()

Make fflagsFormat() accept any numeric type

Make permsFormat() work with any numeric type

Make triggertypeFormat() work for any numeric type

Make shescapeFormat() to accept any numeric type

Take advantage of rpmtdGetNumber() for date format specifiers
- dateFormat() and dayFormat() now work for any numeric type

Take advantage of rpmtdGetNumber() for numeric format specifiers
- octalFormat() and hexFormat() now work for any number, not just int32,
  fixing rhbz#471820
- use a common helper-function for both formats, they only differ by o/x

Simplify stringFormat() by using rpmtdGetNumber()

Add rpmtdGetNumber() for getting numeric values from tag container
- returns the value (as opposed to pointer to, like the rpmtdGetUint32()
  and the like do) of any numeric type as uint64_t (largest supported
  integer type so everything can be converted to it)
- handy when you don't really care what the internal presentation is
- there's no rpmtdGetNextNumber() as there's no meaningful way to return
  end-of-iteration here

Introduce rpm tag "classes"
- rpm tag data can be either numeric, strings or binary data, each with
  their own "subclasses" (different sized integers etc), add new
  enumeration for these
- add rpmTagGetClass(), rpmtdClass() public functions for retrieving the
  base class of tag and container
- useful for getting a basic idea how to handle tag/container data

Use rpmpsmNew() + rpmpsmFree() instead of manually doing the same

Eliminate bogus const from fi->apath

Eliminate match iterator from psm struct
- it's only needed in local scope anywhere, no point having it psm global

Avoid having to reconstruct half of rpmfi manually in genCpioListAndHeader()
- We need the on-disk paths for the cpio list in here, so instead of
  creating an rpmfi with wrong paths (for build purposes) and then
  painfully reconstructing half the fileinfo structures by hand, pass
  the on-disk path into rpmfiNew() to begin with and then switch the
  intended cpio path back into header.
- Duplicates and excludes are already filtered out from the header filelist
  so walk that instead of spec filelist to avoid having to filter out
  duplicates and excludes, again.

Lose useless fsm/rpmfi astriplen field
- only "used" for debugging output, duh...

Don't bother creating a dummy transaction element for rpmfi in build
- nothing in the build path requires fi->te to be present, so don't
  muck with rpmte internals needlessly

Minor genCpioListAndHeader() cleanups
- rpmfiNew() only cares about ts for install-time relocations, nothing
  to "fix" wrt that
- we know fip is never NULL (this is a static function with whopping two
  callers), don't try to cover up for potential programmer errors

Don't have to free what's not allocated to begin with...

Add more flags to control rpmfi creation
- RPMFI_NOFILELANGS to avoid loading lang data (only useful on install)
- RPMFI_NOFILEOWNER to avoid loading of user+group data

Yank rpmfi state update out of rpmtsRun()
- stuff it into internal rpmfiUpdateState() function to get it out of
  sight for now, the mechanism needs some...

Move open + close of files during install to separate functions
- internal rpmteOpen() and rpmteClose() functions replacing copy-paste
  slop between rpmtsRun() and runTransScripts()
- eliminates bunch of rpmte privacy violations
- rpmtsRun() doesn't need the file descriptor for anything, might as well
  keep it hidden

Allow setting custom parameters to patch (rhbz#471006)
- added new _default_patch_flags macro with the rpm default
  "-s" parameter

Fix capability verification
- the previous implementation was pretty busted up, cap_size() is the
  size of external representation, not sizeof(*cap_t) as pointed out by
  Andrew Morgan
- check for cap_compare() in configure and use it if available (it's
  a Linux-specific extension to the posix draft and only very recent
  libcap has it)
- if cap_compare() isn't available, use internal variant that does it
  the hard way by converting the capabilities to external presentation
  and comparing those

Filter out mtime difference of shared files on verify (rhbz#426672, #444661)
- Since we permit shared files to differ by mtime on installation, it makes
  no sense to consider them as verification failure either. This has been
  especially bad on multilib where timestamp differences make verify warning
  lights look like Las Vegas at night. If file mtime on disk and header
  differs, check if it's owned by more than one package and if so, ignore
  the timestamp difference.

Move ts and preferred color fetching out of the loop
- neither ts or preferred color changes per-file

Remove unused fsm subbuf field

Don't bother retrieving db offset for erase elements
- not needed anymore, use -1 to mean "unused" and document dboffset
  to rpmtsAddEraseElement() as unused

Take advantage of headerGetInstance() in removePackage()
- avoids an extra argument, making dboffset argument to
  rpmtsAddEraseElement() unused

Take advantage of headerGetInstance() in rpmte
- rpmte db_instance and dboffset are exactly the same thing, unify
- pull rpmte->db_instance from header directly, ignore dboffset argument
  to rpmteNew() (removing the arg would just break api needlessly, otoh
  it's unlikely that anything outside rpm actually uses rpmteNew())
- eliminate the now unnecessary added/removed union from te struct
- rpmteDBOffset() is now just an alias for rpmteDBInstance(), deprecate?

Grab rpmfi record from header on rpmfiNew()
- avoids having to muck with it from rpmtsRun()

Add rpmdb record number to headers retrieved from rpmdb
- set "instance" number on retrieval from rpmdb
- add public headerGetInstance() function for retrieving the value
- ported from rpm5.org, useful for number of things

Convert obvious bits of rpmtsRun() to use rpmfiFAction()
- remaining access to fi->actions is the nasty part of copying things
  around to transfer state

Convert skipFiles() to rpmfi(Set)FAction()

Convert handleOverlappedFiles() to rpmfi(Set)Faction()

Convert handleRmvdInstalledFiles() to rpmfiSetFAction()

Convert handleInstInstalledFiles() to rpmfi(Set)Faction()

Add rpmfi get + set methods for file actions

Remove unnecessary initialization
- rpmfiNew() zeroes out everything, including striplen + astriplen

Move file action calculations to rpmfiNew() where possible
- for build operations and src.rpm install this is known

Cpio flags are only relevant for fsm, get them out of rpmfi

Figure out cpio mapflags in rpmfiNew()
- all the necessary info is there, no need to litter the logic all over
  the place

Push archivePos from rpmfi to fsm
- archive position is only meaningful in fsm context anyway

Adjust rpmfi directory count while messing with it in build
- otherwise we can miss some entries in rpmfiFN() max calculation and
  blow up in entertaining ways

Add the post-build package checker feature in rpmbuild (upstream ticket #2)
- allow to run helper programs to check:
  - generated SRPM
  - each generated RPM after it is successfully written
  - whole successfully written binary package set
- it is possible to use it for sanity checks

Add macros for the post-build package checkers feature (upstream ticket #2)
- _build_pkgcheck: helper that is ran for every binary RPM successfully
  written
- _build_pkgcheck_set: helper ran for the whole binary package set
- _build_pkgcheck_srpm: helper ran for SRPM
- _nonzero_exit_pkgcheck_terminate_build: indicates whether the build
  should fail if helper returns non-zero exit status

Reset return values to zero + NULL's on not found in hash GetEntry()
- avoids having to separately check for return value in some cases and
  accidents from not resetting the values in caller
- in line with headerGet() behavior

Rename doLookup to fpLookup to get rid of one function just calling the other

cleanup fpLookupSubdir
 * rename end -> endsubdir
 * fix assignment mess
 * fix memory leak (no longer free current_fp.subDir which may be NULL)
 * set endsubdir to NULL while not valid

Reverse hash GetEntry() return code vs found/not found
- be in line with hash HasEntry() and other similar functions
- internal-only function with just one place actually checking the return

rpmsqAction() proto needs SA_SIGINFO conditionals too (ticket #4)

Nuke the ugly hardwired "known arch" list from label parsing
- we have list of known archs in rpmrc, might as well use it
- avoids having to patch & recompile rpm just to have rpmtsInitIterator()
  recognize new archs when it's runtime configurable otherwise
- checking against compatible arch would be saner except then people could
  get into even more trouble using --ignorearch...

Calculate filename buffer len lazily in rpmfiFN() as needed
- avoids having to separately calculate in genCpioListAndHeader()
- avoids unnecessary fnlen field in rpmfi struct
- avoids having to calculate it at all if rpmfiFN() is never called

Remove yet more unused fields from rpmfi struct

Remove fperms and dperms from rpmfi struct
- only fsm internals need and just use straightforward defaults, move
  the defaults there...

Use cpioMapFlags type consistently for map flags

Stop messing with fi->fstates allocations in psm
- it's alloced and freed by rpmfiNew()/Free() always anyway

Kill off per-file cpio map flags
- nothing uses/needs them, one less murky fi->foo allocation less
- apparently a leftover from initial multilib attempts long long ago

Remove unused variable

Remove useless rpmfi->gid and uid "defaults"
- they were only ever set on src.rpm installation yet overridden by
  package contents unless src.rpm didn't contain RPMTAG_FILEUSERNAME
  (and GROUP). Which would mean rpm 1.0 packages or thereabouts, or a
  malformed package. In that case we just fall back to root uid,
  it hardly matters

fi->actions is already initialized earlier in rpmfiNew()

fi->actions cannot be non-NULL in rpmfiNew()

Dead code removal

Stop messing around with rpmfi user+group names from psm
- these are loaded and freed by rpmfiNew/Free() as necessary

Fix segfault on relocation (rhbz#468319)
- a bogus free() / thinko

Avoid leaking C99 options into stock %{__cc} macro

Return proper exit code when exec*() functions fail
- the correct exit code should be 127 accoring to bash(1),
  we want to keep the same exit code behaviour as bash for rpm

Fix building on systems where SA_SIGINFO isn't available (ticket #4)
- patch from Adam Tkac

Use more portable file access mode tests (ticket #5)
- don't make assumptions about how O_RDWR and friends are defined, use the
  portable way of masking flags with O_ACCMODE and explicit comparison
  against the mode we're interested in

Ensure PATH_MAX is defined (ticket #3) Adam Tkac
- POSIX doesn't require PATH_MAX to be defined as constant, and Hurd
  doesn't define it...
- deal with it just once in system.h for now, the proper fix is to get
  rid of needing it at all (ie add a mallocing realpath() clone to librpmio)
- patch from Adam Tkac

Drop unnecessary trailing slash in pkgconfig includedir

Adjust for file output change (rhbz#468129)
- file util used reported file details along with mime type, newer ones don't

Kill of scareMem / keep header flag from rpmds
- unnecessary complication which doesn't benefit anything (anymore)

Change all in-tree rpmdsNew() uses to non-scaremem
- shouldn't affect anything at all

Remove unnecessary scaremem flags, hardwired to no scaremem anyway
- no functional changes

Save bunch of memory by not loading file classes in ts elements
- install/remove doesn't need or use file class information at all,
  don't bother loading it, saves a few megs of memory on large transactions

Add couple of new rpmfi flags for file classes and file depends

Move rpmfi color union calculation out of rpmfiNew()
- nothing at all uses the calculated value or rpmfiColor() call, take
  the unnecessary calculation out of common path into rpmfiColor()

Remove unused package color calculations from 2003

Don't bother fetching and storing pre- and posttrans scripts in rpmfi
- the psm script machinery requires the full header to do anything at
  all, so the script + scriptprog were only used to check if the package
  *has* such scripts, a single integer will do just fine there thank you

Convert in-tree users of rpmfiNew() to use flag names
- easier to grep for than scareMem, preparing for further flags

Turn rpmfiNew() scareMem argument into bitfield of flags
- preserve behavior with 0/1
- more room for controlling what gets loaded into file info set, not
  everything is needed at all times

Set timezone to UTC every time to avoid local timezone confusion

Add --filecaps popt alias for looking at file capabilities
- doesn't need much as the capabilities are stored in human readable format

Add proper file capability verification support
- at least for now, no capabilities is treated like empty capability set
  which are not exactly the same thing but probably sufficient for our
  purposes

Add fsm and rpmfi machinery for file capabilities
- new rpmfiFCaps() API to retrieve the info from rpmfi set
- fsm internals quite similar to selinux handling
- plenty of #ifdef's, another possibility would be adding cap_foo dummies to
  system.h like for selinux

Start adding POSIX 1.e draft file capability support for real
- Parse %caps() from spec filelists, making best-guess verification of
  capability string sanity by passing to cap_from_text()
- The posix draft specifies capability export presentation through
  cap_copy_ext() which would be fine, except that we don't have support for
  arrays of binary data. So we simply store the textual representation
  of the capabilities in a string array which we do have.
- Only add capability tag on packages which actually have capabilities to
  avoid unnecessary header bloat.
- Add a new rpmlib() dependency for file capabilities, packages relying
  on file capabilities wont work correctly unless the capabilities are set.
  To be fully correct, support for on-filesystem file capabilities should
  be checked at runtime, as this depends on kernel versions and such...

Don't try to verify capabilities of directories and non-regular files

fix compiler warning
by removing unnesseccary return address for the hash key

Beginnings of acl (POSIX.1e draft 17) verification support
- if built with --with-acl, check that files and directories don't have
  any extra acl's set
- for now, any acl beyond the regular unix permission set is reported as
  file mode difference as the acl's cannot have been set by rpm itself
- patch from Andreas Gruenbacher, modified to use libacl instead of raw
  xattrs for portability, BUT atm this uses non-portable acl_equiv_mode()
  Linux libacl extension, the posix draft doesn't seem to have much in
  the way of comparing entries :-/

Beginnings of file capability (POSIX.1e draft 15) verification support
- add minimal bits and pieces to check for capabilities in files on verify
- for now, any capability set is a verification failure as the capability
  cannot have been set by rpm itself
- patch from Andreas Gruenbacher, modified to use libcap instead of raw
  xattrs for portability

Remove timezone autoconf checks and system.h reference
- timezone is no more used so it's no more needed

Fix "timewarp" bug while listing changelog via "rpm -q --changelog"
- date is no more one day in advance than written in changelog

Use better cache sizes

Improve finger print hash function to avoid degeneration of hash tables