...but make sure the pre-requisite isn't also an erasure-dependency
- fixes eg Requires(pre,postun) case which must not be filtered out here

Pre-requisites of installed packages dont need to be verified (RhBug:223642)

Update a few tag descriptions + visibility
- RPMTAG_CAPABILITY and RPMTAG_RHNPLATFORM are truly obsolete and
  should've been taken out a long time ago

Use correct tag for NOPATCH
- dumb copy-paste error from commit a317eaadc3f7b9b4ec373719e7d60fb8b98e2c21

RPMTAG_NOSOURCE and NOPATCH aren't internal, they end up in nosrc headers

Gnu tar sometimes prints a "tar: Record size = 16" message, breaking
rpmbuild -ta.

Adapt the Makefile as rpmdb_svc can no longer be built.

New gcc versions comlpain about undefined behaviour...

readLineFromOFI may modify the fileStack, thus we have to re-set
ofi after calling it

Do not load keyring if signature checking is disabled.

Seems like a change was made to make %attr(-) go back to the
defattr setting. Unfortunatelly this broke %defattr(-).

Clean up dbiOpen() a bit
- eliminate now redundant rc-condition, move local variables to the
  scope their used in

Move environment open "refcounting" to db_init() / db_fini()

And now lose db_use_env from rpmdb struct
- Environment is always used, shared if possible private otherwise

Simplify the dbenv->open() flags shuffle
- We always create an environment. If we dont have permissions to create
  or join a shared environment, we use a private environment. Instead
  of trying to figure out what to do beforehand, retry dbenv->open()
  with different flags to see if it succeeds. This eliminates some
  potential races when others might create/remove the environment
  while we're pondering about appropriate flags.
- Lose the "create" bdb config option, this is something we always
  want to decide internally.
- Remove "force" bdb config option, DB_FORCE is dbenv->remove() option
  and its value clashes with DB_CREATE...

Move dbi_use_dbenv from dbi to rpmdb, this is a per-rpmdb setting

Move dbhome directory create/verify to openDatabase()
- all db opens must go through openDatabase(), no need to track this
  with a separate variable on each index open

Eliminate DB_JOINENV usage and config option
- DB_JOINENV is a no-op since BDB >= 4.4

Eliminate "thread" BDB option
- this is not something that users should be tweaking
- its never been enabled, leave it that way for now

Lose redundant dbi_use_env check
- its always enabled when we get here, but might get turned off
  in the initialization process (at least for now)

A little bit of sanity to dbiNew()
- lose the hysterical dbi_use_env condition and usedbenv option
  this is always enabled
- ensure mpool is always initialized, lose the config option (this
  is a mandatory BDB subsystem, not a bleeping configurable)
- let pagesize be what it is even when no configuration is present
- avoid overriding dbi_mmapsize and dbi_cachesize if set in the configuratio
- these are all per-environment, not per-dbi settings but for now...

Eliminate exclusive and rdonly BDB configuration options
- we never want DB_CREATE to cause failure - let BDB create the
  db if it needs to, otherwise DB_CREATE doesn't do anything
- rdonly is decided elsewhere, not in bleeping configuration

Mop up ancient BDB leftovers

Eliminate BDB transaction tuning knobs from configuration
- we dont support transactions yet, and this stuff doesn't
  belong to "user configuration" anyhow, these are just useless

db_env_create() doesn't take any flags currently

Remove per-dbi perms and configuration
- the file permissions are per-db global, no need to be able to
  speficy different modes for indexes

Remove per-dbi mode and configuration
- read/write/create mode is global to the entire db, either its
  just readonly or its not
- this doesn't belong to configuration

Require BDB >= 4.5 for sanity's sake
- replace unused feature tests in configure.ac with a simple version check

Remove unused dbi_txnid member from dbiIndex
- transactions aren't currently used, we'll need txnid handles eventually
  but the dbi is unlikely to be the right level to store this, add
  where actually needed when the time comes

db_create() doesn't take any flags currently

Eliminate dbiIndexNewItem()
- this is just simple struct with no special allocation, and contents
  are memcpy()'ed so theres no need to malloc it

Change nextInstance() to permit retrieval too, use it for bitmap alloc
- The "was this verified already" bitmap uses header instance numbers
  which are monotonically increasing and can be much much larger than
  the number of actually installed headers. Grab the current instance
  number for a better idea how much we'll at least need.
- Throw out the db stats based dbiNumKeys(), this doesn't have the
  kind of performance penalty that stats have and suits our purposes
  better anyway

Make rpmdbAdd() and rpmdbRemove() internal-only
- all additions and removals to the rpmdb need to go through the
  transaction machinery, these are very low-level functions that
  API users have no business messing with

Stash rpmdb struct definition out of sight into dbi.h
- not perhaps the ideal place for it but the backend level needs
  access to the main db in several places anyway, and this leaves
  rpmdb_internal.h clear for internal-only APIs

Unify rpmdbAdd/Remove() interfaces
- rpmdbRemove() takes now a header as argument too - we need both
  the header number and the header itself there anyway, so might as
  well use the header we already have instead of flipping through backwards
  hoops to get to it

Eliminate the headerCheck() foo from rpmdbAdd/Remove()
- for rpmdbRemove() these have been completely unused, and for rpmdbAdd()
  the higher levels have had more than one chance of verifying the
  header if checking is enabled in the transaction set

Lift RPMDBI_PACKAGES handling from db add/remove to separate helper
- Remove vs add is rather similar, unify the copy-pasteish code
- Be a bit more careful about error returns from the primary db functions

Lift header instance number figuring out of rpmdbAdd() to helper function

Verify header sanity as the first thing in rpmdbAdd()
- avoid allocating header instance for something that we'd fail to add
- avoid possibly adding data to indexes even if adding the header
  itself fails, duh

Move header INSTALLTID manipulation out of rpmdbAdd()

Remove unused require- and provideversion indexes
- For completely unused indexes, these are fairly expensive too

Eliminate redundant/bogus temporary variable shuffling
- This fetches the special key zero which is used for bookkeeping of
  header "instance" numbers, the actual header to be stored is NOT
  relevant here
- keyp, keylen, datap and datalen are just redundant fluff, manipulate
  the key and data DBT directly

Remove unused _DBI defines

Lose unused fluff from openDatabase()
- As RPMDBI_PACKAGES is always the first index, the for-loop never
  did anything else than open the Packages db and exit with success/fail.
  For the same reason RPMDB_FLAG_MINIMAL didn't do anything at all here,
  lose the unnecessary flag..

Eliminate static _rebuildinprogress hack, use per-db flag instead
- replace unused RPMDB_FLAG_CHROOT with RPMDB_FLAG_REBUILD and pass
  around as necessary

Bury rpmdb flags inside rpmdb.c, not used by anything outside it

Unbreak --rebuilddb
- managed to bust up the rebuilddb-path generation once again in commit
  890cea56c91ff6ff887a439c036d345715e6c5c2, doh
- make the intended logic more obviours

l10n: Updates to Ukrainian (uk) translation

Transmitted-via: Transifex (www.transifex.net)

l10n: Updates to Russian (ru) translation

Transmitted-via: Transifex (www.transifex.net)

Nuke bogus comments + related NULL-checks
- db->_dbi cannot be NULL if db is not NULL, it's allocated at
  newRpmDB() and freed in rpmdbClose() and nothing else touches it

Clean up rpmdb struct initialization
- lose the dumb template assignment and related defines

Lose unused static initialization foo

Clean up formatting in rpmdbCountPackages() (just cosmetics)

Remove unused rpmdbCloseDBI()
- this was used by the Depends temporary db hack, forcing it
  to be exposed in the API... not anymore

Generate package database statistics on close & make use of it on open
- Turn dbiStat() into more useful: return the number of keys in the
  index, hiding away the BDB internal access method stuff into the backend
- Force statistics gathering at Packages db close, take advantage of
  that when its opened to get a fairly accurate count of packages for
  initial "header verified" bitmap allocation. Previously DB_FAST_STAT
  was used on open but it never returns anything when no stats have
  been previously collected, hence the need for the expensive slow stat.
- The performance hit from stat generation is hardly worth it for
  the bitmap allocation alone, but lets see if there are other uses...
- Also gets rid of dbi_stats member, this is not particularly useful

Lose "support" for truly ancient BDB versions
- BDB >= 4.3 is required now, and even thats several years old

Rename dbiOpen() + dbiOpenDB() to make their layer obvious from the name
- dbiOpen() in rpmdb.c operates on "rpmdb layer" so calling it
  rpmdbOpenIndex(), dbiOpenDB() on the other hand is the lowest level
  backend thing, which is now called dbiOpen() to be in line with
  the other operations like dbiClose()

Pass rpmdb to dbiFindByLabel() / rpmdbiFindMatch() directly
- avoids having to go backwards from the dbi to rpmdb when we very well
  have the rpmdb handle at hand here...

Eliminate dbi_rpmtag member from dbiIndex struct
- the backend doesn't care about the tag beyond initialization,
  and the tag is only used for error messages from rpmdb layer which
  already knows what the tag is

Define + use enum for primary/secondary index types + api to get it
- no functional changes, making the type more obvious than
  "case 2*sizeof(int32_t):"

Rename the dbi access method member dbi_type -> dbi_dbtype
- make it more obvious what it is

Split dbiIndex out of rpmdb_internal.h into header of its own
- the dbi presents an internal api of its own, and deserves a separate
  header (baby steps to making dbiIndex opaque outside the backend)
- move dbiVerify() to the backend where it belongs
- mark all the dbiFoo() functions as internal

Eliminate unnecessary argument to dbiCopen()
- txnid (which is unused...) is stored in dbi so dbiCopen() can already
  get to it

Move dbi_no_dbsync logic to inside dbiSync()
- avoids having to check for it in every single caller

On header iteration format queries, dump the contents as-is
- This makes eg --xml query of packages give out the true contents
  of the package, raw i18n strings and all, which is what you probably
  want when examining the contents on this level. Regular queries still
  run with all bells 'n whistless enabled.

Disable header i18n lookup in headerCopyTags() (RhBug:578299)
- without HEADERGET_RAW here, only one of the translations in header
  (whatever happens to be "current" locale) gets copied

Remove BDB recno + queue access methods from configuration
- these rather limited access methods have never been used for anything,
  rpm always used hash access and later btree to varying degree

Database metadata integer byte order is not user serviceable config

Lose dbi file configuration option
- index names are rpm tag names, period
- also we dont need to strdup() the tag name, rpmTagGetName() returns
  pointers to const strings these days

Make dbiVerify() and dbiOpen() static, not needed outside rpmdb.c

Bury dbiIndexItem and dbiIndexSet structs inside rpmdb.c

Make dbiIndexSet operations static inside rpmdb.c

Lose the BDB access method configuration options
- In the ~10 years of their existance none of them have been used,
  and besides users have no business messing with fundamental database
  behavior such as whether duplicates are allowed or not

We really dont care about macro expanded per-db error prefix

Lose shmkey + shared BDB config options
- we dont want anything to do with System V IPC, really... BDB uses
  file backed mmap when it needs, this suits us just fine

Remove "truncate" BDB config option
- Truncate is so dangerous that dbiOpenDB() discarded it even if
  set in configuration. How useful is that?

We dont want to use DB_HOME environment, ever
- rpm has its own switches for overriding the db home when that's
  necessary, DB_HOME it should use not

We dont have any temporary databases anymore, lose the config switches
- The last temporary db was Depends which is now replaced with an
  in-memory hash, and the temporary dbs were troublesome with
  chroot operations anyway, good riddance

Lose pointless per-db and per-dbi errfile and errcall pointers
- BDB uses stderr for errfile when not set, which is what we set it
  to through various hoops. Just leave it alone.
- db_errcall has always been (intentionally) NULL to disable that output,
  the errors get logged though cvtdberr() with slight filtering

Shred a pile of BDB config goo that's never been actually implemented
- with the exception subfile (used by now extinct sqlite backend)
  none of this was ever even connected to the bdb configuration...

Shed the pointless per-dbi memory allocators
- we want to use our own allocators for exit-on-enomem behavior but
  having per-index allocator configuration makes no sense whatsoever

Eliminate now unnecessary db_ndbi field, number of indexes is constant

Make rpmdb index list hard-wired
- We dont grow new indexes every other day, and especially this
  is not activity that users need to be able to do
- Gets rid of the hysterical initialization and million can't happen
  NULL-checks

Minor cleanup to rpmdbAdd()
- move dbiOpen() + check to more sensible place, curing the
  busted indentation
- initialize at declaration save a few unnecessary lines

Add + use a helper function for retrieving single header at offset

Minor cleanups to rpmdbAdd() and rpmdbRemove()

Minor cleanup to rpmdbRebuild(), use rstreq() instead of hand-comparing

Hide rpmte open/close/error etc logic inside rpmteProcess()
- make a bunch of helper functions static now that they're not needed
  elsewhere, rpmte is slowly becoming self-aware ;)

Move the little thats left of psm.h into rpmte_internal.h
- no functional changes, just shuffling declares around

l10n: Updates to Polish (pl) translation

Transmitted-via: Transifex (www.transifex.net)

Oops, scriptlet related error messages had dropped out of potfiles

Update translatios again to shed non-existent messages from recent churn

Lazy allocation of rpmds color array
- only requires and provides in transaction elements get colored,
  dont allocate unused memory

Lose long since unused fdReadable() and fdWritable()
- these have been unused since rpm 4.6.0, and can be implemented
  without access to rpmio internals too if somebody cares...

Lose long since unused url control structure and related functions
- these have been unused since rpm 4.6.0, and rpm is not in the
  url business, rip.

Remove unused dependency set file reference count
- this has never been used for anything at all, there are better things
  to use our memory for than unused arrays

Remove unused and non-sensible depedency set build-time
- Buildtime was never part of rpm version comparison nor should it be,
  on rpm level the only sensible differentiator between two identical
  dependencies is the dependency color if any. Other than that, random()
  just as "appropriate" as build time.

headerNextTag() fits the checkForDuplicates() use-case better
- as we're only looking at tags, not their contents... avoid unnecessary
  data shuffling and the code is a little simpler too

Unbreak cpio archive path generation
- fixes regression from commit 45e2d4e1d0260a9c07eb1daf0c68b82f602ad16f
- striplen was never set on build at all, and the psm part was unused
  too - either unused since forever or possibly related to repackaging

Oops, being a bit too strict on dependency qualifiers
- Forgot how nasty hack parseBits() was... its abusing multilang
  support and the multilang support passes "C" if no lang is specified,
  which certainly isn't a valid qualifier. Make multilang vs tag with
  optional qualifiers different types in PreambleRec, which allows
  handling this more sanely
- Additionally build-dependencies dont take any qualifiers now