Remove web2 from trunk

[itamar]

Web2 isn't being maintained any more, and confuses people, so we should probably remove it from the repository. To make it easy to find the code if we want to reuse parts of it (as we did for http headers), a tag of trunk from right before the deletion may be helpful.

References to web2 will also probably need to be removed from the packaging code.

[exarkun]

I suppose if we do this, we can also close all of the other open web2 tickets (or redirect them against web, as appropriate).

Anyone want to speak up against doing this? Otherwise I think we should, and sooner rather than later.

The one objection I've heard so far is that removing it will make it hard to get the code. I think we can address this by doing a tag before the delete.

[exarkun]

(In [30703]) Branching to 'remove-web2-4821'

[thijs]

Replying to exarkun:

Anyone want to speak up against doing this? Otherwise I think we should, and sooner rather than later.

Closed #3870.

[exarkun]

(In [30872]) Branching to 'remove-web2-4821-2'

[exarkun]

Okay, I removed it. There was a dependency on web2 in vfs, so I deleted some vfs functionality and disabled the unit test for it. Since vfs isn't part of what we release and no one appears to be working on or using vfs, this doesn't really bother me. The functionality wouldn't be that hard to re-instate based on the old producer/consumer interfaces, if someone really wanted to.

[jesstess]

I can't tell if this is a trick, but here's some feedback in the hopes that it will help web2 die:

• the StreamProducer and IByteStream imports were removed from ftp.py in 30707 but are still used in FTPReadVFS:

  twisted/vfs/adapters/ftp.py:270: undefined name 'StreamProducer'
  twisted/vfs/adapters/ftp.py:270: undefined name 'IByteStream'
  

• There are a couple of lingering references to web2:

  ./twisted/python/zsh/_mktap:"web2:An HTTP/1.1 web server that can serve from a filesystem or application resource."
  ./twisted/python/zsh/_mktap:web2)
  ./twisted/python/zsh/_mktap:'--class=[A class that will be used to serve the root resource.  Must implement twisted.web2.iweb.IResource and take no arguments.]:class:_files' \
  ./twisted/python/zsh/_twistd:"web2:An HTTP/1.1 web server that can serve from a filesystem or application resource."
  ./twisted/python/zsh/_twistd:web2)
  ./twisted/python/zsh/_twistd:'--class=[A class that will be used to serve the root resource.  Must implement twisted.web2.iweb.IResource and take no arguments.]:class:_files' \
  ./twisted/web/_auth/__init__.py:HTTP header-based authentication migrated from web2
  ./twisted/web/test/test_cgi.py:    filters = sys.executable,  # web2's version
  

• If there's not already a ticket for this, the website, including TwistedWebPlan, will need to get updated.

Other than that,  the build looks fine.

[exarkun]

(In [30898]) Remove web2 zsh completion information (manually, untested)

refs #4821

[exarkun]

(In [30927]) Branching to 'remove-web2-4821-3'

[exarkun]

(In [30930]) Similar zsh completion info removal; untested, manual, like the _mktap change

refs #4821

[exarkun]

(In [30931]) Get rid of the thing for web2's version

refs #4821

[exarkun]

Thanks for the review! It is sort of difficult to decide if this ticket is a trick or not, I'm not entirely sure myself. :)

1. Addressed vfs with #4934 (sort of cheating, but effective)
2. Got rid of almost all those references, but I did leave behind the one in the web auth package, since it's still true.
3. Good idea! I'll just go make some edits now, actually.

Here are the  build results

[glyph]

CONGRATURATION, A WINNER IS YOU.

94 files changed, 12 insertions(+), 20256 deletions(-)

(Land.)

[exarkun]

(In [30932]) Merge remove-web2-4821-3

Author: exarkun Reviewer: jesstess, glyph Fixes: #4821

Remove twisted.web2, unreleased, unmaintained, incomplete.