You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by jo...@apache.org on 2019/04/04 04:47:05 UTC

[mesos] branch master updated: Fixed compile error with a built-in logrotate module.

This is an automated email from the ASF dual-hosted git repository.

josephwu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git


The following commit(s) were added to refs/heads/master by this push:
     new b3a6a9b  Fixed compile error with a built-in logrotate module.
b3a6a9b is described below

commit b3a6a9bf41584021e2502922bace5092dc46c71a
Author: Andrei Budnik <ab...@mesosphere.com>
AuthorDate: Wed Apr 3 21:43:13 2019 -0700

    Fixed compile error with a built-in logrotate module.
    
    This patch gets rid of redundant use of `Error` wrapper for an error
    message that is passed to EXIT macro. This fixes a compile error on
    Mesos versions that do not support output operator for `Error` type.
    
    Review: https://reviews.apache.org/r/70369/
---
 src/slave/container_loggers/logrotate.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/slave/container_loggers/logrotate.cpp b/src/slave/container_loggers/logrotate.cpp
index 528dea4..94a411a 100644
--- a/src/slave/container_loggers/logrotate.cpp
+++ b/src/slave/container_loggers/logrotate.cpp
@@ -339,7 +339,7 @@ int main(int argc, char** argv)
   Try<LogrotateLoggerProcess*> process = LogrotateLoggerProcess::create(flags);
   if (process.isError()) {
     EXIT(EXIT_FAILURE)
-      << Error("Failed to create Logrotate process: " + process.error());
+      << "Failed to create Logrotate process: " << process.error();
   }
 
   spawn(process.get());