dub icon indicating copy to clipboard operation
dub copied to clipboard

Qualify NativePath members

Open nordlow opened this issue 3 years ago • 7 comments

Trivial. Ready for merge.

nordlow avatar Aug 18 '22 13:08 nordlow

Doesn't compile because Vibe.d has different qualifiers.

Geod24 avatar Aug 18 '22 17:08 Geod24

Doesn't compile because Vibe.d has different qualifiers.

Oops, thanks. Forgot to compile with unittests.

Fixed via https://github.com/dlang/dub/compare/e230fd0234ac42c60af9eeefefa6611c72e65072..fcbed3642763e9db8c2189e8f2e2eb9ec1400ad4.

dub test pass locally now. Ok to merge.

nordlow avatar Aug 18 '22 18:08 nordlow

Ehh, I'm clueless about the CI errors. How can a simple adding of qualifiers break CI in any way when dub test passes locally?

nordlow avatar Aug 18 '22 19:08 nordlow

How do you compile? Test what the CI is doing. Building optionally with vibe-core is probably what breaks it.

Geod24 avatar Aug 18 '22 20:08 Geod24

Building optionally with vibe-core is probably what breaks it.

How do I do that? I find no reference to vibe-core in top dub.sdl.

Do you mean any of the dub.sdl-configurations

configuration "library-nonet" {
	dependency "vibe-d:http" version=">=0.9.0 <0.10.0" optional=true
	targetType "library"
	excludedSourceFiles "source/app.d"
}

configuration "dynamic-library-nonet" {
	dependency "vibe-d:http" version=">=0.9.0 <0.10.0" optional=true
	targetType "dynamicLibrary"
	excludedSourceFiles "source/app.d"
}

?

nordlow avatar Aug 18 '22 20:08 nordlow

How do you compile? Test what the CI is doing. Building optionally with vibe-core is probably what breaks it.

Yes, but the problem is that the travis.sh script fails locally in on master and I find it really to hard to understand its output. We really need to do something about its output so it becomes obvious what are errors and not.

nordlow avatar Aug 19 '22 07:08 nordlow

How do you compile?

I do dub test locally.

Test what the CI is doing. Building optionally with vibe-core is probably what breaks it.

Grepping for vibe-core gives no hits. I don't understand what you mean.

nordlow avatar Aug 19 '22 07:08 nordlow

Modified and rebased but surprisingly fails in UT...

Geod24 avatar Oct 27 '22 14:10 Geod24

Any clues why some platforms are passing and some are not?

nordlow avatar Oct 27 '22 19:10 nordlow

Older compilers don't run test, they just compile

Geod24 avatar Oct 27 '22 19:10 Geod24

I recommend to go with something like https://github.com/dlang/dub/pull/2821 instead. Closing as stalled.

Geod24 avatar Jun 18 '24 11:06 Geod24