dlang / ci

dlang CI testing pipelines
https://buildkite.com/dlang
Boost Software License 1.0
13 stars 29 forks source link

Buildkite should ideally define TRAVIS_OS_NAME and TRAVIS_CPU_ARCH correctly #433

Closed p0nce closed 4 years ago

p0nce commented 4 years ago

See: https://github.com/AuburnSounds/intel-intrinsics/issues/49

The problemis that we need those envvars.

For example, intel-intrinsics is tested on both arm64 and amd64 so we need this variable to perform any testing in the first place. Also used to discriminate and test 32-bit x86 on Linux.

I think Buildkite should define TRAVIS_OS_NAME and TRAVIS_CPU_ARCH correctly else it will manage to bypass most unittests.

p0nce commented 4 years ago

Ok, I'm the only one using it it seems. Closing in favour of #434