Replace equal/not equal uses of str[n]cmp() with rstreq[n] in fsm & psm

sameProblem() return value is illogical, reverse it

Replace equal/not equal uses of str[n]cmp() with rstreq[n] in rpmrc code

Replace equal/not equal uses of str[n]cmp() with rstreq[n] in backend code

Ugh, fix brokenness from previous commit

Replace equal/not equal uses of str[n]cmp() with rstreq[n] in io code

Replace equal/not equal uses of str[n]cmp() with rstreq[n] in build code

Hum, rename rstrneq() to rstreqn() to avoid confusing neq with "not equal"

Eliminate now unnecessary compressor -> ioflags conversions

Accept direct compressor names as Fopen() modes too

Switch rpmio mode checking to use rstreq()

Add two helper functions for comparing string equality

Eliminate unused result code from psm struct

Eliminate payload file handle from psm struct
- it's only needed locally inside PSM_PROCESS on PSM_INSTALL

Don't try to incref NULL ds

Remove extra trailing } copy-paste error from python_foo macros, duh

We always add EVR and Flags to dependencies, remove bogus condition

Fix duplicate dependency checking on build
- Broken by commit af8b41c64af39ce07d85fcd92fa78d566747d815 which
  simplified too much.
- There's no guarantee that rpmdsNew() returns a sorted dependency set
  so rpmdsFind() doesn't work correctly here. Walk the ds manually instead.
- With multiple triggers of same type, identical trigger conditions on
  different trigger script were seen as duplicates (RhBug:490378)
- Split the duplicate checking to separate helper function for clarity

Copy BUGURL to source rpm's too
- should've been in commit 86032bfbb61f1bdee9ac42989a14ea40bed0deec, duh

Add some commonly used python macros (ticket #83)
- macros to determine python version and library paths

Eliminate unused code (leftover from ordering rewrite)

Don't bother relocating packages no files
- fixes a small memleak

Avoid adding Lua sources and patches twice when recursing (ticket #82)
- parseSpec() operates on newly created empty spec structures even when
  "recursing" but the lua variables are global and need recursion
  protection to avoid adding multiple times

Add default %clean section unless overridden in spec (ticket #81)
- avoids silly clutter in every specfile in the regular case but permits
  overriding if needed for whatever reason
- it's a spec behavior change but mostly backwards compatible:
  existing specs will do whatever the %clean does, and if you need to avoid
  calling %clean for whatever reason just keep an empty %clean in the spec

Add -d option to %patch.

Add isSource() method to python header object (ticket #32)

Add %make_install macro that does the "right thing" wrt modern autotools
- old %makeinstall for broken/ancient autotools left untouched for
  backwards compatibility

Add bug URL macro so that it could be set distro-wide (rhbz#512774)

Update querytags test to match current output
- should've been in commit 86032bfbb61f1bdee9ac42989a14ea40bed0deec, duh

Build script environment tweaks to improve reproducability
- unset CDPATH (RhBug:426955) and DISPLAY environment variables
- always run in C locale

Don't set --target in %configure (RhBug:458648)
- --target is only ever useful for handful of compiler toolchain packages
  and cross-compiler packages are better off setting it themselves if
  necessary, rpm messing here only gets in the way

Nuke long since dead version of %configure macro + update commentary

Add an option bug reporting URL tag to packages (RhBug:512774)

Add missing <pthread.h> include in rpmsq.h (ticket #87)
- patch from Michal Čihař

Bump version to appear newer than 4.7.x branch

rpmbuild won't segfault when %including spec with zero size

Add *.xz and *.lzma recompress support to brp-compress.

Correctly inherit default %attr(-,-,-) attributes from %defattr (rhbz#515685)

Kill off lclint remnants everywhere
- remove ancient and likely irrelevant LCL comments and bogus NULL checks

Fix memory allocation for token array

This fixes a memory corruption due to write access out of
bounds of token array, whose size was computed incorrectly.
It was assumed that only '%' characters separate tokens,
which could lead to crashes on useless uses of '[' tokens,
such as "rpm -qa --qf '[]lalala'".

Log an error on short lead (RhBug:511101)
- if lead read returns short without it being an IO error, treat it
  as a case of "not an rpm package"

Honor --nofiledigest on install too (RhBug:508021)
- query and install options clash, use popt callback to enable processing

Document --nodeps in rpmbuild manual (RhBug:482921)

Dont accept '-' in manifests (RhBug:461353)

Dont treat filenames starting with - as stdin

Oops, missing include for dirname()

Create the directory used for transaction lock if necessary
- otherwise at least rpm --initdb can fail due to transaction locking failing

Fix package relocatability checking, broken since forever
- at the time relocation problems are added, ts problem set is NULL
  and rpmtsCheck() wipes it out anyway so the problems from relocation
  are never seen anywhere
- The header isn't available at transaction prepare stage where most
  problems are checked for, so we can't validate the prefixes at that
  point. Add a per-element problem set to carry the problem data and
  merge it to ts problem set when checking for other problem types.

Make rpmpsUnlink() behave the other fooUnlink() functions
- pretty much everything else in rpm returns NULL from unreference functions,
  make be more consistent here...
- also tolerate NULL in both reference and unreference

Add reference counting to rpmProblems

Unbreak --excludepath (RhBug:409637)
- excluded paths aren't counted as actual relocations, dont bother
  checkig addPrefixes() return

Fix compiler warning on const free
- we dont need popt to set the file variable as we manually process

Make verify result not depend on verbosity level, uff... (RhBug:510213)

Add an index for obsoletes (RhBug:507702)

Unify extension + header data getter calling conventions
- simplifies headerGet() & friends a bit and reduces redundancy
- also just return an error instead of dying with assert on NULL td argument

Pass headerGet flags to extensions too (unused for now)

Eliminate unused calculations originating from the "8/98 bug"
- the tl calculation has been unused since commit
  5994a308bc062fff53ff0184b8dea15d19fcfb63 in 2001 and we dont even
  read rpm 2.x packages anyway...

Re-enable rdlen vs dl header check
- we dont even read packages from the "8/98 bug" era anymore, hardly relevant

Band-aid to dependency caching misbehaving with chrooted verify (RhBug:508074)
- disable dependency caching on chrooted verify to avoid ugly spew
  from paths containing outside paths while we're actually inside the
  root during verification
- correct fix would be to fix the temporary db path calculation

Handle hook unregistering itself

Fix rpm.unregister() Lua extension
- hook data is userdata, not light userdata

Add base64 encode and decode extensions to the lua interface

Base64-encode %policy files to ensure it can be presented as strings
- RPMTAG_POLICIES is a string array but there's no guarantee that
  something marked %policy is a plaintext file that can be represented
  as \0-terminated string, base64-encoding them fixes that. Baby steps
  towards making %policy remotely usable, related to RhBug:505066.
- Also remove unnecessary failure code setting, processMetadataFile()
  assumes failure already, and dont try to insert NULL strings in case
  b64encode() or pgpArmorWrap() fails

Kick out silly constfree() use

Simplify logging a bit
- precalculate the message length, relying on C99 vsnprintf() semantics
- generate the log message and rest of the record in rpmlog() already,
  and just pass the record to lower level to do actual logging
- also something was wrong in the old version as valgrind complained
  about illegal reads, that is also cured here

Set HEADERFLAGS_ALLOCATED centrally from headerLoad()
- requiring every caller to handle this separately, violating header type
  opaqueness in the process, doesn't seem that bright an idea
- also fix a memleak on signature header read in case of failure

Teach %files to accept multiple filelists (ticket #70, rhbz#475359)

Allow ':' character in EVR, it's needed for Epoch separation

Check arg in argvJoin() before dereferencing

Updated Polish translation (ticket #74)

Fix some typos in javadeps usage message (ticket #72)

Add -h/--help to rpm2cpio (ticket #63)
- patch originally from Debian, with slight adjustments to indentation

Updated German translation from Micha Pietsch (RhBug:506607)

Updated German translation from Micha Pietsch (RhBug:506607)

Pile of OSGi dependency generator fixes from Alphonse Van Assche (ticket #39)
- use Temp perl module to provide temp dir
- re-enable deps solving in Require-Bundle, Import-Package, Export-Package
  OSGI properties
- remove uses bundle of Export-Package OSGI property
- use RPM '>=' as version operator to match OSGI '='
- remove all .0 at the end of the version string
- typo fixes

Delay fingerprint cache init + lookup until we know there are some matches
- no point going to fs if there are no matching files in the rpmdb
- use allMatches count as size hint to fingerprint cache instead of
  arbitrary 20

Permit NULL in fpCacheFree()

Fix invalid memory access from freeing dirName too early
- if it doesn't crash, it can cause rpmdb provided file not seen in chroot,
  the other half of RhBug:506323
- streamline exit points to enable freeing allocated resources sanely

Fix Segfault for to be installed symlinks pointing to existing locations
fixes rhbz#505777

Eliminate now unnecessary artifacts of rpmdsNext() side-effects

Generate DNEVR string lazily on rpmdsDNEVR() calls
- avoids useless string churning on rpmdsNext() when most paths dont
  actually use the DNEVR string for anything
- make sure DNEVR is freed whenever iterator index changes

Eliminate epoch promotion debug cruft
- might've been useful back in the day but...

Determine rpmlib deps from flags instead of comparing strings

Validate rpmlib dependencies on src.rpm install (RhBug:490613)
- other src.rpm dependencies are only relevant for building, but rpmlib
  features can affect src.rpm "install" too, such as unsupported file
  digests, payload compressors etc

Don't check Provides/Requires for bad characters, they could be UTF-8
- don't check Name but EVR in parseRCPOT()
- use rpm namespacing for charCheck()
- remove unused variable

Add debugedit support for DWARF-3 (RhBug:505774)
- patch from Jakub Jelinek

Rip hardcoded libgcc + glibc static helper kludges
- shouldn't be needed anyway anymore as package of preferred color should get
  always installed first, and the issue is solvable in packaging anyway

Check for valid chars in Requires/Provides as well

Fix out of source directory build (ticket #65)

Add parentheses (as kindly suggested by new gcc...)

Make peace with newer git

Make rpmalSatisfiesDepend() smarter
- Instead of blindly picking the first match, try to pick the best provider
  for the dependency: for colored dependencies, try to find a provider of
  matching color. For non-colored dependencies, prefer providers of
  transaction preferred color.
- This avoids creating bogus and loopy relations between 32- and 64-bit packages
  where they dont exist, and makes behavior with things like /sbin/ldconfig
  consistent by returning the preferred colored element.

Add preferred color to rpmal

Update translations for a change (due to depends.c -> order.c split)

Split ordering code to separate source file

Add new function to check valid characters in NVR (ticket #59)

Nuke useless compatibility stuff and put V/R checking where it belongs

Make whitelist of characters allowed in NVR (ticket #59, rhbz#493157)

rpmdsInit() and rpmdsNext() can handle NULL ds, no need to separately test

Minor rpmtsOrder() cleanup
- initialize on declaration
- remove unused code