Do not try to find available port if set by user
authorBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Sat, 27 Feb 2016 15:58:42 +0000 (16:58 +0100)
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Sat, 27 Feb 2016 15:58:42 +0000 (16:58 +0100)
He/she probably really meant it.

Fixes #1901

commands/server.go

index 94d155ab9c84d63ffb25bc256820d154c7f78a38..53c38099111ba23a3aa24bc0e22109fd6d6d38f6 100644 (file)
@@ -118,6 +118,10 @@ func server(cmd *cobra.Command, args []string) error {
        if err == nil {
                l.Close()
        } else {
+               if flagChanged(serverCmd.Flags(), "port") {
+                       // port set explicitly by user -- he/she probably meant it!
+                       return newSystemErrorF("Port %d already in use", serverPort)
+               }
                jww.ERROR.Println("port", serverPort, "already in use, attempting to use an available port")
                sp, err := helpers.FindAvailablePort()
                if err != nil {