Opened 10 years ago

Closed 10 years ago

#3907 task closed fixed (fixed)

Remove --static option for twisted.web

Reported by: Thijs Triemstra Owned by:
Priority: normal Milestone:
Component: web Keywords:
Cc: Thijs Triemstra Branch: branches/static-web-3907-2
branch-diff, diff-cov, branch-cov, buildbot
Author: thijs

Description

There's a --static option in [source:trunk/twisted/web/tap.py 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 (13)

comment:1 Changed 10 years ago by Glyph

Owner: changed from Glyph to Thijs Triemstra

comment:2 Changed 10 years ago by Glyph

Component: coreweb

comment:3 Changed 10 years ago by Jean-Paul Calderone

Sounds good.

comment:4 Changed 10 years ago by Thijs Triemstra

Author: thijs
Branch: branches/static-web-3907

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

comment:5 Changed 10 years ago by Thijs Triemstra

Branch: branches/static-web-3907branches/static-web-3907-2

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

comment:6 Changed 10 years ago by Thijs Triemstra

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

comment:7 Changed 10 years ago by Thijs Triemstra

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

comment:8 Changed 10 years ago by Thijs Triemstra

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

comment:9 Changed 10 years ago by Thijs Triemstra

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

comment:10 Changed 10 years ago by Thijs Triemstra

Keywords: review added
Owner: Thijs Triemstra deleted

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

comment:11 Changed 10 years ago by Moshe Zadka

Keywords: review removed
Owner: set to Thijs Triemstra

Looks good. Please merge.

comment:12 Changed 10 years ago by Thijs Triemstra

Resolution: fixed
Status: newclosed

(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

comment:13 Changed 9 years ago by <automation>

Owner: Thijs Triemstra deleted
Note: See TracTickets for help on using tickets.