Merge pull request #2120 from mmerickel/fix/pserve-deprecation-messages

Bert JW Regeer

2015-11-13 2cb0419e3ebcfc4c6671d0325b7fb280bc986001

Merge pull request #2120 from mmerickel/fix/pserve-deprecation-messages

improve pserve deprecation messages

 CHANGES.txt

@@ -242,9 +242,10 @@
Deprecations
------------

- The ``pserve`` command's daemonization features have been deprecated. This
  includes the ``[start,stop,restart,status]`` subcommands as well as the
  ``--daemon``, ``--stop-server``, ``--pid-file``, and ``--status`` flags.
- The ``pserve`` command's daemonization features have been deprecated as well
  as ``--monitor-restart``. This includes the ``[start,stop,restart,status]``
  subcommands as well as the ``--daemon``, ``--stop-daemon``, ``--pid-file``,
  and ``--status`` flags.

  Please use a real process manager in the future instead of relying on the
  ``pserve`` to daemonize itself. Many options exist including your Operating
@@ -252,6 +253,7 @@
  solutions like Circus and Supervisor.

  See https://github.com/Pylons/pyramid/pull/1641
  and https://github.com/Pylons/pyramid/pull/2120

- Renamed the ``principal`` argument to ``pyramid.security.remember()`` to
  ``userid`` in order to clarify its intended purpose.

 pyramid/scripts/pserve.py

@@ -69,6 +69,7 @@

    If start/stop/restart is given, then --daemon is implied, and it will
    start (normal operation), stop (--stop-daemon), or do both.
    Note: Daemonization features are deprecated.

    You can also include variable assignments like 'http_port=8080'
    and then use %(http_port)s in your config files.
@@ -100,13 +101,13 @@
            '--daemon',
            dest="daemon",
            action="store_true",
            help="Run in daemon (background) mode")
            help="Run in daemon (background) mode [DEPRECATED]")
    parser.add_option(
        '--pid-file',
        dest='pid_file',
        metavar='FILENAME',
        help=("Save PID to file (default to pyramid.pid if running in "
              "daemon mode)"))
              "daemon mode) [DEPRECATED]"))
    parser.add_option(
        '--log-file',
        dest='log_file',
@@ -127,7 +128,7 @@
        '--monitor-restart',
        dest='monitor_restart',
        action='store_true',
        help="Auto-restart server if it dies")
        help="Auto-restart server if it dies [DEPRECATED]")
    parser.add_option(
        '-b', '--browser',
        dest='browser',
@@ -137,7 +138,8 @@
        '--status',
        action='store_true',
        dest='show_status',
        help="Show the status of the (presumably daemonized) server")
        help=("Show the status of the (presumably daemonized) server "
              "[DEPRECATED]"))
    parser.add_option(
        '-v', '--verbose',
        default=default_verbosity,
@@ -169,7 +171,7 @@
        dest='stop_daemon',
        action='store_true',
        help=('Stop a daemonized server (given a PID file, or default '
              'pyramid.pid file)'))
              'pyramid.pid file) [DEPRECATED]'))

    _scheme_re = re.compile(r'^[a-z][a-z]+:', re.I)

@@ -306,6 +308,17 @@
                raise ValueError(msg)
            writeable_pid_file.close()

        # warn before forking
        if (
                self.options.monitor_restart and
                not os.environ.get(self._monitor_environ_key)
        ):
            self.out('''\
--monitor-restart has been deprecated in Pyramid 1.6. It will be removed
in a future release per Pyramid's deprecation policy. Please consider using
a real process manager for your processes like Systemd, Circus, or Supervisor.
''')

        if (
            getattr(self.options, 'daemon', False) and
            not os.environ.get(self._monitor_environ_key)