solver: Free favorq when running solver_solve()

Otherwise, favorq leaks from previous executions.

Fixes: https://github.com/openSUSE/libsolv/issues/312
Fixes: 2645bc64b322460ac77d0d051de63434d4a90a0a
Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

Add solv_cloexec() function

Windows needs a different implementation, and with that solv_cloexec
we need to patch just one place in the code.

Fix leak in last commit

Allow to build libsolv without stdio cookie opens

build: use GNUInstallDirs

Fixes: #128
Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

Add changes, bump version to 0.7.4

Merge pull request #305 from ignatenkobrain/matchsolvable

bindings: Add matchsolvable() for Pool and Selection

Implement complex conda version matches

This turned out to be easier than expected.

Add support for REL_CONDA

Still to do: support for chaining versions with ',' or '|'.

Revert last two commits, we need to do this different

Seems like we really need REL_CONDA to implement conda's MatchSpec.

Fix REL_EQ for conda, it's "==" and not "="

"=" is a fuzzy match.

Support DISTTYPE_CONDA in dep2str()

This basically strips the spaces from the relops and uses ',' for
REL_WITH and '|' for REL_OR.

repo_add_rpmdb: do not copy bad solvables from the old solv file

Document rpmdb2solv options

repodata_unset_uninternalized: allow to delete all attributes

This is done when the keyname is zero.

Add testcase for cleandeps fix

Optimize last commit a bit

There's no need to look at already conflicted packages, they cannot
be in the package list.

Fix cleandeps updates not updating all packages

Issue #309

Merge pull request #303 from wolfv/patch-1

Add zlib include directories

Add DISTTYPE_CONDA and implement version comparison

Drop uio.h include

I don't know why it needs to be there. Lets see if removing it
breaks anything.

bindings: Add matchsolvable() for Pool and Selection

Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

Add ZSTD_INCLUDE_DIRS

Add include_directory command to more compression libs

Add zlib include directories

Split SOLV_DEBUG_WATCHES from SOLV_DEBUG_PROPAGATE

Looking at the watches move from literal to literal is tedious
and distracts from the interesting messages.

Also make the debug output a bit less noisy.

Simplify moldule code

Use repodata_set_poolstr instead of converting the string to an id
and calling repodata_set_id.

Merge pull request #296 from j-mracek/module-updateinfo

Add support for modular updateinfo.xml data

Remove identical pool check for now

We'll use SolvableId like DepId in the future, so this needs to
be done in a different way.

Merge pull request #298 from ignatenkobrain/bindings

bindings: Add  best_solvables/whatmatchessolvable

bindings: Add pool.whatmatchessolvable()

Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

bindings: Add pool.best_solvables()

Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

doc: remove whitespaces at EOL

Signed-off-by: Igor Gnatenko <i.gnatenko.brain@gmail.com>

Add support for modular updateinfo.xml data

Module advisory data can specify which advisory is applicable for which
modular stream. Module information can be absent or there could be
multiple entries.

If there is a module entry in advisory and none of modules in advisory
is enabled on the system, advisory is not relevant for the system and
should be ignored.

Merge pull request #297 from adelcast/dev/adelcast/recommends

solver_solve: only disfavor recommends if there are any

solver_solve: only disfavor recommends if there are any

In a repo that have pkg 'a' and 'b' available, and 'b' is disfavored,
but 'a' doesn't recommend 'b', libsolv segfaults on
solver_addrecommendsrules, since solv->recommendsruleq is null. Only
call solver_addrecommendsrules if there are recommends rules.

Signed-off-by: Alejandro del Castillo <alejandro.delcastillo@ni.com>

Add changes, bump version to 0.7.3

Specify source directoy for cmake call

This becomes mandatory with cmake 3.13.3.

zchunk: support skipping of optional elements

Make solver_addinfarchrules slightly more readable

Merge pull request #295 from pvuorela/master

Fix repo2solv to work with Busybox find tool

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