Skip to content

Commit

Permalink
[Y-Cable] add the definition inside setup.py to include sonic_y_cable…
Browse files Browse the repository at this point in the history
….credo as a package (sonic-net#211)

Signed-off-by: vaibhav-dahiya <[email protected]>
Curreently the way python wheel includes packages is all the subdirectories inside a package need to be also included
for them to be a package. Since in the refactor for multiple vendor Y-Cable xcvrd calls these vendor specific modules by including their name which it obtains by reading the eeprom it is required to include them inside setup.py for the wheel to know it has a package.

Signed-off-by: vaibhav-dahiya [email protected]

Description
add the definition inside setup.py to include sonic_y_cabe.credo as a package

Motivation and Context
This is required by all calling API's for vendor agnostic Y-Cable API's to be called for credo specific cables.

How Has This Been Tested?
Built an image and validated the presence of the pacakge.
Ran the API's on a Arista7050cx3 switch.
  • Loading branch information
vdahiya12 authored Aug 12, 2021
1 parent cd3cca7 commit ce51894
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
'sonic_sfp',
'sonic_thermal',
'sonic_y_cable',
'sonic_y_cable.credo',
],
# NOTE: Install also depends on sonic-config-engine for portconfig.py
# This dependency should be eliminated by moving portconfig.py
Expand Down
Empty file added sonic_y_cable/credo/__init__.py
Empty file.

0 comments on commit ce51894

Please sign in to comment.