You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@karaf.apache.org by jb...@apache.org on 2016/08/23 06:56:58 UTC
[1/2] karaf git commit: [KARAF-4511] grep,
fixed '--color never' support
Repository: karaf
Updated Branches:
refs/heads/master 5e310d510 -> 329e260f1
[KARAF-4511] grep, fixed '--color never' support
Project: http://git-wip-us.apache.org/repos/asf/karaf/repo
Commit: http://git-wip-us.apache.org/repos/asf/karaf/commit/6c993776
Tree: http://git-wip-us.apache.org/repos/asf/karaf/tree/6c993776
Diff: http://git-wip-us.apache.org/repos/asf/karaf/diff/6c993776
Branch: refs/heads/master
Commit: 6c993776af7af90ea91d1d50e2cda2613a5906ed
Parents: 5e310d5
Author: Paolo Antinori <pa...@redhat.com>
Authored: Fri Apr 29 14:40:55 2016 +0200
Committer: Jean-Baptiste Onofr� <jb...@apache.org>
Committed: Tue Aug 23 08:33:20 2016 +0200
----------------------------------------------------------------------
.../karaf/shell/commands/impl/GrepAction.java | 5 +++--
.../karaf/shell/commands/impl/GrepTest.java | 21 ++++++++++++++++++++
2 files changed, 24 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/karaf/blob/6c993776/shell/commands/src/main/java/org/apache/karaf/shell/commands/impl/GrepAction.java
----------------------------------------------------------------------
diff --git a/shell/commands/src/main/java/org/apache/karaf/shell/commands/impl/GrepAction.java b/shell/commands/src/main/java/org/apache/karaf/shell/commands/impl/GrepAction.java
index 4045cf5..3d866f3 100644
--- a/shell/commands/src/main/java/org/apache/karaf/shell/commands/impl/GrepAction.java
+++ b/shell/commands/src/main/java/org/apache/karaf/shell/commands/impl/GrepAction.java
@@ -33,7 +33,6 @@ import org.apache.karaf.shell.api.action.Option;
import org.apache.karaf.shell.api.action.lifecycle.Service;
import org.apache.karaf.shell.support.ansi.SimpleAnsi;
-
@Command(scope = "shell", name="grep", description="Prints lines matching the given pattern.", detailedDescription="classpath:grep.txt")
@Service
public class GrepAction implements Action {
@@ -136,7 +135,9 @@ public class GrepAction implements Action {
nb++;
}
matcher2.appendTail(sb);
- sb.append(SimpleAnsi.RESET);
+ if(color != ColorOption.never) {
+ sb.append(SimpleAnsi.RESET);
+ }
if (!count && lineNumber) {
lines.add(String.format("%6d ", lineno) + sb.toString());
} else {
http://git-wip-us.apache.org/repos/asf/karaf/blob/6c993776/shell/commands/src/test/java/org/apache/karaf/shell/commands/impl/GrepTest.java
----------------------------------------------------------------------
diff --git a/shell/commands/src/test/java/org/apache/karaf/shell/commands/impl/GrepTest.java b/shell/commands/src/test/java/org/apache/karaf/shell/commands/impl/GrepTest.java
index 3845b9d..877b2b8 100644
--- a/shell/commands/src/test/java/org/apache/karaf/shell/commands/impl/GrepTest.java
+++ b/shell/commands/src/test/java/org/apache/karaf/shell/commands/impl/GrepTest.java
@@ -19,7 +19,9 @@
package org.apache.karaf.shell.commands.impl;
import java.io.ByteArrayInputStream;
+import java.io.ByteArrayOutputStream;
import java.io.InputStream;
+import java.io.PrintStream;
import java.util.Arrays;
import junit.framework.TestCase;
@@ -41,4 +43,23 @@ public class GrepTest extends TestCase {
System.setIn(input);
}
}
+
+ public void testHonorColorNever() throws Exception {
+ InputStream input = System.in;
+ try {
+ ByteArrayInputStream bais = new ByteArrayInputStream("abc".getBytes());
+ System.setIn(bais);
+ ByteArrayOutputStream outContent = new ByteArrayOutputStream();
+ System.setOut(new PrintStream(outContent));
+
+ GrepAction grep = new GrepAction();
+ DefaultActionPreparator preparator = new DefaultActionPreparator();
+ preparator.prepare(grep, null, Arrays.<Object>asList( "--color", "never", "b"));
+ grep.execute();
+ assertEquals("abc", outContent.toString().trim());
+ } finally {
+ System.setIn(input);
+ System.setOut(null);
+ }
+ }
}
[2/2] karaf git commit: [KARAF-4511] This closes #180
Posted by jb...@apache.org.
[KARAF-4511] This closes #180
Project: http://git-wip-us.apache.org/repos/asf/karaf/repo
Commit: http://git-wip-us.apache.org/repos/asf/karaf/commit/329e260f
Tree: http://git-wip-us.apache.org/repos/asf/karaf/tree/329e260f
Diff: http://git-wip-us.apache.org/repos/asf/karaf/diff/329e260f
Branch: refs/heads/master
Commit: 329e260f140f66419a9d7e509345ee8a9d09b032
Parents: 5e310d5 6c99377
Author: Jean-Baptiste Onofr� <jb...@apache.org>
Authored: Tue Aug 23 08:56:49 2016 +0200
Committer: Jean-Baptiste Onofr� <jb...@apache.org>
Committed: Tue Aug 23 08:56:49 2016 +0200
----------------------------------------------------------------------
.../karaf/shell/commands/impl/GrepAction.java | 5 +++--
.../karaf/shell/commands/impl/GrepTest.java | 21 ++++++++++++++++++++
2 files changed, 24 insertions(+), 2 deletions(-)
----------------------------------------------------------------------