get repositories for a project automatically

fixes #2334

Author: Vladimir Kotal

opengrok-indexer/src/main/java/org/opengrok/indexer/index/Indexer.java

@@ -98,7 +98,7 @@ public final class Indexer {
     private static String configFilename = null;
     private static int status = 0;
 
-    private static final ArrayList<String> repositories = new ArrayList<>();
+    private static final Set<String> repositories = new HashSet<>();
     private static final HashSet<String> allowedSymlinks = new HashSet<>();
     private static final Set<String> defaultProjects = new TreeSet<>();
     private static final ArrayList<String> zapCache = new ArrayList<>();
@@ -141,7 +141,9 @@ public static void main(String argv[]) {
                 }
                 System.exit(status);
             }
-            
+
+            checkConfiguration();
+
             if (awaitProfiler) {
                 pauseToAwaitProfiler();
             }
@@ -256,8 +258,11 @@ public static void main(String argv[]) {
                 path = path.substring(srcPath.length());
                 if (env.hasProjects()) {
                     // The paths need to correspond to a project.
-                    if (Project.getProject(path) != null) {
+                    Project project;
+                    if ((project = Project.getProject(path)) != null) {
                         subFiles.add(path);
+                        repositories.addAll(env.getProjectRepositoriesMap().get(project).
+                                stream().map(x -> x.getDirectoryNameRelative()).collect(Collectors.toSet()));
                     } else {
                         System.err.println("The path " + path
                                 + " does not correspond to a project");
@@ -292,7 +297,7 @@ public static void main(String argv[]) {
             // Get history first.
             getInstance().prepareIndexer(env, searchRepositories, addProjects,
                     defaultProjects,
-                    listFiles, createDict, subFiles, repositories,
+                    listFiles, createDict, subFiles, new ArrayList(repositories),
                     zapCache, listRepos);
             if (listRepos || !zapCache.isEmpty()) {
                 return;
@@ -499,19 +504,9 @@ public static String[] parseOptions(String[] argv) throws ParseException {
                 cfg.setTagsEnabled(true);
             });
 
-            parser.on("-H", "--history", "=[/path/to/repository]",
-                "Get history for specific repositories (specified as",
-                "absolute path from source root), or ALL repositories",
-                "when none specified.").
-                Do( repo -> {
-                    String repository = (String) repo;
-                    if (repository.equals("")) {
-                        cfg.setHistoryEnabled(true);  // all repositories
-                    } else {
-                        repositories.add((String)repository);  // specific repository
-                    }
-                }
-            );
+            parser.on("-H", "--history", "Enable history.").Do( v -> {
+                cfg.setHistoryEnabled(true);
+            });
 
             parser.on("-I", "--include", "=pattern",
                 "Only files matching this pattern will be examined.",
@@ -645,6 +640,12 @@ public static String[] parseOptions(String[] argv) throws ParseException {
                 LoggerUtil.setBaseConsoleLogLevel(Level.WARNING);
             });
 
+            parser.on("--repository", "=repository",
+                    "Generate history for specific repository specified as relative path to source root. ",
+                    "Can be used multiple times. Assumes history is on.").Do( repo -> {
+                repositories.add((String)repo);
+            });
+
             parser.on("-R /path/to/configuration",
                 "Read configuration from the specified file.").Do( v-> {
                 // Already handled above. This populates usage.
@@ -788,6 +789,13 @@ public static String[] parseOptions(String[] argv) throws ParseException {
         return argv;
     }
 
+    private static void checkConfiguration() {
+        if (repositories.size() > 0 && !cfg.isHistoryEnabled()) {
+            System.out.println("Repositories were specified however history is off");
+            System.exit(1);
+        }
+    }
+
     private static void die(String message) {
         System.err.println("ERROR: " + message);
         System.exit(1);