Skip to content

ubuntu/arm: Fix cmake-data dependency#145

Merged
phlax merged 1 commit intoenvoyproxy:mainfrom
phlax:deps-testing
Sep 8, 2021
Merged

ubuntu/arm: Fix cmake-data dependency#145
phlax merged 1 commit intoenvoyproxy:mainfrom
phlax:deps-testing

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Sep 7, 2021

No description provided.

@phlax phlax requested a review from a team as a code owner September 7, 2021 09:11
Signed-off-by: Ryan Northey <ryan@synca.io>

# todo: remove this once kitware repo is fixed
if [[ "$ARCH" == "aarch64" ]]; then
PACKAGES+=("cmake-data=3.10.2-1ubuntu2.18.04.1")
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lizan there has been a new release of the kitware packages and it has broken ci

this fixes ci, but not sure its the correct fix - i checked through the kitware changelogs, but its not clear what was broken, so i had no way of telling why this was added

checking the last successful build, i see that on arm64 it was using the ubuntu-updates versions of cmake and cmake-data

https://dev.azure.com/cncf/envoy/_build/results?buildId=81674&view=logs&j=61711cda-f957-54e2-0b96-512b22e92cc3&t=b4ce43a0-cf0d-5605-7e86-dd35d793099e&l=9266

...so i guess the alternative would be to pin both if that is still required on arm64

Copy link
Copy Markdown
Member Author

@phlax phlax Sep 7, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems like i added the not so helpful comment about kitware, so apologies ill try and rem to add more helpful info in future

either way, im thinking that this means that this fix should be good

@phlax phlax merged commit e3c4fda into envoyproxy:main Sep 8, 2021
htuch pushed a commit that referenced this pull request Sep 8, 2021
  [skip ci]
  ubuntu/arm: Fix cmake-data dependency (#145)

Signed-off-by: Ryan Northey <ryan@synca.io>
htuch pushed a commit that referenced this pull request Sep 8, 2021
  [skip ci]
  Regenerate linux toolchains from e3c4fda

  [skip ci]
  ubuntu/arm: Fix cmake-data dependency (#145)

Signed-off-by: Ryan Northey <ryan@synca.io>
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.

2 participants