Fix: Memory leaks

Fix: Add va_end() before return

The va_end() performs cleanup.
If va_end() is not called before a function that calls va_start() returns,
the behavior is undefined.

Fix: Dereference of null pointer

Do favor evaluation before pruning

This allows to support favoring single packages. As a side effect,
we'll have the best version displayed as alternative.

This changes the code to use a favor id for every solvable, which
means that it uses quite a bit of memory but makes the code much
easier and faster.

Add changes, bump version to 0.7.2

Get rid of the solve_helper function

We now use a special typemap to convert an array of Job objects
into a queue. (We already did that for tcl.)
Also generalize the DepId array code, we can now easily support
other dual types.

tclsolv: Adapt to Selection-flags change

Should have been in commit c6b300573f9974706db9dfd061b6e6c5b0e81bf8

Docu tweaks

Tweak README

Do not autouninstall packages because of forcebest updates

Note to self: do not commit unfinished test cases...

Do not auto-erase disabled packages

Switch order of tilde and caret check in evr.c

Tilde must come before caret, so that 1.0~1 < 1.0^1
Also add this to the testcase.

Also look at recommends in only_namespace_recommended mode

We cheat by just checking if the dependency contains a namespace
relation. This is not 100% correct for complex dependencies where
we have multiple blocks after normalsation. But doing it right
is a lot of effort and there should be no difference in practice.

Enable support for new rpm's caret comparison

And add a testcase.

Add a small json parser

It's always good to have a json parser...

Add experimental SOLVER_FLAG_ONLY_NAMESPACE_RECOMMENDED flag

This limits the recommends/supplements to namespace dependencies.

Fix last commit. Oops.

Get rid of the xmlparser's error callback

It's much simpler to just return an error from the parse
call.

Add a solvable_free() inline helper function

Allow arch == NULL packages if arch testing is disabled

Fix off-by-one error in the "oneof" parsing code

Fixes issue#289

Support caret in rpm's EVR comparison

Currently only active if RPM_CARET_DEPS is defined, so that Igor
can do some testing.

bindings: implement set/get_considered_list

This is just a first version to play with. It's based
on pull request #288 from milan. Thanks!

Add changes, bump version to 0.7.1

repo_autopatten: also copy the category from the solvable defining the pattern

This makes it possible to have the (translated) categories in susedata.xml
instead of getting them from the package provides.

Fix off-by-one error in relation writing code

This tended to mess up the first relation in the solv file.
Counting is hard sometimes...

fix

Format a bit

Add changes, bump version to 0.7.0

Oops, bring back repo_write_filtered and repodata_write_filtered

They are deprectaed, but old code may still use them for now.

Adapt documentation to the last change.

Bindings: make Selection-flags an attribute

Incompatible API change! I hope this doesn't bite too many
bindings users. But better late than never, having a flags()
function does not make much sense.

Always set the filelisttype when creating a filelist stub

We used to do this only if there was a filtered filelist, but that
leads to problems if the filtered filelist is not in the same repodata
as the extension list.

Merge pull request #260 from berolinux/armv8

Add armv8 (32-bit mode of aarch64 CPUs)

Fix reserved identifier violation

See issue#104. We now prefix with "s_" instead of just "_".

Small cleanups

Incompatible ABI change: change transaction_calc_installsizechange from int to long long

Incompatible ABI change: change pool_calc_installsizechange and solvable_lookup_sizek from int to long long

Incompatible ABI change: change diskusage from int to long long

The diskusage is in kilobytes anyway, but this should make things
more future proof.

Merge pull request #282 from jdieter/fix-parens

Add missing parenthesis

Add missing parenthesis

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Refactor selection code

Test selection_matchsolvable

Split away solvable_matchessolvable_int function

To be used in the selection code as well.

Tweak system zchunk code

- rename USE_SYSTEM_ZCHUNK to WITH_SYSTEM_ZCHUNK
- fix file descriptor leak
- fix memory leaks

Merge pull request #270 from jdieter/zchunk

Add zchunk support

Add selection_make_matchsolvable and selection_make_matchsolvablelist

This is like pool_whatmatchessolvable, but works on a selection.
The advantage is that it supports filtering.

Improve alldeps code in example solver

Add pool_whatmatchessolvable() function to match against the provides of a solvable

Clean up filelist mode code

Move key mapping after key pruning

That way we'll not pollute the spool so much. The
downside is that we need to have some extra room on
the needid array for the keys.

Improve main schema generation

Now less complex and may result in a smaller solv file,

Improve handling of dirs in repo_write

Again, what was I thinking?

Drop support for REPOKEY_TYPE_U32

Six years ago we used it for the rpmdbid of installed packages.
See commit d2b47004c22a7df3b92eaa8cbbb3a42ea32d047e

Improve handling of relations in repo_write

The old way was way to complex. What was I thinking?

Add system zchunk support

Signed-off-by: Jonathan Dieter <jdieter@gmail.com>

Make sure that targeted updates don't do reinstalls

We already had code in add_update_target() to do this, but it did
not cover the multiversion case where more than one package with
the same name is installed.

Improve handing over of the parant kv in subarray searches

We now pass it disguised as keyskip.

Change data_fetch so that it always returns the first entry

It did something different for fix/flexarrays.

Allow writing of repodatas in state LOADING

Broken with commit 241123391fdbd0fa4b38b2a30b78e85de9c9fd5d

Generalize uninternalized lookups

We now have the generic repodata_lookup_kv_uninternalized and
repodata_search_uninternalized to access uninternalized data.

Drop the old interface, the functions were only used in some
metadata parsers. So it's an API change but nobody should notice.

Fix share handling in repo2solv

We only search the uninternalized data for the repodata we're
extending, so we need to internalize the data before adding
new repodata areas.

Fix shared handling for 'susetags2solv -d'

We need to record the share data, otherwise we miss the share
information when extending the packages.

Add some casts so that the C++ compiler does not complain

Change order of known ids a bit

INCOMPATIBLE ABI CHANGE

Drop internal data_skip_verify function

Nobody used it.

Support reading of TYPE_DELETED keys

Bring repodata_lookup_num in sync with repo_lookup_num

It used to return a boolean if the num was found and write
the num over a pointer. This was done because repo_lookup_num
needed to iterate over all repodatas and thus needed to know
if the keyname existed or not.

Nowadays we use repo_lookup_repodata_opt to find the correct
repodata, so we do no longer need the boolean return value.
So we can make repodata_lookup_num work like repo_lookup_num.

This is an INCOMPATIBLE API CHANGE but I'm pretty sure nobody
uses repodate_lookup_num, so this should not hurt anybody.

(You can use a call to repodata_lookup_type to check if the
repodata really contains the num in case the "notfound" value
is returned.)

Implement the REPOWRITER_KEEP_TYPE_DELETED flag

Drop undocumented old cruft from the tools code

This is mainly the -b and -n options, which were used to
test solv extension files in the long ago past.

Add new repowriter interface to write a solv file

This replaces the old write_filtered functions that could
not be extended easily. The old functions are deprecated, do
not use them in new code!

drop stuff from TODO list

repo_rpmdb: use new repodata_search_arrayelement method

Add repodata_search_arrayelement method

This searches an element of an array referenced by the kv.

Add repodata_translate_dir method

This translates a dir id from one repodata to another. Change
repo_rpmdb to use it when copying solvables.

repo_add_solv: check that arrays have incore storage

Also fix a tiny bug in the vdata handling.

data_fetch: set kv->entry to -1 in array case

Not that the code is used anywhere...

rpmdb2solv: support including the package changelogs

Second part of fixing the longstanding bug

Change dataiterator, repo_search, and repo_write so that it
uses only the value in the last repodata if multiple repodata
sections have an entry for the keyname.

We do this by first setting up a keyskip array that maps
keynames to a base + repodataid value. We skip a key
if the repodataid does not match the entry in the keyskip
array.

The base is used so that we do not have to clear the array
for the next search. Instead, the base is simply set to a
value greater than all the current entries.

WARNING: As this adds two new elements to the Dataiterator,
this is an INCOMPATIBLE ABI CHANGE. We'll use this to do
a couple of cleanup commits in the next days that have been
postponed until now. We'll bump the soname before doing the
next release.

Add missing repodata methods to bindings

lookup_id()
lookup_num()
lookup_void()
set_num()
set_void()
set_location()
unset()

Cosmetics: fix spaces

Rewrite addfileprovides

It should now mirror what repo_lookup_filelist_repodata does.
It should also be quite a bit faster than the old code.

With the new code SEARCH_COMPLETE_FILELIST is the default.

We may add a SEARCH_FILTERED_FILELIST flag in the future if
there is a use case.

Add an internal repodata_lookup_packed_dirstrarray function

This is faster than using repo_search to iterate over the
filelist. We will use it in the new addfileprovides code.

Support custom filelist filters.

REPOSITORY_FILTEREDFILELIST can now be an idarray where each
element is a glob pattern.

Change lookup/search code that it returns the match from the last repodata

This is the first part of fixing a longstanding bug. This fixes
the lookup cases and the case where a keyname is provided in the
search/iterator.

We do this by adding a repo_lookup_repodata function that returns the
last repodata that has the specified keyname.

There is also a repo_lookup_repodata_opt function that may return
a repodata that does not contain the keyname instead of NULL. This
is a bit faster if you need to do a lookup anyway.

We also now have a repo_lookup_filelist_repodata that contains all
the logic to deal with filtered filelists.

repo_search: stringify filenames if requested

This brings repo_search in sync with what the data iterator
code does.

Add REPOSITORY_FILTEREDFILELIST id and filelist type

REPOSITORY_FILTEREDFILELIST is used to mark that the filelist
in the repodata is filtered with some filter.

The commit also adds a filelisttype element to the repodata
and sets it to filtered if REPOSITORY_FILTEREDFILELIST is set.
It also sets the filelisttype to extension if the repodata
has a filelist and was generated by extending solvables.

We'll use the filelisttype later to decide if we should use
the filtered filelist when searching for files.

Cosmetics: rename repodata_load_stub to repodata_stub_loader

Also keep the callback when loading a stub with repo_add_solv,
as some tools use it as indicator if the repodata was a stub.

Make repo_matchvalue static and remove it from list of exported symbols

It was never declared in a header file. Furthermore it needs
an internal struct definition to work, so nobody from outside
can use it anyway.

Move repodata_filelistfilter_matches into filelistfilter.c

We'll later extend this to support custom filelist filters.

Split away repodata_load

This can be used to force loading of a stub repodata.

Cosmetics: save some space

Work around bogus uninitialized warning from gcc

Also add autopatterns for the system repo

Should have been in commit 276e7586cce0535c194336cd2a3bac9d4f5805e1

libzstd is available on leap15 but not on sle15

Fix out of bounds compilation warning

When compiling with -D_FORTIFY_SOURCE=2, gcc raises the following
warning:

    In file included from /usr/include/string.h:494,
                     from /build/libsolv/src/libsolv-0.6.35/src/policy.c:16:
    In function ‘memcpy’,
        inlined from ‘urpm_reorder.isra.18’ at /build/libsolv/src/libsolv-0.6.35/src/policy.c:1239:9:
    /usr/include/bits/string_fortified.h:34:10: warning: ‘__builtin_memcpy’ forming offset 8 is out of the bounds [0, 7] [-Warray-bounds]
       return __builtin___memcpy_chk (__dest, __src, __len, __bos0 (__dest));
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Culprit code is:

     memcpy(kn, "kernel", 8);
     memcpy(kn + 6, flavor, release - flavor + 1);

The first memcpy copies 8 bytes whereas the string "kernel" as only 7
bytes. It does not have serious consequence since the second memcpy
overwrites the unwanted byte. Anyway, it is better to fix it.

Add changes, bump version to 0.6.35

Fix indent