Skip to content

[platform/cel-dx010]: Add python3 installer for sonic_platform package#233

Merged
mudsut4ke merged 2 commits intoSONIC-DEV:master_dx010_fix_failed_test_cases_of_apifrom
mudsut4ke:master_dx010_add_python_3_support
Feb 3, 2021
Merged

[platform/cel-dx010]: Add python3 installer for sonic_platform package#233
mudsut4ke merged 2 commits intoSONIC-DEV:master_dx010_fix_failed_test_cases_of_apifrom
mudsut4ke:master_dx010_add_python_3_support

Conversation

@mudsut4ke
Copy link

- What I did

  • Add python3 installer for sonic_platform package

- How I did it

  1. Add rule to compile sonic_platform package with python3
  2. Add command to install python3 package of sonic_platform

- How to verify it

  • sonic_platform must installed in this /usr/local/lib/python3.7/dist-packages/sonic_platform under sonic host and pmon docker

- Description for the changelog

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

@mudsut4ke mudsut4ke changed the title [platform/cel-dx010]: Add python3 installer for sonic_platformpackage [platform/cel-dx010]: Add python3 installer for sonic_platform package Feb 3, 2021
@mudsut4ke mudsut4ke merged commit e890bd5 into SONIC-DEV:master_dx010_fix_failed_test_cases_of_api Feb 3, 2021
fi; \
cd $(MOD_SRC_DIR)/$${mod}; \
python2.7 setup.py bdist_wheel -d $(MOD_SRC_DIR)/$${mod}/modules; \
python3 setup.py bdist_wheel -d $(MOD_SRC_DIR)/$${mod}/modules; \

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know why we need setup both pyth2.7 and python3?

mudsut4ke pushed a commit that referenced this pull request Jan 28, 2022
c4127c2 [psud] Fix PSU log issue (#235)
07542cb [pmon][xcvrd]xcvrd process show backtrace on the internal port. (#233)
3e432e7 [Y-Cable] Increased unit test coverage of y_cable_helper.py (#229)
7c363f5 [ledd] prevent led crash on recirc port event (#232)
e9ccd82 [sonic-platform-daemons] fix dependency issue on py2 wheels by correcting the path (#234)
2b0acfb [sfp-refactoring] xcvrd: add initial support for CMIS application initialization (#217)
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