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: build fixes #8770
cmake: build fixes #8770
Conversation
@cbodley here's the PR |
lgtm |
@@ -14,6 +14,9 @@ | |||
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |||
# GNU Library Public License for more details. | |||
# | |||
source $(dirname $0)/detect-build-env-vars.sh | |||
CEPH_ROOT=${CEPH_ROOT:-..} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't need to set CEPH_ROOT in scripts that run detect-build-env-vars.sh
@dzafman This is the PR I was talking about in the core meeting |
@alimaredia The detect-build-env-vars.sh looks reasonable. I had a version that used $(pwd) to get an absolute path instead relative paths but I can't say that one is better than the other. |
@alimaredia make check test cluster tests are still failing, see jenkins output |
Signed-off-by: Ali Maredia <amaredia@redhat.com>
Signed-off-by: Ali Maredia <amaredia@redhat.com>
Signed-off-by: Ali Maredia <amaredia@redhat.com>
Signed-off-by: Ali Maredia <amaredia@redhat.com>
This makes it possible to run applicable unit tests without setting environment varibles after building with autotools. This will be reverted once autotools is deprecated. Signed-off-by: Ali Maredia <amaredia@redhat.com>
Signed-off-by: Ali Maredia <amaredia@redhat.com>
Signed-off-by: Ali Maredia <amaredia@redhat.com>
plugin_dir and compressor_dir not needed by test_compression_snappy & test_compression_zlib because they statically link compressor libraries. Signed-off-by: Ali Maredia <amaredia@redhat.com>
Signed-off-by: Ali Maredia <amaredia@redhat.com>
@dachary or anyone else, since this isn't causing Jenkins problems, can this be good to merge? |
This PR makes all the tests that need environment variables run in
make check
have them set automatically as long as the tests are being executed out of the proper directory.