Const pedantry

Fix some const issues in macros

tgi doesnt need fts private include

Fix multi-line macro expansion in last line of spec (mdvbz#27417)

Patch from Pascal Rigaux:

1. creating functions restoreFirstChar(), copyNextLineFromOFI() and
   copyNextLineFinish() out of copyNextLine()
2. creating function readLineFromOFI() out of readLine()
3. inline copyNextLine
4. if we can restoreFirstChar, then use it, otherwise read from file
   (ie don't read from file when we have a remaining multiline macro in
   spec->nextline)

Hum, NSS needs to be shut down too
- some ~80 bytes are still leaked apparently from somewhere within nss/nspr

Add dumb maintainer snapshot generator make rule

Ordering fix (don't use RPMSENSE_PREREQ) from Pascal Rigaux

Drop duplicate locale.h check

Avoid exiting too early due to nested rpmdbCheckSignals()

Update translations

Todo-update...

Drop bogus const from rpmReadPackageManifest argvptr
- it's malloced and needs freeing by the caller..

Cast to silence gcc..

One leftover reloc constfree..

Remove bunch of bogus cli-related consts...

Use constfree on relocations for now
- they're totally exposed all they way to rpmqv.c.. eww

Remove bogus const from rpmInstallSource*() parameters
- spec and cookie are malloced and need to be freed by caller
- unconst various other rpmQVK arguments & friends, no api exists to
  free them so caller needs to handle anyway

Use constfree for rpmts suggestions
- goes down to rpmal returning fnpykey which is declared as const..
  check this later

Stick constfree to extension cache cleanup
- deal with the header mess later on

Drop bogus const from headerFreeTag() data pointer
- no effect as it's hft is unused
- mark headerFreeData(), the real problem case, with _constfree() for
  easy finding (inlined in header.h so can't use _constfree())

Drop bogus const from temp variable

Remove bogus const from rpmGlob argv return type
- it's malloced and must be freed by caller

Make fsm failedFile non-const like it really is
- callers need to free it so const is wrong
- cast away the hardlink-hackery in fsm instead

Drop bogus const from temp variable

Drop bogus const from zapRelation return
- the retval is malloced from rpmdsNewDNEVR() and must be freed by the caller

Drop bogus const from rpmts rootDir + curDir

Remove bogus const from rpmdb match iterator contents

Use constfree for rpmdb_s contents for now
- rpmdb_internal is included from several places over the tree

Use constfree for rpmhash key + data for now
- sometimes they point to real const data, sometimes not, eww

Use constfree for fsm path, opath for now
- neither should really be const but they sometimes point to real
  const variables, sometimes not.. needs careful review

Drop const from fs mountpoint, contained within fs

Drop const from temporary variable

Use constfree for rpmte internals for now
- the internals are exposed all over the tree...

Use constfree for rpmfi internals for now
- the internals are exposed all over the tree...

Remove const from rpmal dirInfo dirname
- add comments wrt needle dirName - those are actual const pointers to
  within rpmfi, must not free

Use constfree for psm failedFile for now
- failedFile is malloced by fsm, but removing the const from fsm
  gets a bit hairy as failed it's not always malloced
  (see changeset 4062:02b0c237b675) so we'd have const-disabling casts instead
  of freeing consts...

Include toplevel Makefile from Makefile.maint
- allow "make dist" in one step with .maint..

Drop db3 from dist subdirs
- db3/configure is covered by EXTRA_DIST already

Unconst psm pkgURL and rpmio_flags members
- fully contained within psm

Make psm opaque, add minimal methods to cover internal needs

Drop bogus const from dnlFreeIterator() param

Drop bogus const from fsmFsPath() return type
- path is malloced and caller needs to free
- additionally use size_t, not int for string size

transaction.c doesn't need fsm.h, rpmfi_internal does

Stick cpioMapFlags to fsm.h, they're only used by fsm.c
- allows FSM* typedefs to move to where they belong

Hide fsmIterator_s struct details, not needed outside fsm.c

Stuff hardLink_s inside fsm.c, add pointer typedef to fsm.h
- nothing outside fsm.c needs except for the declaration in fsm_s

badDeps is encapsulated within depends, doesn't need const
- remove ancient reference code

Use constfree for cleaning up pgp digest contents
- the contents are exposed within rpm (although not public interfaces)
  but pgpCleanDig() should be used to free the struct
- pgpDig and pgpDigParams need encapsulation + api...

Don't bother configuring db3 directory if using external db

Make ChangeLog creation depend on .hg directory
- .hg dir timestamp changes on commits, otherwise updating ChangeLog
  requires removing it

Unbreak internal dep generator
- more signed -> unsigned count type breakage

Unbreak array query formats
- signed -> unsigned count type change broke it..

pgpReadPkts returns malloced memory, unconstify

Lone module-private const .. remove

Unconstify macro contents, fully contained within macro module

Bogus const in temporary variables

More const removals
- both username cache and log records are fully contained within their
  modules...

More constfree for spec exposed stuff for now
- openfileinfo seems easily made opaque, revisit later

Much of spec internals is exposed, let them be const for now

Remove couple of bogus const uses in temp variables

Unconstify FileListRec and AttrRec contents
- not exposed outside files.c

Unconstify rpmrc internal tables
- not exposed outside rpmrc, "protection" buys nothing

Fix default queryformat setting when _query_all_fmt not set
- \n in the rpmExpand() causes the result to always be non-null...
- also avoid unnecessary const free

Remove const from rpmMkTempFile file name parameter
- the temp name is malloced and needs to be freed by caller

Refactor rpmReSign() and associated file "management" a bit
- remove unused + useless "rc" parameter from manageFile()
- remove tempfile handling from manageFile(), create + open a tempfile
  explicitly in rpmReSign()
- both copyFile() and manageFile() now require file names to be passed always
- split file closing into separate closeFile() function

Stupid reversed-logic thinko..

Remove const from bunch of error message return types
- headerCheck(), rpmReadHeader() and rpmReadSignature() return malloced
  error messages the caller needs to free, shouldn't be const
- adjust rpmdb hdrchk callbacks accordingly

IDT* structures are exposed, let them use constfree for now..

And yet more bogus const removals...

Couple of bogus const removals from rpmgi and rpmds structs
- in both cases the data is already directly inaccessible from outside,
  no further protection necessary

Make bunch of build internal code take const parameters

Further bogus const removals...

Fix incorrect rpmCleanPath() retval comment

Another bunch of bogus const removals

Avoid mixing local malloc with const parameter pointer

Use _constfree() for freeing urlinfo contents

Expose compiler warnings from freeing data declared as const
- _free() is just a wrapper to free() which additionally returns NULL
- add _constfree() for use in the cases where const is used to protect
  malloced "read-only" data in long-lived otherwise exposed structures etc

Remove bogus const from rpmProblemString() return type
- its malloced so it needs to be freed
- fix all users to actually free

Remove bogus const from rpmGenPath() temporary targets

Remove bogus const from rpmGetPath() temporary targets

Remove bogus const from rpmGenPath and rpmGetPath return type
- both return malloced strings you're supposed to free

Bogus const removal continued...

Another bunch of bogus const removals

Remove bogus consts..

Bunch of const char* corrections
- functions returning string constants as char * etc...

Bunch of silly ssize_t casts
- cuts down the amount of spurious warnings from inlined code...

Oops, major + minor are not dev_t...

Yet more type-pedantry...
- use rpm_tag_t for signature tag types too
- size_t, not rpm_count_t in signature generation
- add couple of explicit casts to rpmpgp.h to avoid spurious noise from
  inlined functions

More size_t type pedantry

Oops, rpmpsi index needs to be signed type

More misc size_t issues

Use dev_t instead of ints..

Couple of more size_t uses + related format string fixes

Big bunch of int -> size_t corrections for various signature items

Use rpm_tagtype_t everywhere for rpm (header) tagtype type
- typedef'ed as uint32_t, doesn't matter much but negative types dont exist
- easy to grep, easy to change...
- define RPM_FORCEFREE_TYPE instead of -1 "magic" for forcing
  headerFreeData (ugh), easier to grep for and change than "-1"

Use rpm_tag_t everywhere for rpm (header) tag type
- typedef'ed as int32_t for now, negative values used in some places for
  error cases
- easy to grep, easy to change...
- add RPMTAG_NOT_FOUND define, used in place of -1 "magic",

Use rpm_count_t everywhere for header data count
- typedef'ed as uint32_t as that's the key size limit imposed by BDB,
  relevant for RPM_BIN_TYPE
- easy to change to whatever later on as it's now consistent everywhere
- explicit casts where needed to avoid new warnings from signedness

Use rpm_count_t everywhere for header data count
- typedef'ed as uint32_t as that's the key size limit imposed by BDB,
  relevant for RPM_BIN_TYPE
- easy to change to whatever later on as it's now consistent everywhere

Install the new fancy perldep extractor

Register the new SCRIPT_ERROR callback value to python