fix: roundup_demo should use port assignment

There are three port values:

  urlport (-P) - used in the URL when docker maps real port to external port
  port (-p) - used to set port roundup listens on
  DEFAULT_PORT = 8917 used for -p if not set.

Before the -p argument was not used if -P was not set. So you
can override the default 8917.

Author: rouilj

CHANGES.txt

@@ -133,6 +133,8 @@ Fixed:
   Rouillard)
 - issue2551285 - Remove StructuredText support. reStructuredText is
   still supported. (John Rouillard)
+- Use roundup-demo -p option to set listening port. Was ignored
+  before. (John Rouillard)
 
 Features:
 

roundup/scripts/roundup_demo.py

@@ -170,7 +170,7 @@ def run():
         # install
         demo.install_demo(home, backend,
                           admin.AdminTool().listTemplates()[template]['path'],
-                          use_port=cli_args.urlport or DEFAULT_PORT)
+                          use_port=cli_args.urlport or cli_args.port or DEFAULT_PORT)
     else:
         # make sure that no options are specified that are only useful on initialization.
         if ( cli_args.backend or cli_args.template or