Merge into trunk : lp844905 : Code : OpenStack Compute (nova)

Code review comment for lp://staging/~johannes.erdfelt/nova/lp844905

Revision history for this message
Johannes Erdfelt (johannes.erdfelt) wrote :

It *does* duplicate the code in webob, but only because webob ignores all parameters except for 'q'. We need the 'version' parameter as well (which is kind of the purpose of the branch).

I should also mention that the Accept parsing code in this branch is based off of the code in Werkzeug. It's BSD licensed and appears to be compatible with the Apache license Openstack uses. I'm thinking I might need to move that code to a separate file to keep the license separate.

« Back to merge proposal