opendev/gerrit
Code Issues Proposed changes

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

Browse Source 
Allows listing of projects together with their respective
description.

Line-feeds are escaped to allow ls-project to keep the
"one project per line"-style.

Just like the --branch option, --description not compatible
with the --tree option.

Change-Id: Ia2ec52db80af4fe4343f6a0263d7c02fa84c12cb
This commit is contained in:
Gustaf Lundh
2011-09-01 15:56:59 +02:00
parent f7b65a20d4
commit 87cea02528
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) {