Project: Only print warning about mismatching dependency specs for versions ...

When using path-based dependency or repository dependencies,
we shouldn't need to validate those as they always match.

Merge pull request #2458 from Geod24/segment-location ...

PackageManager: Move scanPackageFolder to Location

Signed-off-by: Nicholas Wilson <thewilsonator@users.noreply.github.com>
Signed-off-by: Razvan Nitu <RazvanN7@users.noreply.github.com>
Merged-on-behalf-of: Mathias LANG <pro.mathias.lang@gmail.com>

PackageManager: Move scanPackageFolder to Location ...

A simple move for now to keep it simple,
the breach of abstraction will be resolved in a latter commit.

PackageManager.scanPackageFolder: Do some cosmetic fixes

Fix #2450 - issue1474 testcase is broken on Windows

Merge pull request #2453 from tom-tan/issue2452 ...

Fixes #2452 - Accept non-existing file in `sourceFiles`

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

Add test

Fixes #2452 - Accept non-existing file in `sourceFiles`

Merge pull request #2449 from tom-tan/issue2448 ...

Fixes #2448 - Accept sourceFiles without module declaration 

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

Add test

fixes #2448 - Accept sourceFiles without module declaration

Posix: Add default SONAME/install_name for dynamic libraries ...

Setting it to the filename (ELF) or `@rpath/<file name>` (Mach-O).

This seems to be required in order to use the libraries copied to
the output directory of dependees - either by setting the
LD_LIBRARY_PATH env var when starting the executable, or baking a
RUNPATH into the executable (e.g., via `patchelf --set-rpath` for
deployment).

Otherwise the loader tries to load the library from the original
location specified in the linking cmdline, which is some
`<path to dep pkg>/.dub/…` path (and apparently potentially
relative, and thus even more brittle). For ELF, I would assume
that all SONAME + DT_NEEDED entries (of executable + all .so libs)
would need to be post-processed to enable loading the libs from the
executable's directory (well, any non-original location).

update version to v1.29.2-beta.1

Temporarily disable ldc-master as the dustmite test is broken

CI: Update versions of dub for fetch test ...

Those versions are old and broken with the latest compiler,
hence dub run will fail. Use the three most recent versions
(at the time of writing) to solve this issue.

PackageManager: Move class' extra searchPath/Package to an internal location ...

Those fields have been aggregated in Location and them being in `PackageManager`
stands out as a double duty (the `PackageManager` is both a `Location`
and act as a `Location` manager). By using a `Location` for them we can
start to streamline the code.

Use configy library to parse dub.json file ...

This will provide a vastly superior UX to the users:
- They will be warned about typos;
- The type information provided will be more accurate and always include file/line infos
  (e.g. no more 'Invalid SemVer versions');
- We can expose a better recipe struct to library users on the long run,
  by separating the actual data DUB uses from what it reads;
- We can be more descriptive about errors / intent, for example by using `SetInfo`;

Trivial: Convert some spaces to tabs

Trivial: Remove always-true condition ...

isExactVersion will never return true if repository.empty returns false.

Trivial: Remove test about the internals of Dependency ...

Now that Dependency is a proper SumType, this can never happen.

Trivial: Remove redundant comparison ...

optional and default_ are indeed checked by opEquals.

Trivial: Correct code to use match's value instead of property ...

This code was accidentally working as it was calling the path property,
essentially entering another match on the SumType.

Trivial: Remove dead import

Remove 'unknown' version ...

It was never used.

Dependency: Deprecate path/repository setters ...

There is no real use case for changing a dependency in place,
instead we better let the user overwrite the dependency itself.

initPackage: Use VersionRange instead of string

Merge pull request #2435 from Geod24/getBestPackage-redundancy ...

Remove redundant lookup when searching for a tool

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

remove unneccessary warning prefix ...

logWarn already adds a warning

Remove redundant lookup for tools package ...

Because the first call to getBestPackage will use VersionRange.Any,
it will match whatever the second call would match, but the second
call will only happen if there is no match, hence it should not be
possible that the second call ever returns something.

Trivial: Remove superfluous argument to getBestPackage ...

It's already the default.