[Calypso] How may I help with calypso?

[Jelmer Vernooij]

Hi Petter,

On Sat, Jan 23, 2016 at 11:24:26PM +0100, Petter Reinholdtsen wrote:
> I've volunteered to join the Alioth project, and would be happy to write
> test scripts and merge some of the patches floating around in git
> repositories, but wonder how to best assist with this.
> 
> So far the little response I got on this list was useful (thank you
> Guido), but none of my patches got any replies so far, and I wonder if
> sending patches to the list is the best way to help.
Patches to the list is the best way. Please give us some time to
review. :-)

> 
> From old git commits in unofficial repos, I notice Guido did some
> protocol testing in 2014 caldav-testers.  Is it a goal to pass this test
> suite?  Some of the old changes are trivial, like this:
> 
> commit 760939bd0193a2624d42970a0254b4d91ce3bb67
> Author: Guido G<C3><BC>nther <agx at sigxcpu.org>
> Date:   Sun Mar 30 15:41:42 2014 +0200
> 
>     Return 404 when trying to delete a non existing item
>     
>     Found by caldav-testers CalDAV/delete.xml
> 
> diff --git a/calypso/__init__.py b/calypso/__init__.py
> index 06845ac..f99588a 100644
> --- a/calypso/__init__.py
> +++ b/calypso/__init__.py
> @@ -341,6 +341,10 @@ class CollectionHTTPHandler(server.BaseHTTPRequestHandler):
>                  self.send_header("Content-Type", "text/xml")
>                  self.end_headers()
>                  self.wfile.write(self._answer)
> +            elif not item:
> +                # Item does not exist
> +                self.send_calypso_response(client.NOT_FOUND, 0)
> +                self.end_headers()
>              else:
>                  # No item or ETag precondition not verified, do not delete item
>                  self.send_calypso_response(client.PRECONDITION_FAILED, 0)
> 
> Is there a reason this and simlar patches are not merged into the
> official repo yet, or is it just lack of time from the small group of
> commiters?  I would be happy to extract patches like that from various
> repos and post them to the list, if that would help get a new Debian
> package uploaded into Debian soon.
I think these should be merged, but perhaps there was a reason they
weren't submitted. Guido?

Cheers,

Jelmer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://keithp.com/pipermail/calypso/attachments/20160124/41089b8e/attachment.sig>