basic-server: make execution more user friendly

Motivation:
- expecting user any key to shutdown makes it hard to run as service
- mutating name make it hard for scripting

Modification:
- Update Main class to expect interrupt
- fixed name for all-in-one jar

Result:
simpler to test integrations

Acked-by: Marina Sahakyan
Target: master

Author: kofemann

basic-server/pom.xml

@@ -41,6 +41,8 @@
                                     <Build-Time>${maven.build.timestamp}</Build-Time>
                                 </manifestEntries>
                             </archive>
+                            <finalName>${project.artifactId}</finalName>
+                            <appendAssemblyId>false</appendAssemblyId>
                         </configuration>
                     </execution>
                 </executions>

basic-server/src/main/java/org/dcache/nfs4j/server/Main.java

@@ -24,10 +24,11 @@ public class Main implements Callable<Void> {
     private boolean withPortmap;
 
     public static void main(String[] args) throws Exception {
-        new CommandLine(new Main()).execute(args);
+        int rc = new CommandLine(new Main()).execute(args);
+        System.exit(rc);
     }
 
-    public Void call() throws IOException {
+    public Void call() throws IOException, InterruptedException {
 
         ExportFile exportFile = null;
         if (exportsFile != null) {
@@ -39,8 +40,8 @@ public Void call() throws IOException {
         }
 
         try (SimpleNfsServer ignored = new SimpleNfsServer(nfsVers, rpcPort, root, exportFile, null)) {
-            //noinspection ResultOfMethodCallIgnored
-            System.in.read(); //any key to shutdown
+            System.out.println("Press Ctrl-C to stop the server...");
+            Thread.currentThread().join();
         }
 
         return null;