Compiz

Merge lp://staging/~muktupavels/compiz/protobuf into lp://staging/compiz/0.9.13

  1. protobuf
  2. Merge into 0.9.13
Proposed by Alberts Muktupāvels
Status: Merged
Approved by: Dmitry Shachnev
Approved revision: 4203
Merged at revision: 4205
Proposed branch: lp://staging/~muktupavels/compiz/protobuf
Merge into: lp://staging/compiz/0.9.13
Diff against target: 14 lines (+2/-1)
1 file modified
compizconfig/libcompizconfig/src/compizconfig.proto (+2/-1)
To merge this branch: bzr merge lp://staging/~muktupavels/compiz/protobuf
Reviewer Review Type Date Requested Status
Dmitry Shachnev Approve
Review via email: mp+362826@code.staging.launchpad.net

Commit message

Fix protobuf warning by explicitly specifying proto2 syntax.

Description of the change

[libprotobuf WARNING google/protobuf/compiler/parser.cc:562] No syntax specified for the proto file: compizconfig.proto. Please use 'syntax = "proto2";' or 'syntax = "proto3";' to specify a syntax version. (Defaulted to proto2 syntax.)

To post a comment you must log in.
Revision history for this message
Dmitry Shachnev (mitya57) wrote :

I think double blank lines for separating toplevel messages were useful, but approving anyway.

review: Approve
lp://staging/~muktupavels/compiz/protobuf updated
4203. By Alberts Muktupāvels

Fix "No syntax specified for the proto file" warning by setting syntax to proto2.

Revision history for this message
Alberts Muktupāvels (muktupavels) wrote :

I decided to stick with proto2 syntax. We can upgrade to proto3 later...

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches