Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci,azure-pipelines: use matrix strategy, and user artifacts from libiio's pipeline #62

Merged
merged 7 commits into from
Dec 22, 2020

Conversation

commodo
Copy link
Contributor

@commodo commodo commented Dec 10, 2020

Matrix strategy adapted from Travis Collins' work on libiio with Azure Pipelines.
This reduces duplication of the same steps for Linux & Mac builds.

Added artifact publishing. These are pipeline artifacts.
Also, using libiio's artifacts for the build.
For now, we're picking up the latest artifact, but later we may add branch matching (by un-commenting the bits in the yaml file).

Signed-off-by: Alexandru Ardelean alexandru.ardelean@analog.com

@commodo commodo requested review from tfcollins, mhennerich and a team December 10, 2020 15:22
@tfcollins
Copy link
Contributor

Are we waiting for libiio to make artifacts available for this job to run?

@commodo commodo force-pushed the staging/azure-pipelines branch from eef9824 to e48344d Compare December 11, 2020 12:29
@commodo
Copy link
Contributor Author

commodo commented Dec 11, 2020

Are we waiting for libiio to make artifacts available for this job to run?

yep; this is one of those PRs that is also testing that PRs work with CI :p

@commodo
Copy link
Contributor Author

commodo commented Dec 11, 2020

Seems the artifacts for the master branch have been picked up this time for the PR.
This will take a bit of fine-tuning in the next few weeks/months, but I figure this would work reasonably well over time.

We may have to publish some sort of release artifacts at some point.
But just for having these [as they are now] is good enough for me.

@commodo commodo force-pushed the staging/azure-pipelines branch 7 times, most recently from b6d63cf to 0be2d4a Compare December 18, 2020 14:24
@commodo commodo force-pushed the staging/azure-pipelines branch 4 times, most recently from 93b4c92 to 02d8f0c Compare December 22, 2020 14:25
Includes rework to use matrix strategy and artifacts publishing for
libad9361-iio for Linux & Mac & Win.

This build exports the ad9361.h header instead of the iio.h header.

Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
Up until now libiio was always installed.
But for Azure we want to install it from the pipeline artifacts.

Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
This is useful for CIs or mechanisms that already provided the artifact for
us. This way we can install it and run the build.

Also pass it to the inside-docker.sh script via INSIDE_DOCKER_TRAVIS_CI_ENV.

Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
…nux/Mac)

With this, the libad9361-iio build will download artifacts from the libiio
pipeline, and use them.

Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
The lib.sh script library has been updated (now with the move to Azure).
It did require some changes to have some cleaner support for multi-arch
docker images.

This change updates the build scripts for Linux in a similar way.

Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
Required on some Ubuntu/Debian dockers, so that timezone setup doesn't
kick-in.

Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
Roughly adapted from libiio.

Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
@commodo commodo force-pushed the staging/azure-pipelines branch from 02d8f0c to 919fa69 Compare December 22, 2020 14:31
@commodo
Copy link
Contributor Author

commodo commented Dec 22, 2020

Closing and reopening to trigger a new build

@commodo commodo closed this Dec 22, 2020
@commodo commodo reopened this Dec 22, 2020
@commodo
Copy link
Contributor Author

commodo commented Dec 22, 2020

disabled appveyor builds for this repo;
azure looks fully green

@commodo commodo merged commit bc1ac92 into master Dec 22, 2020
@commodo commodo deleted the staging/azure-pipelines branch December 22, 2020 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants