The bug has also been fixed in 2.0.2 of protobuf upstream.
** Changed in: drizzle
Importance: Undecided => High
Assignee: (unassigned) => Monty Taylor (mordred)
Status: New => Fix Released
--
Trailing comma after enum definition list fails with pedantic warning
https://bugs.launch
This bug was fixed in the package protobuf - 2.0.0~b-3ubuntu2
---
protobuf (2.0.0~b-3ubuntu2) intrepid; urgency=low
* src/google/protobuf/compiler/cpp/cpp_enum.cc: added patch to stop
generating trailing comma after enum definition. (Going to be applied
upstream soon: http:/
Daniel: certainly, I'll do that now and provide a new debdiff shortly.
--
Trailing comma after enum definition list fails with pedantic warning
https://bugs.launchpad.net/bugs/264124
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
--
ub
Elliot: do you think you can add a reference to the origin of the patch
to debian/changelog or at least mention the files that were changed, so
it's easier for the next person who merges stuff either from Debian or
upstream?
--
Trailing comma after enum definition list fails with pedantic warning
The failing test case is fixed by applying the patch and then running
the following utility from the GPB source code root directory:
bash generate_descriptor_proto.sh
Cheers!
Jay
Elliot Murphy wrote:
> This has a failing test in the PPA, so I need to update the patch.
> Doesn't look like I can u
Updated debdiff attached.
** Attachment added: "fix-bug-264124.debdiff"
http://launchpadlibrarian.net/17894464/fix-bug-264124.debdiff
--
Trailing comma after enum definition list fails with pedantic warning
https://bugs.launchpad.net/bugs/264124
You received this bug notification because you
This has a failing test in the PPA, so I need to update the patch.
Doesn't look like I can unsubscribe ubuntu-universe-sponsors, so I'll
just try to get an updated debdiff attached quickly.
--
Trailing comma after enum definition list fails with pedantic warning
https://bugs.launchpad.net/bugs/26
Applied the patch from Jay. There is no patch system currently in the
package, but this is a really simple single patch, so I decided not to
add a patch system at this point. Instead I'd like to wait until there
is a second patch that needs applying, and add the patch system then. If
one of the uni
** Attachment added: "cpp_enum.cc.patch"
http://launchpadlibrarian.net/17248271/cpp_enum.cc.patch
** Also affects: drizzle
Importance: Undecided
Status: New
--
Trailing comma after enum definition list fails with pedantic warning
https://bugs.launchpad.net/bugs/264124
You received
