Skip to content

[CENTEC ARM64]LIBSAIREDIS isn't depend on CENTEC_SAI, Remove this dependency#5834

Merged
lguohan merged 2 commits intosonic-net:masterfrom
guxianghong:sonic-master-202011060932
Nov 16, 2020
Merged

[CENTEC ARM64]LIBSAIREDIS isn't depend on CENTEC_SAI, Remove this dependency#5834
lguohan merged 2 commits intosonic-net:masterfrom
guxianghong:sonic-master-202011060932

Conversation

@guxianghong
Copy link
Copy Markdown
Contributor

@guxianghong guxianghong commented Nov 6, 2020

- Why I did it
Build depends have been optimized in PR #4880 and PR #5039. These optimizations need to be merged to Centec ARM64 platform.

- How I did it
Remove the dependency

- How to verify it
Compile Centec ARM64 based images

- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006

- Description for the changelog

LIBSAIREDIS isn't depend on CENTEC_SAI, Remove this dependency

- A picture of a cute animal (not mandatory but encouraged)

@guxianghong
Copy link
Copy Markdown
Contributor Author

Retest vs please

…Merge these optimization to Centec ARM64 platform.
@ghost
Copy link
Copy Markdown

ghost commented Nov 9, 2020

CLA assistant check
All CLA requirements met.

@lguohan lguohan merged commit eeffbb2 into sonic-net:master Nov 16, 2020
@lguohan
Copy link
Copy Markdown
Collaborator

lguohan commented Nov 16, 2020

i do not recall why #5039 and #4880 missed this centec arm64 platform. maybe grep miss, i guess, but sorry for that.

@guxianghong
Copy link
Copy Markdown
Contributor Author

i do not recall why #5039 and #4880 missed this centec arm64 platform. maybe grep miss, i guess, but sorry for that.

Thanks guohan for reviewing and merging this PR.

santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
…endency (sonic-net#5834)

* LIBSAIREDIS isn't depend on CENTEC_SAI remove this dependence

* Build depends are optimized in PR sonic-net#4880 and sonic-net#5039. Merge these optimization to Centec ARM64 platform.
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