[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-11-28 Thread Monty Taylor
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-10-09 Thread Launchpad Bug Tracker
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:/

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-24 Thread Daniel Holbach
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-25 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-02 Thread Jay Pipes
** 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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-10-09 Thread Launchpad Bug Tracker
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:/

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-24 Thread Daniel Holbach
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-25 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-02 Thread Jay Pipes
** 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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-11-28 Thread Monty Taylor
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-10-09 Thread Launchpad Bug Tracker
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:/

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-24 Thread Daniel Holbach
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-25 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-02 Thread Jay Pipes
** 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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-11-28 Thread Monty Taylor
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-10-09 Thread Launchpad Bug Tracker
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:/

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-10-09 Thread Launchpad Bug Tracker
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:/

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-24 Thread Daniel Holbach
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-25 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-02 Thread Jay Pipes
** 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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-11-28 Thread Monty Taylor
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-24 Thread Daniel Holbach
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-25 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-02 Thread Jay Pipes
** 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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-10-09 Thread Launchpad Bug Tracker
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:/

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-24 Thread Daniel Holbach
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-25 Thread Elliot Murphy
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-02 Thread Jay Pipes
** 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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-11-28 Thread Monty Taylor
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

[Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-11-28 Thread Monty Taylor
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

Re: [Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Jay Pipes
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

Re: [Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Jay Pipes
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

Re: [Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Jay Pipes
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

Re: [Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Jay Pipes
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

Re: [Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Jay Pipes
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

Re: [Bug 264124] Re: Trailing comma after enum definition list fails with pedantic warning

2008-09-23 Thread Jay Pipes
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