Code review comment for lp:~vila/bzr/webdav-in-core
Revision history for this message
| Jelmer Vernooij (jelmer) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
In general, I like the idea of shipping the webdav plugin.
Wouldn't it make sense to merge some (all?) of this code into the HTTP implementation? I don't see a particular reason why WebDAV support couldn't be in the default HTTP implementation. At the very least it will mean people no longer see a mkdir() not implemented error when they try to write over HTTP but instead a "remote server doesn't support MKDIR" error.
That said, I don't think this integration should be a blocker for merging the webdav plugin into core. Let's move it one step at a time. :-)