-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[201811][mlnx] add option to build sdk from sources #4378
Conversation
24a28e1
to
25b46fe
Compare
@@ -116,6 +116,7 @@ RUN apt-get update && apt-get install -y \ | |||
libdb-dev \ | |||
iptables-dev \ | |||
swig \ | |||
ctags \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why adding ctags?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On line L106, these packages are for mellanox sdk build. ctags is build dependency for python-sdk-api package
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everyone has no access to MLNX_SDK_SOURCE_BASE_URL, but need one extra package in sonic-slave. It is weird for a package to depend on ctags during build time. Could you check if you can remove this dependency internally?
In reply to: 404943374 [](ancestors = 404943374)
![:shipit: :shipit:](https://github.githubassets.com/images/icons/emoji/shipit.png)
![:shipit: :shipit:](https://github.githubassets.com/images/icons/emoji/shipit.png)
Signed-off-by: Stepan Blyschak <stepanb@mellanox.com>
25b46fe
to
9318ee9
Compare
retest broadcom please |
Signed-off-by: Stepan Blyschak stepanb@mellanox.com
- What I did
- How I did it
- How to verify it
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)