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

ARROW-7615: [CI][Gandiva] Ensure gandiva_jni library has only a whitelisted set of shared dependencies #6423

Closed
wants to merge 4 commits into from

Conversation

projjal
Copy link
Contributor

@projjal projjal commented Feb 14, 2020

No description provided.

@projjal
Copy link
Contributor Author

projjal commented Feb 14, 2020

@ursabot crossbow submit -g gandiva

@ursabot
Copy link

ursabot commented Feb 14, 2020

AMD64 Conda Crossbow Submit (#90712) builder has been succeeded.

Revision: 4f10a74

Submitted crossbow builds: ursa-labs/crossbow @ ursabot-506

Task Status
gandiva-jar-osx TravisCI
gandiva-jar-trusty TravisCI

@github-actions
Copy link

@@ -53,6 +53,9 @@ script:
- dev/tasks/gandiva-jars/build-cpp-osx.sh || travis_terminate 1
- dev/tasks/gandiva-jars/build-java.sh || travis_terminate 1
# deploy using crossbow
# pip install pygit2 fails if MACOS_DEPLOYMENT_TARGET is set to an earlier version. Since the
# gandiva libraries are already built we can reset MACOS_DEPLOYMENT_TARGET to curresnt osx version
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit! curresnt ->current

@@ -23,9 +23,33 @@ CPP_BUILD_DIR=$TRAVIS_BUILD_DIR/dist/

pushd java
if [ $TRAVIS_OS_NAME == "linux" ]; then
ldd $CPP_BUILD_DIR/libgandiva_jni.so
SO_DEP=ldd
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we do this config based..so that it does not affect the nightly ones that the arrow community has..

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added the check inside an if condition to see if environment variable CHECK_SHARED_DEPENDENCIES is set. The variable can be set in travis repository settings.

@projjal
Copy link
Contributor Author

projjal commented Feb 14, 2020

@ursabot crossbow submit -g gandiva

@ursabot
Copy link

ursabot commented Feb 14, 2020

AMD64 Conda Crossbow Submit (#90757) builder has been succeeded.

Revision: d03c3e3

Submitted crossbow builds: ursa-labs/crossbow @ ursabot-508

Task Status
gandiva-jar-osx TravisCI
gandiva-jar-trusty TravisCI

@projjal
Copy link
Contributor Author

projjal commented Feb 15, 2020

@ursabot crossbow submit -g gandiva

@ursabot
Copy link

ursabot commented Feb 15, 2020

AMD64 Conda Crossbow Submit (#90852) builder has been succeeded.

Revision: fefe0d1

Submitted crossbow builds: ursa-labs/crossbow @ ursabot-510

Task Status
gandiva-jar-osx TravisCI
gandiva-jar-trusty TravisCI

Copy link
Contributor

@praveenbingo praveenbingo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 thanks @projjal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants