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

[platform mellanox] Substitute '-' symbol to '.' in Mellanox SDK vers… #5756

Merged

Conversation

ayurkiv-nvda
Copy link
Contributor

…ion to prevent problems with finding deb package

Signed-off-by: Andriy Yurkiv [email protected]

- Why I did it
In case of non-GA SDK version there is '-' symbol in Mellanox SDK version name. (For example: 4.4.1306-006)
In appropriate .deb packet there is '.' instead of '-'. Because of this there was problem while building SDK
- How I did it
I have replaced '-' symbol with '.' for MLNX_SDK_DEB_VERSION
- How to verify it

  1. build SDK (non-GA version with suffix '-00X') without fix. Building will fail because it is impossible to find appropriate package
  2. build SDK (non-GA version with suffix '-00X') with fix. Everything should be OK

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

  • 201811
  • 201911
  • 202006

- Description for the changelog

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

…ion to prevent problems with finding deb package

Signed-off-by: Andriy Yurkiv <[email protected]>
@volodymyrsamotiy
Copy link
Collaborator

retest this please

@stepanblyschak stepanblyschak self-requested a review November 3, 2020 15:13
@keboliu
Copy link
Collaborator

keboliu commented Nov 4, 2020

retest this please

@ayurkiv-nvda
Copy link
Contributor Author

retest vs please

@ayurkiv-nvda
Copy link
Contributor Author

retest vsimage please

@lguohan lguohan merged commit 8100711 into sonic-net:master Nov 11, 2020
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
…ion to prevent problems with finding deb package (sonic-net#5756)

In case of non-GA SDK version there is '-' symbol in Mellanox SDK version name. (For example: 4.4.1306-006) 
In appropriate .deb packet there is '.' instead of '-'. Because of this there was problem while building SDK 

Signed-off-by: Andriy Yurkiv <[email protected]>
@ayurkiv-nvda ayurkiv-nvda deleted the master_fix_sdk_notation_issue branch May 30, 2023 21:45
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.

5 participants