Fix repo2solv to work with Busybox find tool

Busybox find got quite confused by "." as arg0. Added a new instance of
dot as first real parameter as I could assume that being the original
intention.

Add testcase for last commit

Do not disable infarch rules when they don't conflict with the job

Otherwise installing a package with the best arch might allow to
pull in the inferior arch as well.

Change pool_str2id calls to use "1" instead of "9"

testcase: support more selection types in disable commands

Small cleanup

Rename bestrules_pkg to bestrules_info, choicerules_ref to choicerules_info

Add testcase for the last commit

Make disfavoring recommended packages work if strong recommends is enabled

Unfortunately this needs a new rule type, which is just the original
recommends rule minus the disfavored packages.

bindings: support SOLVER_FLAG_ONLY_NAMESPACE_RECOMMENDED

Refactor weak rule handling

Cleanups

Merge pull request #291 from jrohel/fix/memLeaks_memAccess

Fix: memory leaks, memory access, non used values

Cosmetics

Don't set values that are never read

Fix: Be sure that NONBLOCK is set

Fix: testsolv segfaults

ERROR: AddressSanitizer: SEGV on unknown address 0x0000000002f0 (pc 0x7f31501d3bd2 bp 0x7ffcfe4d4a50 sp 0x7ffcfe4d4a30 T0)
0 0x7f31501d3bd1 in pool_whatprovides /home/company/real_sanitize/libsolv-master/src/pool.h:331
1 0x7f31501d895e in testcase_str2solvid /home/company/real_sanitize/libsolv-master/ext/testcase.c:793
2 0x7f31501e8388 in testcase_read /home/company/real_sanitize/libsolv-master/ext/testcase.c:2807
3 0x402aa5 in main /home/company/real_sanitize/libsolv-master/tools/testsolv.c:148
4 0x7f314fa8da3f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x20a3f)
5 0x401bb8 in _start (/home/company/real_sanitize/libsolv-master/build/install/bin/testsolv+0x401bb8)

ERROR: AddressSanitizer: SEGV on unknown address 0x000000000000 (pc 0x7f5af9e7815f bp 0x7ffc4c843a40 sp 0x7ffc4c8436c0 T0)
0 0x7f5af9e7815e in testcase_read /home/company/real_sanitize/libsolv-master/ext/testcase.c:2799
1 0x402aa5 in main /home/company/real_sanitize/libsolv-master/tools/testsolv.c:148
2 0x7f5af971da3f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x20a3f)
3 0x401bb8 in _start (/home/company/real_sanitize/libsolv-master/build/install/bin/testsolv+0x401bb8)

Fix: testsolv segfault

ERROR: AddressSanitizer: SEGV on unknown address 0x000000000000 (pc 0x7fab0e11bf2b bp 0x7ffdfc044b70 sp 0x7ffdfc044a90 T0)
0 0x7fab0e11bf2a in testcase_str2dep_complex /home/company/real_sanitize/libsolv-master/ext/testcase.c:577
1 0x7fab0e11c80f in testcase_str2dep /home/company/real_sanitize/libsolv-master/ext/testcase.c:656
2 0x7fab0e12e64a in testcase_read /home/company/real_sanitize/libsolv-master/ext/testcase.c:2952
3 0x402aa5 in main /home/company/real_sanitize/libsolv-master/tools/testsolv.c:148
4 0x7fab0d9d2a3f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x20a3f)
5 0x401bb8 in _start (/home/company/real_sanitize/libsolv-master/build/install/bin/testsolv+0x401bb8)

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