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)

Replace bunch of wtf evr formatters with headerGetNVR()

Add headerGetEVR() convenience function
- return formatted evr string + optionally pointer to name

Make headerNEVRA() actually return epoch too
- ...but epoch is an integer type in headers, change proto to match that

And one more url->path rename for local variable...

Rip the last remaining disk/file URL stuff in processSourceFiles()

And more url ripping in build internals... (addFile)

URL -> path renames continued... (build internal)

Rename FileListRec_s disk + cpio path variables
- on-disk is diskPath, rename previous fileURL to cpioPath to make it
  more obvious it's the path in the package cpio
- neither is an URL

Yet more url ripping - buildroot is never an url
- this is a bit nasty as it changes public interfaces but rename to make
  it explicit they're just plain paths, not urls

Remove remote host stuff from build doScript()

More url-path ripping in build/
- processBinaryFile() and processMetadataFile() don't need steenking urls

Whoops.. gpg-pubkey "release" on import got busted accidentally

Simplify rpmtsImportPubkey() a bit

Rewrite pgpArmorWrap(), the former one was insecure/hackish

Collapse unknown / unsupport RSA sigtypes into one fallthrough

verifyRSASignature() buffer & other fixes
- avoid stpcpy() into static sized buffer
- avoid potential sigp NULL-dereferences

verifyDSASignature() buffer & other fixes
- avoid stpcpy() into static sized buffer
- avoid potential sigp NULL-dereference

verifyDSASignature() buffer & other fixes
- avoid stpcpy() into static sized buffer
- avoid potential sigp NULL-dereference

Eliminate four copy-paste versions of nibble(), stick to rpmstring.h
- the one in signature.c wasn't even used...

Eep, fix bunch of segfaults and other fun from previous commits
- leftover stpcpy()'s to non-malloced addresses and other "minor"
  issues in verify*Signature() stpcpy() elimination ;)

Avoid stpcpy() to static sized buffer in verifySizeSignature()

Avoid stpcpy() to static sized buffer in verifyMD5Signature()

Avoid stpcpy() to static sized buffer in verifySHA1Signature()

Eliminate several copy-paste hex converters, use pgpHexStr() instead

Uh, return allocated pointer, not the end of string...

Remove static print buffer from pgpMpiStr(), return malloc'd data instead
- convert callers
- only rpmpgp.c internal needs, move it there and make static?

Merge pgpHexCvt() into pgpHexPrt(), removing it from public headers

Minimally convert all pgpHexCvt() users to use pgpHexStr() instead

Remove static print buffer from pgpHexStr, return malloc'ed memory instead
- inlined to get xmalloc() from system.h for consistent malloc fail behavior
- convert callers for new behavior, apart from some debug fprintf()'s

argvPrint() doesn't modify argv, make it const

Add ARGI_const_t, use where appropriate

ARGV_t const pendantry
- const on typedef'ed ARGV_t doesn't mean a thing, add a new,
  (exec* compatible) ARGV_const_t and use where appropriate consistently in
  argv.h, rpmcli.h etc
- popt's argv const is the wrong way around for exec() family, add explicit
  ARGV_t casts on popGetArgs() uses where needed
- compiler silence, aaahh...

Remove ARGstr_t typedef, it serves no purpose
- argv strings are just regular strings, the structure is what's special

Remove bogus const from ARGV_t

Missing rpmstring.h include

Eliminate the uuuuugly and unsafe static tagname buffer
- generate the "human friendly" strings at build time, store into the tag
  table structure
- also avoids copying of strings and lowercasing strings at runtime

Move the tagtbl.c generator monster awk out of Makefile.am
- stick it into a separate script, reformat so it's possible to actually
  read it...

Oops, no newline at end of filename please...

%{_builddir} is not an url, ever

We only ever deal with local, regular files in doUntar()

We only ever deal with local, regular files in doPatch()

Rip the url-craziness in rpmMkdirPath()

Get rid of the horrid zcmd hackery in build
- rpmExpand() with %uncompress macro etc simplifies things a lot

Missing decompression arguments for bzip in uncompress macro

getcwd(NULL,0) has undefined behavior, use rpmGetCwd() instead

Stop pretending spec can be an URL or stdin, neither works
- simplifies buildForTarget() somewhat
- while at it, eliminate remaining unsafe string operations

Escape '%' in format

Wait for popen() to finish before passing to isSpecFile()

Move spec extraction from tarball out of buildForTarget(), simplify
- use basename() and dirname() instead of manual pointer manipulation games
- static buffer only used for reading tar output, sizes checked

Oops, add --wildcards back to spec extraction
- shouldn't really have to hardcode such stuff but...

Simplify formatting

Simplify package comparison and format creation.
- also avoid static buffer usage

Assume failure in buildForTarget() to remove redundancy

Refactor spec extraction on tar build to address several issues
- simplify the code
- use %{__tar} instead of tar as the extraction command
- avoid sprintf to "big enough" buffer
- basic spec sanity check while extracting, fixes rhbz#281391

Spec of size zero cannot be a valid spec file

Remove copy-paste currentDirectory() from sqlite, use rpmGetCwd()

Rename currentDirectory() to rpmGetCwd() and move to librpmio

We don't support url's in db paths, rip.
- file:// vs / is silly and anything else wont work, so pretending
  url support just plain bogus

Replace silly static rpmioFileExists() in rpmdb.c with plain old access()
- accepting url's there is not only useless but just plain bogus

Remove ancient db1->db3 conversion cruft

Don't use static buffers in parseForRegexLang()
- use dynamic allocation instead to avoid overflows
- also update addFile(), which is actually the only function
  that calls parseForRegexLang()

Lose the remaining static buffer from rpmVerifySignatures()
- Simple and stupid: catenate previous buffer + latest message
  with rasprintf() over and over again. Not the most efficient way but hardly
  matters here...

Typo...