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

Avoid noise from python bytecompile on non-python pkgs (RhBug:539635)
- If a pattern doesn't match, we'd prefer an empty list instead of
  unexpanded garbage thank you... nullglob shell option is just what the
  doctor ordered but unfortunately its a bash-specific thing. Otoh there
  are number of other bash-specific scripts in rpm so probably not
  much of an issue...

Removed obsolete "db3" entry in dbi_config_default

Remove support for sqlite based rpm database
Also remove abstraction layer between rpmdb.h and the back end
in lib/backend/

Preparing for 4.8.0-beta1

Bump soname for all our libraries
- while the "really public" API of librpmio has only been enhanced,
  librpm and librpmbuild use some supposedly internal bits which *have*
  changed considerably, so just to prevent anybody thinking they can
  combine librpmio from rpm 4.7 and librpm from this...
- librpmbuild hasn't seen much real change but some (unfortunately)
  exposed struct members have changed so...
- librpm is mostly compatible but as rpmal has become internal API,
  we need to bump (annoyingly, as nothing actually uses rpmal ... sigh)

ts.check() is supposed to return empty list when no problems

Disable the entire rpm.td type for now
- need to figure out saner semantics & stuff...

Make python header iterator return keys, not values
- this is how python built-in dictionary works, be consistent with that
- also avoids having to decide what to do with rpm.td type right now...

Always return malloced data from i18n extensions for consistency

Bump up the python version requirement to match reality
- Python >= 2.6 is known to work, 2.5 might be possible with __future__
  imports but until somebody tests it...

Fix the posix chown test for selinux enabled systems (RhBug:543035)
- on selinux enabled systems, 'ls -l' mode string is longer than
  expected here, just trim the ls output to our expected length

Back out overly strict file type check on %ghost files
- we currently only support creating regular files for %ghosts which
  dont exist in the buildroot, but for files that do exist in the buildroot
  there's no reason to place such limits (eg its perfectly ok to have
  symlinks marked as %ghosts)

Use AS_HELP_STRING() for configure option output

Remove pointless message translation from check-buildroot
- these scripts aren't processed for translation and unlikely to ever be...
- fixes the remaining complaint from ticket #62

Remove ancient broken scripts from the repository
- also eliminates one more "bashism" complaint of ticket #62

Fiddling with ulimit settings does not belong to dep extractor scripts
- fixes one of the bashisms in ticket #62

Correctly pass optional tag argument from hdr.dsFromHeader() (ticket #108)

Dont try to pass anything to rpm.fi() from hdr.fiFromHeader()
- hdr.fiFromHeader() is just a deprecated backwards compatibility dummy
  and doesn't need to pass anything as there were no meaningful
  parameters in older versions

Dont fail build on unrecognized non-executable files (ticket #105)
- Generally only executable files are critical for dependency extraction,
  whereas oddball application data files can cause unnecessary build
  failure due to libmagic misdetections etc, so just let non-executables
  pass with a warning and mark them as unknown data

Just use autoreconf in autogen.sh (ticket #109)
- instead of doing stuff by "hand" and getting it wrong, let autoreconf
  worry about getting it right (if nothing else, we can then blame
  autoreconf instead of ourselves ;)

Fix signature password checking result on abnormal conditions (RhBug:496754)
- Execve() failure wasn't returning an error code, causing rpm to
  think the password was ok when we couldn't even try verifying
- Stricter return code checking from the password checking child:
  the password can only be ok if the child exits with WIFEXITED() *and*
  WIFEXITCODE() of 0. Also WIFEXITCODE() should only be called if
  WIFEXITED() returns true.

We can't handle OpenPGP subkeys or secret keys, so dont even try
- parsing subkeys ends up overwriting data in the main key, causing
  bogus signature checking failures
- this is the final missing piece of RhBug:436812, short of adding
  proper support for subkeys (maybe someday...)

Dont blow up on headers without requireflags in rpmdsNew()

Nuke leftover debug junk

Support constructing header data from python
- no proper i18n string support yet, just the basic types
- basic data type validation only, no overflow checking for integer types
- for lists, validate all items before trying to insert anything
- duplicate tag creation not permitted
- appending not permitted, only assignment (and deletion already)
- there are probably bugs...
- watch out for librpm bugs when passed incomplete headers, ugh

Disable rpm.td type for now
- its not that clear that exporting the td type is really useful
  (certainly not in its current form), avoid exporting an interface we
  might not want to support

Fix hdrContains() return value on error

Remove bogus rpmpopt.pot file (ticket #77)
- this is supposed to automatically generated on update-po time from
  toplevel rpmpopt file and merged into rpm.pot but has been broken for
  a long long time and also the $ translation marking has been deprecated
  for security reasons

Remove hdr.has_key() method, support 'key in h' style instead
- Python 3 removed has_key() from dictionaries, as the 'in' way is the
  way of the future support that from the start (has_key() is not in
  any released rpm version so its safe to remove)

Remove unnecessary hdr.instance() method
- this is available through tag extension now, no point having a
  separate method for it

Typos in exception messages

Error out early trying to install on readonly fs (ticket #99, RhBug:464750)
- simply make readonly filesystems appear as full, preventing install
  early instead of failing with cpio errors in middle of transaction

Convert French man page to Unix line-endings

Fix build with binutils-gold (ticket #107)
- patch from Michal Čihař

Add RPMTAG_EPOCHNUM tag extension
- return RPMTAG_EPOCH but convert non-existent epoch to 0, as rpm internals
  treat it

Add spec package type with access to headers to python rpmb module
- permits python to access parsed headers from spec files
- ticket #14, RhBug:462726

Permit header object generation from PyCObjects
- needed for generating header objects from other extension modules &
  might be handy in some other cases too
- patently type-unsafe but as long as python doesn't provide a better
  way, not really our headache

Lose the silly specFromSpec()
- its our own private data structure we're accessing here, no need
  for extra wrappers and nothing outside spec-py.c needs this

Turn current python spec methods into read-only attributes via getseters
- incompatible, but nobody is really using the spec bindings for
  anything as there's been very little you've been able do with them
- having the stuff accessible as getseters means we dont need 50 different
  fooSet() methods if/when we permit modifying these from python

Sanitize python spec.sources()
- spec without sources is perfectly valid, just return an empty list
  instead of invalid NULL return without exception set and simplify

Put a bit of sanity into python spec methods
- prep, build etc missing from spec are not errors
- instead of returning NULLs (with no exception set!), just return None
  for anything that doesn't exist
- dont bother with NULL checks, if s->spec is NULL then something else
  has screwed up big time

Oops, tp_dealloc doesn't return anything

Fix rpmProblem reference counting in python bindings

Declare a few python functions only taking one argument as such

Rip python checkSignals() wtf'ery
- since the function takes no arguments, declare it as such...

Python archscore() only ever takes one argument, simplify

Make python ts.problems() return a python list, not rpm.ps object
- rpm.ps object only supports iteration and subscript (with wonderfully
  wacko semantics), returning a regular list serves us better
- rip the now useless rpm.ps object type

Remove unused psFromPs()

Eliminate broken rpmps append() method from python bindings (RhBug:538218)
- The code to insert new problems has been using invalid conversion code
  causing crashes since 2004 and nobody noticed, safe to say this is an
  unused interface. Additionally the method argument flags were wrong, it
  was declared as METH_VARARGS but actually expected METH_O semantics. RIP.

Add in-memory hash for caching rpmdb dependency lookups
- worst case behavior for uncached dependency lookups can be disastrous,
  eg > 35s vs < 1s on my laptop for trying to remove /bin/sh provider
- we only bother caching rpmdb lookups, the other cases plenty fast already
- using in-memory cache avoids nasty in vs out of chroot issues with
  temporary db files, which otherwise were about as fast