You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by ma...@apache.org on 2018/10/28 16:45:31 UTC

[kafka] branch 1.0 updated: MINOR: Remove unintentional tilde character from kafka-run-class.bat script

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

manikumar pushed a commit to branch 1.0
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/1.0 by this push:
     new 4d3910f  MINOR: Remove unintentional tilde character from kafka-run-class.bat script
4d3910f is described below

commit 4d3910f106b78654fd55d6530f856633f4e0c610
Author: u214578 <fl...@sbb.ch>
AuthorDate: Sun Oct 28 11:59:08 2018 +0530

    MINOR: Remove unintentional tilde character from kafka-run-class.bat script
    
    It seems a tipo was made on a avariable for the log dir with commit 81e789ae3dc6ea8369db181c5aef440491d74f19. Then Windows tries to access by default the /log directory which of cause not exists.
    klesta490 Is it ok so? Or was the tilde ~ intentional?
    
    To test:
    On Windows with the a fresh downloaded Kafka, adapt the properties:
    * dataDir in config/zookeeper.properties with what you want, windows-compatible
    * log.dirs in config/server.properties with what you want, windows-compatible
    and executes:
    bin\windows\zookeeper-server-start.bat config\zookeeper.properties
    
    Author: u214578 <fl...@sbb.ch>
    
    Reviewers: Vladimír Kleštinec <kl...@gmail.com>, Vahid Hashemian <va...@gmail.com>
    
    Closes #5837 from florianhof/feature/fix_windows_log_param
---
 bin/windows/kafka-run-class.bat | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bin/windows/kafka-run-class.bat b/bin/windows/kafka-run-class.bat
index 1dfd9a5..a21171b 100755
--- a/bin/windows/kafka-run-class.bat
+++ b/bin/windows/kafka-run-class.bat
@@ -111,7 +111,7 @@ IF ["%JMX_PORT%"] NEQ [""] (
 
 rem Log directory to use
 IF ["%LOG_DIR%"] EQU [""] (
-    set LOG_DIR="%BASE_DIR~%/logs"
+    set LOG_DIR="%BASE_DIR%/logs"
 )
 
 rem Log4j settings