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

[slave.mk]: Adding support to specify debs dependencies for python-de… #4849

Merged
merged 1 commit into from
Jun 26, 2020

Conversation

praveen-li
Copy link
Collaborator

…bs package.

Adding support to specify build time DEB package dependencies for python-debs package.

Signed-off-by: Praveen Chaudhary pchaudhary@linkedin.com

- Why I did it
Adding support to specify build time DEB package dependencies for python-debs package.

Why it is needed:
To run new tests in sonic_utils, I need to add libyang*.deb as bbuild time dependencies.

- How I did it
added below line:
$$(addsuffix -install,$$(addprefix $(DEBS_PATH)/,$$($$*_DEBS_DEPENDS))) \

- How to verify it
Added below dependencies in sonic-utils

SONIC_UTILS = python-sonic-utilities_1.2-1_all.deb
$(SONIC_UTILS)_SRC_PATH = $(SRC_PATH)/sonic-utilities
$(SONIC_UTILS)_DEBS_DEPENDS = $(LIBYANG) $(LIBYANG_CPP) $(LIBYANG_PY2) \  <<<<<<<<<<<
                                     $(LIBYANG_PY3)
$(SONIC_UTILS)_WHEEL_DEPENDS = $(SONIC_CONFIG_ENGINE) $(SONIC_YANG_MGMT_PY)
SONIC_PYTHON_STDEB_DEBS += $(SONIC_UTILS)

Build the PKGs successfully.

- Description for the changelog

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

…bs package.

Adding support to specify build time DEB package dependencies for python-debs package.

Signed-off-by: Praveen Chaudhary pchaudhary@linkedin.com
slave.mk Show resolved Hide resolved
@praveen-li
Copy link
Collaborator Author

@jleveque

Kindly review this one as well. Thx a lot.

@jleveque jleveque merged commit 94d448e into sonic-net:master Jun 26, 2020
pjaipakdee19 pushed a commit to pjaipakdee19/sonic-buildimage that referenced this pull request Jul 7, 2020
…bs package. (sonic-net#4849)

**- How I did it**
added below line:
$$(addsuffix -install,$$(addprefix $(DEBS_PATH)/,$$($$*_DEBS_DEPENDS))) \

**- How to verify it**
Added below dependencies in sonic-utils
```
SONIC_UTILS = python-sonic-utilities_1.2-1_all.deb
$(SONIC_UTILS)_SRC_PATH = $(SRC_PATH)/sonic-utilities
$(SONIC_UTILS)_DEBS_DEPENDS = $(LIBYANG) $(LIBYANG_CPP) $(LIBYANG_PY2) \  <<<<<<<<<<<
                                     $(LIBYANG_PY3)
$(SONIC_UTILS)_WHEEL_DEPENDS = $(SONIC_CONFIG_ENGINE) $(SONIC_YANG_MGMT_PY)
SONIC_PYTHON_STDEB_DEBS += $(SONIC_UTILS)
```
Build the PKGs successfully.

Signed-off-by: Praveen Chaudhary pchaudhary@linkedin.com
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.

2 participants