Ticket #3907 (closed task: fixed)

Opened 14 months ago

Last modified 8 months ago

Remove --static option for twisted.web

Reported by: thijs Owned by: thijs
Priority: normal Milestone:
Component: web Keywords:
Cc: thijs Branch: branches/static-web-3907-2
Author: thijs Launchpad Bug:

Description

There's a --static option in twisted web that does the same --path. It's marked as deprecated:

    def opt_static(self, path):
        """Same as --path, this is deprecated and will be removed in a
        future release."""
        print ("WARNING: --static is deprecated and will be removed in"
               "a future release. Please use --path.")
        self.opt_path(path)
    opt_s = opt_static

Let's get rid of it.

Change History

Changed 14 months ago by glyph

  • owner changed from glyph to thijs

Changed 14 months ago by glyph

  • component changed from core to web

Changed 14 months ago by exarkun

Sounds good.

Changed 14 months ago by thijs

  • branch set to branches/static-web-3907
  • branch_author set to thijs

(In [27115]) Branching to 'static-web-3907'

Changed 8 months ago by thijs

  • branch changed from branches/static-web-3907 to branches/static-web-3907-2

(In [28008]) Branching to 'static-web-3907-2'

Changed 8 months ago by thijs

(In [28009]) Remove --static option, refs #3907

Changed 8 months ago by thijs

(In [28010]) Revert r28009, refs #3907

Changed 8 months ago by thijs

(In [28011]) Coding standard, refs #3907

Changed 8 months ago by thijs

(In [28012]) Remove --static option, refs #3907

Changed 8 months ago by thijs

  • owner thijs deleted
  • keywords review added

Added news file in r28013. Up for review. Sorry about r28009 :-/

Changed 8 months ago by moshez

  • keywords review removed
  • owner set to thijs

Looks good. Please merge.

Changed 8 months ago by thijs

  • status changed from new to closed
  • resolution set to fixed

(In [28020]) Merge static-web-3907-2: Remove '--static' option from twistd web, that served as an alias for the '--path' option.

Author: thijs Reviewer: moshez Fixes: #3907

Note: See TracTickets for help on using tickets.