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

[build] No longer install scapy Debian package in host #6919

Merged
merged 1 commit into from
Mar 1, 2021
Merged

[build] No longer install scapy Debian package in host #6919

merged 1 commit into from
Mar 1, 2021

Conversation

jleveque
Copy link
Contributor

Why I did it

As of the merging of PR #6799, we are now installing a newer version of scapy via pip, therefore there is no longer a need to install the older Debian package.

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

  • 201811
  • 201911
  • 202006
  • 202012

@jleveque jleveque self-assigned this Feb 27, 2021
@jleveque jleveque changed the title [build] No longer install scpay Debian package in host [build] No longer install scapy Debian package in host Feb 27, 2021
@lguohan lguohan merged commit 5d48581 into sonic-net:master Mar 1, 2021
@jleveque jleveque deleted the remove_scapy_deb branch March 1, 2021 16:36
yxieca pushed a commit that referenced this pull request Mar 4, 2021
As of the merging of PR #6799, we are now installing a newer version of scapy via pip, therefore there is no longer a need to install the older Debian package.
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
As of the merging of PR sonic-net#6799, we are now installing a newer version of scapy via pip, therefore there is no longer a need to install the older Debian package.
lolyu pushed a commit to lolyu/sonic-buildimage that referenced this pull request Sep 13, 2021
As of the merging of PR sonic-net#6799, we are now installing a newer version of scapy via pip, therefore there is no longer a need to install the older Debian package.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants