From e5c14e6d7e24bbf6807a65e56560a13e7bd7eca3 Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Fri, 29 Mar 2013 10:49:37 -0400
Subject: [PATCH] Fixed NPE in FedClient and support --baseFolder in FedClient (issue 219)

---
 src/main/java/com/gitblit/FederationClient.java |   20 +++++++++++++++-----
 1 files changed, 15 insertions(+), 5 deletions(-)

diff --git a/src/main/java/com/gitblit/FederationClient.java b/src/main/java/com/gitblit/FederationClient.java
index 4aeb211..d34aadb 100644
--- a/src/main/java/com/gitblit/FederationClient.java
+++ b/src/main/java/com/gitblit/FederationClient.java
@@ -44,7 +44,16 @@
 			usage(jc, t);
 		}
 
-		IStoredSettings settings = new FileSettings(params.registrationsFile);
+		System.out.println("Gitblit Federation Client v" + Constants.getVersion() + " (" + Constants.getBuildDate() + ")");
+
+		// command-line specified base folder
+		File baseFolder = new File(System.getProperty("user.dir"));
+		if (!StringUtils.isEmpty(params.baseFolder)) {
+			baseFolder = new File(params.baseFolder);
+		}
+
+		File regFile = com.gitblit.utils.FileUtils.resolveParameter(Constants.baseFolder$, baseFolder, params.registrationsFile);
+		IStoredSettings settings = new FileSettings(regFile.getAbsolutePath());
 		List<FederationModel> registrations = new ArrayList<FederationModel>();
 		if (StringUtils.isEmpty(params.url)) {
 			registrations.addAll(FederationUtils.getFederationRegistrations(settings));
@@ -67,8 +76,6 @@
 			System.exit(0);
 		}
 		
-		System.out.println("Gitblit Federation Client v" + Constants.getVersion() + " (" + Constants.getBuildDate() + ")");
-
 		// command-line specified repositories folder
 		if (!StringUtils.isEmpty(params.repositoriesFolder)) {
 			settings.overrideSetting(Keys.git.repositoriesFolder, new File(
@@ -76,7 +83,7 @@
 		}
 
 		// configure the Gitblit singleton for minimal, non-server operation
-		GitBlit.self().configureContext(settings, null, false);
+		GitBlit.self().configureContext(settings, baseFolder, false);
 		FederationPullExecutor executor = new FederationPullExecutor(registrations, params.isDaemon);
 		executor.run();
 		if (!params.isDaemon) {
@@ -106,7 +113,7 @@
 	private static class Params {
 
 		@Parameter(names = { "--registrations" }, description = "Gitblit Federation Registrations File", required = false)
-		public String registrationsFile = "federation.properties";
+		public String registrationsFile = "${baseFolder}/federation.properties";
 
 		@Parameter(names = { "--daemon" }, description = "Runs in daemon mode to schedule and pull repositories", required = false)
 		public boolean isDaemon;
@@ -126,6 +133,9 @@
 		@Parameter(names = { "--frequency" }, description = "Period to wait between pull attempts (requires --daemon)", required = false)
 		public String frequency = "60 mins";
 
+		@Parameter(names = { "--baseFolder" }, description = "Base folder for received data", required = false)
+		public String baseFolder;
+
 		@Parameter(names = { "--repositoriesFolder" }, description = "Destination folder for cloned repositories", required = false)
 		public String repositoriesFolder;
 

--
Gitblit v1.9.1