opendev/gerrit
Code Issues Proposed changes

Merge "Adds --description (-d) option to ls-projects"

Browse Source
This commit is contained in:
Shawn Pearce
2011-10-26 17:56:41 -07:00
committed by gerrit code review
parent 1c17c06a77 87cea02528
commit ec19ac8e2b
2 changed files with 24 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
gerrit-sshd/src/main/java/com/google/gerrit/sshd/commands/ListProjects.java
View File

@@ -97,6 +97,9 @@ final class ListProjects extends BaseCommand {
@Option(name = "--type", usage = "type of project")
private FilterType type = FilterType.CODE;
@Option(name = "--description", aliases = {"-d"}, usage = "include description of project in list")
private boolean showDescription;
private String currentTabSeparator = DEFAULT_TAB_SEPARATOR;
@Override
@@ -115,6 +118,10 @@ final class ListProjects extends BaseCommand {
throw new UnloggedFailure(1, "fatal: --tree and --show-branch options are not compatible.");
}
if (showTree && showDescription) {
throw new UnloggedFailure(1, "fatal: --tree and --description options are not compatible.");
}
final PrintWriter stdout = toPrintWriter(out);
final TreeMap<String, TreeNode> treeMap = new TreeMap<String, TreeNode>();
try {
@@ -184,7 +191,15 @@ final class ListProjects extends BaseCommand {
continue;
}
stdout.print(projectName.get() + "\n");
stdout.print(projectName.get());
String desc;
if (showDescription && !(desc = e.getProject().getDescription()).isEmpty()) {
// We still want to list every project as one-liners, hence escaping \n.
stdout.print(" - " + desc.replace("\n", "\\n"));
}
stdout.print("\n");
}
if (showTree && treeMap.size() > 0) {