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

HPCC-16661 Added libhiredis submodule #9372

Merged

Conversation

Michael-Gardner
Copy link
Contributor

Signed-off-by: Michael Gardner michael.gardner@lexisnexis.com

Signed-off-by: Michael Gardner <michael.gardner@lexisnexis.com>
@hpcc-jirabot
Copy link

@Michael-Gardner
Copy link
Contributor Author

Copy link
Member

@xwang2713 xwang2713 left a comment

Choose a reason for hiding this comment

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

Looks good

@Michael-Gardner
Copy link
Contributor Author

Michael-Gardner commented Nov 29, 2016

I've tested this build on centos 6 (no native hiredis package installed) and ubuntu 16.04 (has native hiredis package installed.) Links correctly to our supplied version.

@@ -43,3 +43,6 @@
[submodule "libcouchbase"]
path = plugins/couchbase/libcouchbase
url = https://github.com/hpcc-systems/libcouchbase.git
[submodule "plugins/redis/hiredis"]
path = plugins/redis/hiredis
url = https://github.com/redis/hiredis.git
Copy link
Member

Choose a reason for hiding this comment

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

As with the other submodules, we need to clone this to ensure that it continues to exist

Copy link
Member

Choose a reason for hiding this comment

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

Fork done

@Michael-Gardner
Copy link
Contributor Author

@richardkchapman Just updated the URL. Going to do a quick test build to make sure all is well.

@Michael-Gardner
Copy link
Contributor Author

@richardkchapman tested on centos 6 and ubuntu 16.04. Worked as before (not that I was expecting anything different, the commit it points to is the same despite the fork). I went ahead and squashed the commits.

@HPCCSmoketest
Copy link
Contributor

Automated Smoketest
Sha: 5858561
Build: success
ECL Watch: Rebuilding Site

errors warnings build time
0 65 90.476 seconds

Install hpccsystems-platform-community_6.2.0-rc5.el7.x86_64.rpm
HPCC Start: OK

Unittest result:

total passed failed timeout
88 88 0 0

HPCC Stop: OK
HPCC Uninstall: OK

@richardkchapman richardkchapman merged commit 62aeadf into hpcc-systems:candidate-6.2.0 Dec 1, 2016
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.

5 participants