[CRIU] [PATCH] rpc: change some "required" fields to "optional" and rename rpc.proto to criu-rpc.proto

Pavel Emelyanov xemul at parallels.com
Sun Sep 15 16:44:44 EDT 2013


On 09/16/2013 12:40 AM, Ruslan Kuprieiev wrote:
> On 09/16/2013 12:30 AM, Pavel Emelyanov wrote:
>> On 09/14/2013 09:43 PM, Ruslan Kuprieiev wrote:
>>> Hi!
>>>
>>> The most of fields are to be changed to "optional", because they are
>>> optional in fact.
>>> Also, rename rpc.proto to criu-rpc.proto, so user will not have to mess
>>> with renaming.
>>>
>>> Signed-off-by: Ruslan Kuprieiev <kupruser at gmail.com>
>>>
>>
>>> --- /dev/null
>>> +++ b/protobuf/criu-rpc.proto
>> There's no point in this rename. If 3rd party user will pull the
>> file into its dir he may rename it himself, otherwise the criu/protobuf/rpc.proto
>> name is just fine.
> And how about changing some fields to optional?
> .
> 

This is OK


More information about the CRIU mailing list