Remove a few usages of old version-spec approach from test-suite

improve man page generation

make command exit codes more consistent

dub search: Improve text, rename parameter ...

The 'query' name is inaccurate, as we only ever expect a package name.

Dub.fetch: Remove a redundant getPackage call ...

Since storeFetchedPackage already returns the Package.

Trivial: Deprecate a compatibility overload of Dub.remove

dub.fetch: Do not catch Exception when a local package can't be loaded ...

Given how simple the function being called is,
there is no reason for us to ignore this error.

Also add options when we combine buildsettings (#2363) ...

* Also add options when we combine buildsettings

* include all the buildsettings in add(bs)

Co-authored-by: WebFreak001 <gh@webfreak.org>

Revert "Default `dub init` to SDL" ...

This reverts commit 3d0d867.

Default `dub init` to SDL ...

Átila wants to throw out dub.json, so this is the first step to discouraging dub.json

Dependency: Expose `visit` to `match` on `SumType` ...

To do so, we need to expose `VersionRange`.
The function is a free function as it otherwise triggers
a deprecation for dual contexts due to the template parameters.

Add some const qualifiers to Repository

Trivial: Remove workaround for DMD v2.070

Trivial: Fix indentation of a log statement

Trivial: Sort inner type in Dependency consistently

Dependency: Ensures call to version_ are on version-containing object ...

The initial port of Dependency to SumType kept full backward compatibility
for ease of use and review, but we can start now to weed out bad usages, if any.

Deprecate Dependency.matchesAny in favor of VersionRange

Trivial: Update comments on PackageManager ...

The previous comments were a bit misleading.

Merge pull request #2353 from Geod24/packman-cleanup ...

Some `PackageManager` cleanup

Signed-off-by: Jan Jurzitza <gh@webfreak.org>
Signed-off-by: Nicholas Wilson <thewilsonator@users.noreply.github.com>
Merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>

Rename private packagemanager : Repository to Location ...

Avoid ambiguity with dependency : Repository and matches PlacementLocation.

Deprecate duplicated enum LocalPackageType -> PlacementLocation ...

Two enums, both have the same values, in the same order.

settings/selections: Only warn on unknown keys, add changelog ...

This will give users the time to adapt, without erroring right away.

Support escaped forward slashes in YAML parser ...

Integrate fix for issue dlang-community/D-YAML#302 .

Move environment read outside of DependencyResolver ...

Reading environment variables from random places in the code makes
the flow harder to follow, and unittesting harder.
This moves one environment reads to the `dub` module,
which performs many of them already.

Trivial: Change max_iterations type to ulong

Parse settings.json using configy ...

This will provide superior error messages to the user,
and match what we already do for `dub.selections.json`.

It also allows us to remove many fields from the `Dub` class.
Those fields existed so that we would not have to re-compute the data
from the JSON DubConfig was storing. Now that we use a simple struct,
and do the aggregation eagerly, all of that complexity can go.

File has moved.

Incorporate review by Jan Jurzitza <gh@webfreak.org>.

Don't require PERL regex in unittest.

Make unittest independent of byte value escape sequence.