Unconditionally include <sys/wait.h> where needed
- no point conditionalizing what we cant live without

Move sys/mman.h include out of system.h to the places that care
- just two places where needed, dont pollute system.h needlessly
- include depending on HAVE_MMAP instead of separately checking for
  sys/mman.h, if sys/mman.h doesn't exist or is broken HAVE_MMAP wont be set

Move libacl.h include out of system.h to the only place that cares

Eliminate SEEK_FOO and [RWXF]_OK definitions from system.h
- systems not defining these constants are broken beyond our caring

Remove unnecessary and broken time.h related configure + system.h checks
- the AC_HEADER_TIME check is unnecessary for any remotely recent systems,
  and the HAVE_SYS_TIME_H conditional in system.h is just broken as we
  dont even check for <sys/time.h> header in configure
- dont include from system.h, the time.h and sys/time.h get already
  included through our public headers where necessary

Unconditionally include <netdb.h> where needed instead of system.h
- there's no fallback for missing netdb.h so there's little point
  conditionalizing it
- avoid yet more system.h pollution

Remove pointless & unused configure checks
- vprintf(), vsnprintf() and snprintf() are in C99, no point checking
  especially as we dont have fallbacks for them
- no point testing for inline capability as we dont try to work around

Sanitize signature checking formatResult() a bit
- whole lotta unnecessary wankery here (what have I been thinking)
- avoid using the non-standard strndup(), we checked for it in configure
  but didn't provide an alternative so checking was just bogus

Oops, ftok() is really optional

Clean up poll() vs select() tests
- move the includes out of system.h, not commonly needed
- <poll.h> is conditional as we actually provide a fallback through select(),
  but for <sys/select.h> missing there's no fallback so doesn't make
  much sense to test for (and both poll.h and sys/select.h are posix anyway...)

Remove unused header checks from configure
- maybe these have been used at some point but not anymore...

Eliminate unused id -u tests
- this was used by _fixowner and _fixgroup macros back in the day but they
  no longer exist as they're unusable without building as root

Use MAP_FAILED instead of hardwired (void *)-1

Re-enable mmap test in configure & fix the code build again
- this stuff has been disabled for years, lets see what breaks...

Remove unnecessary stdint check from configure
- a leftover from pre-C99 requirement times

Remove S_IFSOCK and S_IFLNK kludgery for ancient unixes
- these are specified by POSIX 2001, it's not really rpm's job to
  provide them if the system doesn't

Require lchown() and remove ancient broken chown kludgery
- lchown() is POSIX 2001, we dont care about older / nonconformant stuff

Check for our POSIX 2001 requirements and abort if not found

Lose the hysterical fakefork hack
- we're not particularly interested in AmigaOS quirks from 1996

Remove further replacement functions for POSIX 2001 required bits
- getwd(), getcwd(), basename() and realpath() are all POSIX 2001
  defined, we dont care about older stuff

Remove unnecessary setenv() replacement function
- setenv() and unsetenv() are required by POSIX 2001, we dont care about
  older systems

Remove largely unnecessary putenv() replacement
- only the lua posix extension "uses" this by providing putenv()
  to Lua, make it conditional and return error if not supported by
  the underlying operating system

Eliminate myrealloc() replacement function
- realloc(NULL, size) is defined as equal to malloc(size) in C99 and
  POSIX 2001, we dont care about anything older than that

Remove unneeded strdup() replacement in misc/
- rpmio's own version is always there and portable, this is not needed

Use rpm's own strdup() clone everywhere

Always free locally defined macros when they go out of scope
- Prior to this, local defines in constructs like %{!?foo: %define foo bar}
  would remain defined until a parametrized macro gets called, causing
  obscure and confusing errors in specs such as RhBug:551971 and countless
  others. Use of %global is now always required for such constructs.

Remove -98 vintage ifdef'ed dead code

error.c was getting translated too, duh
- should've been in commit 5741334a857a783b033c647223f206a7ca43cef3

Oops, update stubs.c too for the removed replacements

Remove replacements for C99 mandated functions
- rpm requires C99 compiler anyway, no point in carrying these around

Remove unneeded replacement error/warn functions

Include <utime.h> where needed instead of system.h
- include unconditionally as we dont try to deal with utime() missing

Move major, minor + makedev portability stuff out of system.h, simplify
- only cpio.c needs, move it there
- if none of the standard headers define these, its not our headache anymore

Move getopt() portability fiddles out of system.h
- only macro.c needs, no point polluting includes everywhere

Dont include <pwd.h> from system.h, dont bother checking for it either
- no much point conditionalizing something we can't live without
- just three places need, no point polluting include space everywhere

Dont bother including <sys/socket.h>, we dont use anything from there

Dont bother including <err.h>, we dont use anything from there

Only rpmrc.c cares about utsname.h, move out of system.h

Dont include grp.h from system.h, dont bother checking in configure either
- no much point conditionalizing something we can't live without
- just two places need, no point polluting include space everywhere

Dont bother checking and including a header we dont use or need

Move malloc portability tweaks out of system.h
- only rpmmalloc.c needs, no need to pollute system.h

We dont use memchr() anywhere

Move getpass() portability tweaks out of system.h
- only signature.c needs, no need to pollute system.h

Dont bother checking for a header we dont use

Dont bother checking for functions we dont use
- some of this stuff was used in 4.4.x but not anymore
- additionally we dont even use getdomainnname(), check for gethostname()
  instead

Use AC_PROG_MKDIR_P for testing mkdir -p capability
- for rpmbuild's purposes MKDIR_P doesn't cut it though, we need to
  point it to something that exists on an installed system

Use %{__mkdir_p} macro in build instead of hardwired RPM_MKDIR_P

Remove unused h_errno test from configure.ac

Kill off last remaining uses of _constfree()
- practially all the data in rpmfi needs to be treated as const, these
  are just a funky special case which point to header memory for the
  couple of cases where KEEPHEADER is still used

Make header struct really opaque outside header.c
- the low-level stuff in signature.c and package.c need entryInfo_s
  and indexEntry_s but not the header itself
- also move the other defines which have no use outside header.c there

Remove redundant NULL-check

Eliminate unnecessary url parsing from rpmtsRootDir()
- url's in transaction root hasn't been permitted since rpm 4.6.0 days

Lose the now empty header_internal.c source

Move htonll() into header.c and make static, not needed elsewhere

Move the legacy compatibility goo into legacy.c
- no functional changes
- headerGetRawEntry() hasn't had a public prototype since rpm 4.0 days
  but there are some 3rd party users still (sigh), leave it alone as
  exposing the proto now would only cause *more* trouble to the external
  users

Move the higher level convenience functions out of header.c
- header.c is big enough as it is, and none of these things needs
  header struct internals for anything
- no functional changes, just moving stuff around

Rename hdrNVR.c to more obvious and generic headerutil.c
- no functional changes
- remove it from translation list, there's nothing here to translate...

Work on tsortinfo instead of ts elements when ordering everywhere
- ...except addRelation() which still needs rpmteTSI() for grabbing
  tsort info from elements in rpmal

Work on tsortInfo nodes directly when detecting SCC's
- pass the local order info struct to detectSCC() and work on that
  instead of ping-ponging to the TSI's through transaction elements

Record relations to tsortInfos instead of transaction elements

Save pointer to ts element in the tsort info struct

Avoid a bunch of redundant rpmteTSI() calls

Eliminate unnecessary depth member from tsortinfo
- its only used for debug log formatting and we can calculate it from
  the number of parents in the debug case

Eliminate unnecessary npreds member from tsort info
- this is the same as tsi_count

Eliminate unused tree index and degree from ordering calculations

Avoid a few unnecessary rpmteTSI() calls in addRelation()

Remove unused ordering related members from rpmts struct

Bury ordering structures and their alloc+free inside order.c
- nothing outside order.c needs this stuff, TSI alloc+free routines
  have no business being exported
- for now we need to store a pointer to tsortInfo in the elements,
  its just an opaque pointer for ping-ponging the data through ts elements
- all allocation + freeing is local to ordering code

Eliminate the rpmte tsort info manipulation public API
- its not useful outside the ordering internals and API users have no
  business messing with it

Convert rpmtsOrder() to manipulate the tsort data from tsortInfo directly

Convert addRelation() to manipulate the tsort data from tsortInfo directly

Convert collectTE() to manipulate the tsort data from tsortInfo directly

Avoid some unnecessary rpmteTSI() calls in collectTE()
- we have q's TSI in local variable already, use it
- also rename tsi -> q_tsi so it's obvious which one we're manipulating

Move tsort data out of rpmte into, duh, tsortInfo

Move transaction element iterator into rpmts.c where it logically belongs
- make rpmtsi_s really opaque, move rpmtsi typedef to rpmtypes.h
  along with all the other commonly used types

Eliminate unused and unnecessary rpmteBreadth() and rpmteSetBreadth()

Eliminate unnecessary rpmtsiOc()
- it used to be called from ordering code but now there's precisely
  one caller which used it as a fancy way to do increment an integer... rip

Eliminate unused + unnecessary transaction element iterator mode

Eliminate unnecessary ts ping-pong inside rpmte
- rpmteNew() and the stuff it calls doesn't need the transaction set
  for anything since a long time
- document ts argument to be unused

Fix ts element package file size on large packages
- RPMTAG_SIGSIZE is limited to 4GB, grab RPMTAG_LONGSIGSIZE through
  headerGetNumber() grabs the correct tag automatically and simplifies
  the code a bit

Eliminate now unnecessary rpmteFd()
- ...which should've never been exported in the first place

Move payload open into rpmte (internal) method
- the transaction element has all the necessary info to do this,
  having it in psm just requires exposing otherwise unnecessary methods

Hardwire lead major version to 3 (RhBug:546663)
- besides the logic being backwards wrt intended --nodirtokens linkage,
  LSB wants this to be 3 and .. it just makes no difference whatsover
  anyway

Use grep -E and grep -F instead of egrep and fgrep.

egrep and fgrep are deprecated in GNU grep and "historical" in POSIX.

Move python ts element key refcounting back to C-side of things
- it's really the C-side that needs this, doing it in C avoids nasty
  surprises when subclassing rpm.ts
- partial revert of commit 92d18d1d7139b37065ea742cfe0e8cee5e9c3500

Permit unexpanded macros to slip through spec tag sanity checks (RhBug:547997)
- For now, let %{} characters through our valid character set checks
  on specs. This isn't right really, but unexpanded, unrelated macros
  in spec can now cause failure to parse out buildrequires which would
  allow those macros to be expanded. This needs a better fix.

Avoid mucking with fsm->rdbuf in fsmRmdirs() and fsmMkdirs()
- we already calculate the string length, just use a variable local array

Remove now unnecessary const-avoidance construct

Const-correctness on fsmRmdirs() and fsmMkdirs()
- both functions need to copy the data anyway as they modify, just use
  a helper variable to cleanly avoid the const strings from iterator

Remove bogus const from fsm [o]path members and use of _constfree()
- in reality the paths point to allocated memory at all times

Simplify fsmFsPath()

Kill of ancient unused kludgery over buggy packages

Kill off stillborn fsm threading stuff

Kill off stillborn "not yet" code from 2001

Eliminate transaction disk space checking functions from the API
- The DSI stuff runs from deep inside rpmtsRun() with no chance for
  API users to do anything with it. Bury into transaction.c and
  make static to let us change the way it works without API constraints...

Fix memleak in python ts.hdrFromFdno()
- the python object holds reference to header here, decref the header
  so it gets freed eventually

Fix a few list-related memleaks in python bindings
- PyList_Append() bumps the object reference count, callers need to
  explicitly decref them... oops :)

Kick out one remaining sqlite reference (in the translations)

Remove sqlite hacks from rpmdb.c

Avoid exposing libmagic dependency in public headers