search

inexorgame-obsolete / deprecated-cube-engine-inexor

UNMAINTAINED: Please have a look at the vulkan-renderer
https://inexor.org
zlib License
11 stars 1 forks source link

CI prebuilds: Split each jobs into two #542

Closed Croydon closed 5 years ago

Croydon commented 6 years ago

We are getting very close and sometimes over the 1h limit of our jobs, causing the entire CI build to fail.

I'm going to split it into two jobs:

This is not yet critical as it happens mostly on Windows and I have changed the order there: the Conan packages are getting uploaded before testing, which means that we still have the prebuilds, just no reliable tests for each and every single compiler version/configuration anymore. We still test our main compiler (versions)/configs in our main repo.

a-teammate commented 6 years ago

Another note: We have some preprocessor magic in the travis.yml and appveyor.yml. Could we maybe split this into two files? Would make the code clearer. I think there is an option for travis as well as appveyor to use a yml with a different name. Otherwise ignore this comment.

Croydon commented 6 years ago

I think there is an option for travis as well as appveyor to use a yml with a different name.

There is an option for AppVeyor but not for Travis. Since it's just a very minimalistic amount of pre-processor logic for AppVeyor I feel much more comfortable to keep it the same for AppVeyor and Travis.

Croydon commented 5 years ago

Since I have moved things to the Conan build strategy "outdated" the build time got reduced a lot. It's very rare now that we get close to the one hour maximum.

This means that we are not testing every package every time in every configuration, but since the entire Conan ecosystem and also our CI system got a lot more reliable, I don't think that this is required anymore.

Closing the issue because of irrelevance.