Remove python dependency on host when building local rpm

SUSE / DeepSea

A collection of Salt files for deploying, managing and automating Ceph.

GNU General Public License v3.0

161 stars 75 forks source link

Remove python dependency on host when building local rpm #1790

Closed swiftgist closed 4 years ago

swiftgist commented 4 years ago

Signed-off-by: Eric Jackson ejackson@suse.com

Description: The pyc files require build host to have a compatible python3 to build an rpm.

Checklist:

[ ] Added unittests and or functional tests
[ ] Adapted documentation
[ ] Referenced issues or internal bugtracker
[ ] Ran integration tests successfully (trigger with "@susebot run teuthology" in a GitHub comment; see the wiki for more information)

swiftgist commented 4 years ago

@susebot run teuthology

swiftgist commented 4 years ago

/lgtm

Maybe we should also remove the pyc target from the Makefile too ?

Possibly... somebody may want still want this for the make install or maybe in QA. It's not a development environment requirement. If nobody pipes up, I'm okay with removing it altogether in another PR.

susebot commented 4 years ago

Commit e5d2c08b03062c2df35e72f00990f10794b07900 is OK for suite deepsea:tier2. Check tests results in the Jenkins job: http://ci.ses.suse.de:8080/job/pr-deepsea/357/