Allow setting fuzz factor  to 0
- use --fuzz=num instead of -Fnum or -F num to be sure
  /usr/bin/patch won't complain

Don't use static buffers

Fail if it is impossible to chdir to directory to run helper script in

Fix find-requires (rhbz#443015), thanks to Bill Nottingham

Sanitize + simplify %patch "macro" parsing
- Use popt for parsing %patch options, give accurate error messages for
  invalid options instead of bogosity like "bad %patch arg: %build"
- Store all the variants we can get patch numbers into argv to avoid
  special cases when passing to doPatch(), this has the added bonus
  of removing the 1024 patches per line-limit (as if somebody cared)

Single exit point for doPatchMacro()
- assuming failure avoids several redundant free() + return RPMRC_FAILS

Remove ugly hardcoded docDirs list, use a macro instead
- %__docdir_path contains a colon-delimited list of documentation dirs,
  macros expanded before use
- dynamically allocate docDirs as needed using an argv array, removes
  the hardcoded MAXDOCDIRS limit on docdirs (not that it matters much)

Avoid entirely silly rpmio FD_t use in processPackageFiles()
- since we just convert it to FILE anyway, no much point in using
  Fopen() then
- additionally remember to close the file if expandMacros() fails

Change rpmGlob() argvPtr type to ARGV_t to make it obvious what it is
- also fix incorrect doxygen commentary, it wasn't a contiguous blob
  before either

Use argvFree() instead of manual free in loop

Further simplify rpmInitMacros()
- Use argvSplit() for splitting the macro path to components instead of
  manual pointer-parsery.
- If URL's are to be supported or accepted at all (previous code attempted
  to skip them), ':' is a very poor delimiter character.

Use argvFree() to free, well, an argv, instead of manual freeing

Simplify rpmGlob() a bit
- use argv*() routines instead of manual argv manipulation

Dynamically allocate buffers in rpmGetPath()
- avoid unbounded copying to static sized buffer
- use rpmExpand() instead of expandMacros() which requires preallocated
  buffer when we've no idea of expanded size

Avoid entirely silly rpmio FD_t use
- rdcl() converts to FILE internally anyway, might as well use fopen()
  to begin with...

Remove unnecessary xstrdup() of dynamically allocated buffer
- leftover from when target buffer was static...

Cosmetics: group operations more logically...

Simplify macro option + argument processing
- eliminate ninja-voodoo pointer-hackery parsing, replace argvSplit() +
  argvJoin() and friends
- eliminate static sized buffers from the processing

Add argvJoin() for combining an argv array into a string

Put '\0' to the end of xmalloc'd string to avoid surprises

Eliminate splitString() and freeSplitString()
- use ARGV_t instead, it does the same and more...

Accidental typo introduced to rpmrc.c
- seems to be one of those days, sigh

Fix dumb off-by-one errors introduced by rstrlcpy() changes

Bunch of strncpy/strcpy -> rstrlcpy() converts in rpmrc

Add missing include

Use rstrlcpy() for various parseForFoo() functions in build
- just avoids manual termination, no new fixes here..

Use rstrlcpy() for parseRCPOT() version + release
- just avoids manual zero-termination

Use rstrlcpy() for printFileInfo() username + group fields
- just avoids having to manually ensure zero-termination

Make sure lead->name is zero-terminated

Add rstrlcpy() string helper function
- adopted from strlcpy() function of OpenBSD originally developed by
  Todd C. Miller <Todd.Miller@courtesan.com>

More accurate filtering of .rpmnew etc when reading macros
- strstr() can give false positives, use rpmFileHasSuffix() instead

Have gcc check for NULL terminated arglist in rstrscat

Use sizeof() instead of direct element count in rpmluaGetVar()

Don't copy unknown number of args to buffer of fixed size in initGlobs()

Introduce rstrscat()
- concatenates NULL terminated strings to newly (re)allocated memory
- purpose is to prevent overflows caused by writing unknown-sized stuff
to static arrays

Fixup leftovers from lua/local -> luaext/ move

Document external Lua requirement

Convert to Lua 5.1 library loading style
- luaL_openlibs() to pull in all standard libs
- local extensions need to be registered by calling through Lua

Add modemuncher + lua init stuff to extra dist
- modemuncher.c gets included from lposix.c, urks

Start phasing out internal copy of Lua
- don't build internal copy of Lua
- move 3rd party extensions (posix and rexlib) to toplevel luaext/
  directory, built by default (unless --without-lua specified)
- auto*foo checks for external Lua
- minimal tweaks to lposix.c and rpmlua.c to get them build with Lua 5.1

Don't use mktemp(), we are sure mkstemp() is available now

Require mkstemp()
- mktemp() and tempnam() are dangerous

Kill queryHeader(), it is not used and buggy

Kill useless initialization to "(unkown error)"

Plug a memleak from pgpHexStr() in showQueryPackage()
- fmd5 is only needed in a single place, move alloc+free there to avoid
  having to deal with the umphteen continue cases

Don't try to print buf if it's empty

Remove insecure hacks in query.c
- remove useless flushBuffer() function
- update printFileInfo() to not to write to unknown sized buffer,
  but to print stuff which is its sole purpose

Remove useless statements from showQueryPackage()

Make headerNVR() just a special case of headerNEVRA()

Oops, only free if allocated ;)

Plug a memory leak in armorFormat()

Doxygen parameter name mismatch

Get rid of string hackery and overflows while rpm -V

Plug a memleak in pgpArmorWrap()

Introduce rstrcat()

Make find-lang.sh --with-man brp-compress friendly (rhbz#440994)

Patch from Ville Skyttä

Don't exempt *.jar.so.debug from buildroot checks (rhbz#442264)
- jar-kludge no longer necessary...

Patch from Ville Skyttä

Remove rhnplatform, cvsid/svnid, serial and copyright tag from rpmbuild

Abort build on unknown payload compressor
- explicitly check for supported rpmio names, bail if not supported

Avoid unnecessary rpmio_flags alloc+free in psm

Fix indentation, no functional changes

Remove glibc upgrade-helper kludge, only necessary for some ancient RHEL

Lose useless variable

Split scriptlet execv() preparations to separate function
- runScript() is way too big and still is, but this helps a bit...

Use argvAddNum() for passing package count numbers for scriptlets

Add argvAddNum() for adding integers to ARGV_t arrays

Use ARGV_t for passing around scriptlet args psm internally
- simplifies allocation and freeing while getting rid of alloca()'s
- runScript() needs to add arguments anyway, might as well pass in something
  that *can* be modified without having play games with partial const,
  partial alloca() contents in the argv
- removes the need for passing argc around
- (unrelated) move fi->h NULL check in runScript() to beginning *before*
  we trie to access it, duh...

Add argvNew() to create an empty argv array
- yes it's silly ;)

Avoid redundant script name formatting
- remove two points of return to simplify allocations..

Reduce #ifdef clutter

Streamline lua scriptlet name formatting - just do it once

Move legacy prefix compat thing out of the loop
- additionally fix indentation, move block local variable def there...

Don't bother calculating max prefix lengths
- no longer needed, prefixBuf is dynamically allocated as needed

Prevent buffer overflow in readKeys()

Use dynamic allocation in rpmlibNeedsFeature()

Don't warn about BuildRoot being "/" while it is actually empty

Simplify url retrieval in rpmInstall()
- use rpmMkTemp() instead of manual kludgery to static buffer

Simplify url io a lot
- move urlhelper call to urlGetFile()
- have urlOpen() call urlGetFile() instead of going the other way around
  and causing local copies and all sorts of craziness in the process

Remove #ifdef NOTYET junk

Rewrite rpmMkTempFile() for sanity
- Actually use mkstemp() for creating the temp file and return a FD_t
  dupped from the file descriptor returned by mkstemp().
- Simplify the interface while at it.
- Change callers for the new interface.
- Yes we now require mkstemp() to work, mkstemp() is in POSIX.1-2001 and
  this is year 2008...

First crack at adding ISA provides to packages (rhbz#235755)

- Horrible kludgery to get the isa names and bits into platform specific
  macros from installplatform script. That beast needs to die. I mean really

- In build, add provides: name(isa) = evr automatically when it makes
  sense (similarly to name = evr provides). ISA consists of ISA name and
  bitness (or wordsize). This can be used to correctly
  express multilib dependencies without resorting to (expensive!) file
  dependency kludges, eg for dlopen()'ed libraries where automatic
  dep extraction doesn't force dependency on 32bit vs 64bit version, you
  can now use:
      Requires: foo-plugin%{?_isa}
  This expands to foo-plugin(x86-32) for i?86 packages, foo-plugin(x86-64)
  to x86_64 etc, and permits spec to be shared with older distros which
  don't have ISA provides.

- The same could be expressed with "canon arch" just as well, but
  using the ISA to differentiate from %_arch and the like:
  eg i386 could be used instead of x86-32 but it's overloaded with meanings
  (the actual i386 processor vs i386 compatible cpu family etc)

Avoid exporting providePackageNVR() in the library

Just call legacyRetrofit() instead of combined compress + provide retrofits

Add a static, far simpler addPackageProvides() to build
- use instead of providePackageNVR() which needs to deal with all sorts
  of hysteric "rpm 3.0.2 did this but 3.0.5 didn't" issues

Call rpmReadConfigFiles() in rpm2cpio so that rpm2cpio won't complain like:
warning: Unknown system: (null)
warning: Please contact rpm-maint@lists.rpm.org

Remove unnecessary providePackageNVR() from writeRPM()
- packageBinaries() already calls providePackageNVR(), no need to do it twice

Unbreak srpm generation broken by 6c4b0cbd93709a0fa18af1ef07af44da585bb5f6
- cpio path in source packages is the basename of the file, make this
  obvious by using basename() instead of pointer games

Don't use stack allocations in rpm2cpio when not needed

Replace alloca+memset with xcalloc() in rpmdbRemove()
- return value is suspect: should return "ret" instead of 0 at exit
  depending on what happened in removal but preserving previous behavior
  for now...

Replace alloca+memset with xcalloc() in rpmdbAdd()

Replace alloca+memset with xcalloc() in rpmdbCountPackages()

Tonne of indentation + other formatting corrections to rpmdb.c
- no functional changes

Remove dead code

Report full nevra in rpmdb remove debug message

Plug memleak from signature verification now returning malloced msg

Simplify addTE()
- eliminate wtf parsing of NEVR string which doesn't actually handle epoch
- use NEVR and NEVRA formatting from headerGetN*() instead of doubling the
  work here

Make headerGetNEVRA() return "src" as the arch of source packages

Initialize pointers, lowercase local variable

Rewrite headerGetN*() for simplicity
- eliminate stpcpy() games
- reuse evr formatting from headerGetEVR() instead of doing it over and
  over again everywhere

Fix buffer overflow in verifyDSASignature()
- caused by assumption that sizeof(size_t) is always 4 (credited to jbj)