dub icon indicating copy to clipboard operation
dub copied to clipboard

CI: Replace old GDC script with matrix job

Open Geod24 opened this issue 3 years ago • 4 comments

Since COVERAGE is always false, this never ran. With an up-to-date GDC now available, we can re-enable it.

Geod24 avatar Jul 18 '22 16:07 Geod24

Needs https://github.com/dlang-community/setup-dlang/pull/57 merged & released

Geod24 avatar Jul 19 '22 09:07 Geod24

So OpenSSL doesn't support gdc. This can be worked around with something like:

+// Used by the CI
+configuration "library-nonet-gdc" {
+       dependency "vibe-d:http" version=">=0.9.0 <0.10.0"
+    dependency "openssl" version="~>3.0"
+    subConfiguration "openssl" "library-manual-version"
+       targetType "library"
+       excludedSourceFiles "source/app.d"
+}

but then, I found out that -Werror turns on -Werror=deprecated, so essentially -de, which makes GDC a bit unusable (!). CC @ibuclaw . Will see what we can do on dub side when I get a bit of free time.

Geod24 avatar Jul 20 '22 08:07 Geod24

but then, I found out that -Werror turns on -Werror=deprecated, so essentially -de, which makes GDC a bit unusable (!). CC @ibuclaw . Will see what we can do on dub side when I get a bit of free time.

DMD is the weird one for having a split brain between warnings and deprecations. It's all warnings as far as gcc diagnostics are concerned.

ibuclaw avatar Jul 22 '22 06:07 ibuclaw

issue is here: https://github.com/dlang/dub/runs/7418736496?check_suite_focus=true#step:8:76

it tries to run gdc -run which doesn't exist, probably need to adjust a test here.

WebFreak001 avatar Jul 28 '22 18:07 WebFreak001