-
Notifications
You must be signed in to change notification settings - Fork 105
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
hardening/326_libthrift_issue_fix_from_pom_xml #706
hardening/326_libthrift_issue_fix_from_pom_xml #706
Conversation
@@ -62,8 +62,6 @@ Apache Flume can be easily installed by downloading its latests version from [fl | |||
$ mv apache-flume-1.4.0-bin APACHE_FLUME_HOME | |||
$ mv APACHE_FLUME_HOME/lib/httpclient-4.2.1.jar APACHE_FLUME_HOME/lib/httpclient-4.2.1.jar.old | |||
$ mv APACHE_FLUME_HOME/lib/httpcore-4.2.1.jar APACHE_FLUME_HOME/lib/httpcore-4.2.1.jar.old | |||
$ wget http://repo1.maven.org/maven2/org/apache/thrift/libthrift/0.9.1/libthrift-0.9.1.jar -P APACHE_FLUME_HOME/lib | |||
$ mv APACHE_FLUME_HOME/lib/libthrift-0.7.0.jar APACHE_FLUME_HOME/lib/libthrift-0.7.0.jar.old |
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.
This mv
command is necessary. Just think this is the manual installation, if we don't rename the old libthrift library, nobody will do that.
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.
Fixed in 4374a6b
CHANGES_NEXT_RELEASE update is missing. |
CHANGES_NEXT_RELEASE updated in 4374a6b |
LGTM, passing the ball to @vgarciag |
LGTM |
…ue_fix_from_pom_xml hardening/326_libthrift_issue_fix_from_pom_xml
I checked my installation path, in
/usr/cygnus/lib
wherelibthrift-0.7.0.old
has the .old extension andlibthrift-0.9.1.jar
doesn't appear due to it's included in the .jar filecygnus-0.11.0-jar-with-dependencies.jar