[sonic-yang-mgmt]: Handle dependencies in Makefile rather than setup.py

[praveen-li]

Changes:
-- Removing the part where build dependencies are installed in setup.py.
-- Adding build dependencies in corresponsing rules..*.mk file.

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

- Why I did it
Fix below issues reported by Joe:

I  noticed that master branch images are now almost consistently failing to build with the following error signature if building using multiple build jobs (e.g., SONIC_BUILD_JOBS=20):

 

dpkg: error: dpkg frontend lock is locked by another process

 

I tracked the root cause down to the building of the sonic_yang_mgmt-1.0-py2-none-any.whl package. The problem here is that in your setup.py file, you are explicitly installing .deb packages as build requirements using `dpkg -i` at the following line: https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-mgmt/setup.py#L43

 

However, dpkg cannot handle parallel operations. We encountered this problem before and worked around it by serializing all .deb package installations here: https://github.com/Azure/sonic-buildimage/blob/master/slave.mk#L483. The new issue is caused because the whl package recipe is executed in parallel with the aforementioned serialized .deb package installation recipe.

- How I did it
-- Removing the part where build dependencies are installed in setup.py.
-- Adding build dependencies in corresponsing rules..*.mk file.

- How to verify it
Have to depend on JENKINS JOBS. Since Problem happens when SONIC_BUILD_JOBS=20.

- Description for the changelog

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

[lgtm-com]

This pull request introduces 1 alert when merging 13f2bd3 into f0eb1db - view on LGTM.com

new alerts:

• 1 for Unused import

[lguohan]

retest vsimage please