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

cmake: install platlib into a subdir of build-base dir #10666

Merged
merged 1 commit into from Aug 10, 2016
Merged

Conversation

tchaikov
Copy link
Contributor

distutils' install command installs all stuff under $build-base, which
includes the temporary files. and rpm packaging does not like::
Installed (but unpackaged) file(s)
and to support both py2 and py3, we should encode the python version
into the path, otherwise the built files will be overwritten.

Signed-off-by: Kefu Chai kchai@redhat.com

@tchaikov
Copy link
Contributor Author

should wait until gb are happy. i failed to do so last time...

@jdurgin
Copy link
Member

jdurgin commented Aug 10, 2016

jenkins failed - perhaps need to adjust AddCephTest again

distutils' install command installs all stuff under $build-base, which
includes the temporary files. and rpm packaging does not like::
  Installed (but unpackaged) file(s)
and to support both py2 and py3, we should encode the python version
into the path, otherwise the built files will be overwritten.

Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov
Copy link
Contributor Author

@jdurgin thanks. fixed and repushed

@jdurgin
Copy link
Member

jdurgin commented Aug 10, 2016

lgtm if the gitbuilders pass

@tchaikov tchaikov merged commit ffe2c05 into master Aug 10, 2016
@tchaikov
Copy link
Contributor Author

they are happy.

@tchaikov tchaikov deleted the wip-cmake branch August 10, 2016 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants