Add MSVC compilation support with musl, enable OS X testing

Be more correct with multiversion packages that obsolete their own name

We used to simply ignore the obsoletes. There are use cases which
need them, though. So do the right thing.

Use buildversion for comparison if two versions are identical

pool_conda_matchspec: return a simple dependency if we can

Test last commit

Fix SOLVER_FLAG_FOCUS_BEST updateing packages without reason

Better support for SOLVABLE_BUILDFLAVOR

This changes pool_solvable2str, testcase_str2solvid and
testcase_solvid2str to append the build flavor.

Fix a couple of signed/unsigned comparisons

Allow to lookup a single id with repodata_lookup_idarray

I don't see any harm in allowing this and it makes dealing with
things like SOLVABLE_LICENSE easier which can be either an id
(the old way) or an id array (the new way).

Use system type rpm in testcase

Otherwise arch 'noarch' is unknown

Fix tests

All tests should have a results section

Merge pull request #315 from ignatenkobrain/todo

trivial: Drop implemented items from TODO

trivial: Drop implemented items from TODO

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

Tweak solvable_identical and pool_solvable2str for conda

Drop unused diskusagexml2solv.c

Whatever that was...

Split testcase repo reading/writing into repo_testcase.c

This is done so that programs that just want to read/write
repos in testcase format do not need to drag in all of the
solver code. (This matters only for static linking, of course.)

Add repo_add_conda() and conda2solv

Get rid of the favorq

We do not need it anymore. It's easier to create the favormap
directly from the job list.

Merge pull request #313 from ignatenkobrain/solver-free-favorq

solver: Free favorq when running solver_solve()

More conda work

- add pool_conda_matchspec() to create a REL_CONDA dep from a string
- add name globbing for REL_CONDA
- support build flavor matching

Escape \ in file argument before calling ADD_TEST

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