We've accepted patches like this before -- what you can do in this case is to: * upload with the review? flag (to indicate it needs someone to look at it), but no commit? flag (to indicate it's not been tested fully, so that the reviewer can see at a glance that this patch needs some additional attention) * leave a comment stating exactly what you've said in this post
And someone with things set up can apply it and check it, etc.
Basically it's not a problem, so long as we know how much testing the patch has actually received and any known or potential issues.
no subject
We've accepted patches like this before -- what you can do in this case is to:
* upload with the review? flag (to indicate it needs someone to look at it), but no commit? flag (to indicate it's not been tested fully, so that the reviewer can see at a glance that this patch needs some additional attention)
* leave a comment stating exactly what you've said in this post
And someone with things set up can apply it and check it, etc.
Basically it's not a problem, so long as we know how much testing the patch has actually received and any known or potential issues.