[protobuf] Issue 241 in protobuf: Java Tutorial / Generated code doc lacks reference to ExtensionRegistry

2010-11-26 Thread protobuf
Status: New Owner: ken...@google.com Labels: Type-Defect Priority-Medium New issue 241 by palotai@gmail.com: Java Tutorial / Generated code doc lacks reference to ExtensionRegistry http://code.google.com/p/protobuf/issues/detail?id=241 I first met ExtensionRegistry in an exception

[protobuf] Issue 242 in protobuf: Can no longer install protobuf on pypy

2010-11-26 Thread protobuf
Status: New Owner: ken...@google.com Labels: Type-Defect Priority-Medium New issue 242 by ivan.ko...@gmail.com: Can no longer install protobuf on pypy http://code.google.com/p/protobuf/issues/detail?id=242 What steps will reproduce the problem? 1. cd protobuf/python 2. ~/pypy-1.4/bin/pypy

[protobuf] Re: Issue 242 in protobuf: Can no longer install protobuf on pypy

2010-11-26 Thread protobuf
Comment #1 on issue 242 by ivan.ko...@gmail.com: Can no longer install protobuf on pypy http://code.google.com/p/protobuf/issues/detail?id=242 If I comment out 'ext_modules = ...' in setup.py and install, it works, and all of protobuf's unit tests pass. -- You receiv

[protobuf] Re: Issue 240 in protobuf: windows share handling ( \\server\share) bug in protoc compiler

2010-11-26 Thread protobuf
Updates: Status: WorkingAsIntended Comment #1 on issue 240 by temporal: windows share handling ( \\server\share) bug in protoc compiler http://code.google.com/p/protobuf/issues/detail?id=240 Import paths should always be relative paths. Use -I (aka --proto_path) to specify all

[protobuf] Re: Issue 242 in protobuf: Can no longer install protobuf on pypy

2010-11-28 Thread protobuf
Comment #2 on issue 242 by gpsmith: Can no longer install protobuf on pypy http://code.google.com/p/protobuf/issues/detail?id=242 Yes the extension module is intended to be optional. Conditionally checking if we should compile the C extension in the setup.py file before passing ext_modules

[protobuf] Re: Issue 240 in protobuf: windows share handling ( \\server\share) bug in protoc compiler

2010-11-29 Thread protobuf
Comment #2 on issue 240 by david.vo...@gmail.com: windows share handling ( \\server\share) bug in protoc compiler http://code.google.com/p/protobuf/issues/detail?id=240 I think You are partly right. We should use relative paths, that's true. However, windows shares do not work in ne

[protobuf] Re: Issue 240 in protobuf: windows share handling ( \\server\share) bug in protoc compiler

2010-11-29 Thread protobuf
Updates: Status: Accepted Comment #3 on issue 240 by ken...@google.com: windows share handling ( \\server\share) bug in protoc compiler http://code.google.com/p/protobuf/issues/detail?id=240 I see. I suppose we should avoid converting two leading \'s to /'s. -- You rec

[protobuf] Re: Issue 242 in protobuf: Can no longer install protobuf on pypy

2010-11-29 Thread protobuf
Updates: Owner: liuj...@google.com Comment #3 on issue 242 by ken...@google.com: Can no longer install protobuf on pypy http://code.google.com/p/protobuf/issues/detail?id=242 Technically this is a bug report about code that has not yet made it into a release. Code in SVN is

[protobuf] Issue 243 in protobuf: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream

2010-11-29 Thread protobuf
Status: New Owner: ken...@google.com Labels: Type-Defect Priority-Medium New issue 243 by john.carrino: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream http://code.google.com/p/protobuf/issues/detail?id=243 CodedInputStream defaults to 64MB size limit

[protobuf] Re: Issue 243 in protobuf: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream

2010-11-29 Thread protobuf
Comment #1 on issue 243 by john.carrino: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream http://code.google.com/p/protobuf/issues/detail?id=243 I'd like to point out I am referring to the static CodedInputStream.readRawVarint32(InputStream) and no

[protobuf] Re: Issue 243 in protobuf: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream

2010-11-29 Thread protobuf
Updates: Status: Accepted Comment #2 on issue 243 by temporal: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream http://code.google.com/p/protobuf/issues/detail?id=243 You can destroy the CodedInputStream at the end of each message, and construct a

[protobuf] Re: Issue 241 in protobuf: Java Tutorial / Generated code doc lacks reference to ExtensionRegistry

2010-11-29 Thread protobuf
Updates: Status: Accepted Comment #1 on issue 241 by temporal: Java Tutorial / Generated code doc lacks reference to ExtensionRegistry http://code.google.com/p/protobuf/issues/detail?id=241 (No comment was entered for this change.) -- You received this message because you are

[protobuf] Re: Issue 243 in protobuf: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream

2010-11-29 Thread protobuf
Comment #3 on issue 243 by john.carrino: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream http://code.google.com/p/protobuf/issues/detail?id=243 The main issue is that there is no way to get the size of the message because there is no way to do a

[protobuf] Re: Issue 243 in protobuf: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream

2010-11-30 Thread protobuf
Updates: Status: New Comment #4 on issue 243 by ken...@google.com: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream http://code.google.com/p/protobuf/issues/detail?id=243 Woops, for some reason I thought you were talking about C++. Not sure why

[protobuf] Re: Issue 243 in protobuf: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream

2010-11-30 Thread protobuf
Comment #5 on issue 243 by john.carrino: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream http://code.google.com/p/protobuf/issues/detail?id=243 If the message is over 64MB, the CodedInputStream it creates under the covers will barf during parsing. So I

[protobuf] Re: Issue 243 in protobuf: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream

2010-11-30 Thread protobuf
Updates: Status: Accepted Comment #6 on issue 243 by ken...@google.com: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream http://code.google.com/p/protobuf/issues/detail?id=243 Ah, I see. You are talking about individual messages over 64MB... for

[protobuf] Re: Issue 243 in protobuf: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream

2010-11-30 Thread protobuf
Comment #7 on issue 243 by john.carrino: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream http://code.google.com/p/protobuf/issues/detail?id=243 Yeah, this is due to message A having a lot of repeated element B. We could pull B out of A and into the top

[protobuf] Re: Issue 83 in protobuf: protobuf does not compile cleanly in 64-bit mode in Visual Studio 2008

2010-11-30 Thread protobuf
Comment #18 on issue 83 by ken...@google.com: protobuf does not compile cleanly in 64-bit mode in Visual Studio 2008 http://code.google.com/p/protobuf/issues/detail?id=83 Well that's easy enough. Revision 351. Leaving the bug open because I don't think this is the only thing peop

[protobuf] Issue 244 in protobuf: Autogenerated Java code producing error: "Cannot invoke buildParsed() on the primitive type boolean"

2010-11-30 Thread protobuf
Status: New Owner: ken...@google.com Labels: Type-Defect Priority-Medium New issue 244 by kentmchenry: Autogenerated Java code producing error: "Cannot invoke buildParsed() on the primitive type boolean" http://code.google.com/p/protobuf/issues/detail?id=244 What steps will rep

[protobuf] Re: Issue 244 in protobuf: Autogenerated Java code producing error: "Cannot invoke buildParsed() on the primitive type boolean"

2010-11-30 Thread protobuf
Updates: Status: WontFix Comment #1 on issue 244 by ken...@google.com: Autogenerated Java code producing error: "Cannot invoke buildParsed() on the primitive type boolean" http://code.google.com/p/protobuf/issues/detail?id=244 Your protoc version does not match the

[protobuf] Re: Issue 243 in protobuf: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream

2010-12-01 Thread protobuf
Comment #8 on issue 243 by ev...@mit.edu: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream http://code.google.com/p/protobuf/issues/detail?id=243 (I'm not a maintainer, but ...) This is recommended by the documentation: http://code.google.com

[protobuf] Re: Issue 244 in protobuf: Autogenerated Java code producing error: "Cannot invoke buildParsed() on the primitive type boolean"

2010-12-01 Thread protobuf
Comment #2 on issue 244 by kentmchenry: Autogenerated Java code producing error: "Cannot invoke buildParsed() on the primitive type boolean" http://code.google.com/p/protobuf/issues/detail?id=244 @kenton Using "protoc --version" I realized that I was using the 2.2.0 p

[protobuf] Re: Issue 243 in protobuf: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream

2010-12-01 Thread protobuf
Comment #9 on issue 243 by john.carrino: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream http://code.google.com/p/protobuf/issues/detail?id=243 Cool, I am down with writing to a stream with a bunch of messages in it. A useful tool to do so would be the

[protobuf] Re: Issue 243 in protobuf: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream

2010-12-01 Thread protobuf
Comment #10 on issue 243 by ken...@google.com: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream http://code.google.com/p/protobuf/issues/detail?id=243 How about you define a "header" message which you read first, like: message Header { //

[protobuf] Re: Issue 243 in protobuf: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream

2010-12-01 Thread protobuf
Comment #11 on issue 243 by john.carrino: Improve mergeDelimitedFrom in the java Impl to setSizeLimit on the CodedInputStream http://code.google.com/p/protobuf/issues/detail?id=243 That's seems like a good way to go. It's protos all the way down. -- You received this message b

[protobuf] Issue 245 in protobuf: "index" package conflicts with index argument name

2010-12-01 Thread protobuf
Status: New Owner: ken...@google.com Labels: Type-Defect Priority-Medium New issue 245 by yanghatespam: "index" package conflicts with index argument name http://code.google.com/p/protobuf/issues/detail?id=245 If you have: option java_package = "index"; that wi

[protobuf] Re: Issue 245 in protobuf: "index" package conflicts with index argument name

2010-12-01 Thread protobuf
Comment #1 on issue 245 by yanghatespam: "index" package conflicts with index argument name http://code.google.com/p/protobuf/issues/detail?id=245 This is with svn r93 -- You received this message because you are subscribed to the Google Groups "Protocol Buffers" gr

[protobuf] Issue 246 in protobuf: Lots of "warning: [serial] serializable class ... has no definition of serialVersionUID"

2010-12-01 Thread protobuf
Status: New Owner: ken...@google.com Labels: Type-Defect Priority-Medium New issue 246 by yanghatespam: Lots of "warning: [serial] serializable class ... has no definition of serialVersionUID" http://code.google.com/p/protobuf/issues/detail?id=246 The generated classes

[protobuf] Re: Issue 246 in protobuf: Lots of "warning: [serial] serializable class ... has no definition of serialVersionUID"

2010-12-01 Thread protobuf
Comment #1 on issue 246 by yanghatespam: Lots of "warning: [serial] serializable class ... has no definition of serialVersionUID" http://code.google.com/p/protobuf/issues/detail?id=246 This is also against svn r92 (no problems with 2.3.0). -- You received this message becau

[protobuf] Re: Issue 245 in protobuf: "index" package conflicts with index argument name

2010-12-01 Thread protobuf
Updates: Status: WontFix Comment #2 on issue 245 by ken...@google.com: "index" package conflicts with index argument name http://code.google.com/p/protobuf/issues/detail?id=245 The Java package naming convention says that the first component of a package name shoul

[protobuf] Re: Issue 246 in protobuf: Lots of "warning: [serial] serializable class ... has no definition of serialVersionUID"

2010-12-01 Thread protobuf
Updates: Status: Accepted Comment #2 on issue 246 by ken...@google.com: Lots of "warning: [serial] serializable class ... has no definition of serialVersionUID" http://code.google.com/p/protobuf/issues/detail?id=246 I think you have the wrong revision number there. r92 is

[protobuf] Re: Issue 245 in protobuf: "index" package conflicts with index argument name

2010-12-01 Thread protobuf
Comment #3 on issue 245 by yanghatespam: "index" package conflicts with index argument name http://code.google.com/p/protobuf/issues/detail?id=245 We've already changed our code (you gave us a good impetus for finally renaming that poorly named package), but it *is* lega

[protobuf] Re: Issue 246 in protobuf: Lots of "warning: [serial] serializable class ... has no definition of serialVersionUID"

2010-12-01 Thread protobuf
Comment #3 on issue 246 by yanghatespam: Lots of "warning: [serial] serializable class ... has no definition of serialVersionUID" http://code.google.com/p/protobuf/issues/detail?id=246 You're right, I meant r352. -- You received this message because you are subscribed to th

[protobuf] Re: Issue 245 in protobuf: "index" package conflicts with index argument name

2010-12-01 Thread protobuf
Comment #4 on issue 245 by ken...@google.com: "index" package conflicts with index argument name http://code.google.com/p/protobuf/issues/detail?id=245 "index" is just one of many names that will cause this problem. And package names are only one kind of problem. I

[protobuf] Re: Issue 245 in protobuf: "index" package conflicts with index argument name

2010-12-01 Thread protobuf
Comment #5 on issue 245 by yanghatespam: "index" package conflicts with index argument name http://code.google.com/p/protobuf/issues/detail?id=245 I didn't mean to document "index" in particular, I meant just a quick note in the Java docs that proper TLD packag

[protobuf] 9 issues changed in protobuf

2010-12-03 Thread protobuf
Updates: Status: Fixed Labels: FixedIn-2.4.0 Comment by liuj...@google.com: Fixed in r353 Affected issues: issue 166: by_symbol_.insert(iter, make_pair(name, value)); fails with Sun Studio 12 http://code.google.com/p/protobuf/issues/detail?id=166 issue 167: Python

[protobuf] 6 issues changed in protobuf

2010-12-05 Thread protobuf
Updates: Status: Fixed Labels: FixedIn-2.4.0 Comment by liuj...@google.com: Fixed in r354. Affected issues: issue 200: Bug on CodedOutputStream.writeEnum http://code.google.com/p/protobuf/issues/detail?id=200 issue 202: RepeatedField::MoveArray generates "wa

[protobuf] Re: Issue 211 in protobuf: common.cc calls "abort" on FATAL messages

2010-12-06 Thread protobuf
Comment #3 on issue 211 by goo...@thenuthand.com: common.cc calls "abort" on FATAL messages http://code.google.com/p/protobuf/issues/detail?id=211 Though better than abort, "throw -1" is not an extremely helpful exception: it won'y be possible to locate the s

[protobuf] Re: Issue 211 in protobuf: common.cc calls "abort" on FATAL messages

2010-12-06 Thread protobuf
Updates: Status: Accepted Owner: liuj...@google.com Labels: -FixedIn-2.4.0 Comment #4 on issue 211 by temporal: common.cc calls "abort" on FATAL messages http://code.google.com/p/protobuf/issues/detail?id=211 Yes, you're right. We should define a

[protobuf] Re: Issue 211 in protobuf: common.cc calls "abort" on FATAL messages

2010-12-06 Thread protobuf
Comment #5 on issue 211 by goo...@thenuthand.com: common.cc calls "abort" on FATAL messages http://code.google.com/p/protobuf/issues/detail?id=211 Putting the parameters of the log statement would also be great, or at least the error message (message_) that was written. -- Yo

[protobuf] Re: Issue 211 in protobuf: common.cc calls "abort" on FATAL messages

2010-12-06 Thread protobuf
Comment #6 on issue 211 by liuj...@google.com: common.cc calls "abort" on FATAL messages http://code.google.com/p/protobuf/issues/detail?id=211 OK, will do. -- You received this message because you are subscribed to the Google Groups "Protocol Buffers" group. To post

[protobuf] Re: Issue 242 in protobuf: Can no longer install protobuf on pypy

2010-12-06 Thread protobuf
Updates: Status: Accepted Comment #4 on issue 242 by liuj...@google.com: Can no longer install protobuf on pypy http://code.google.com/p/protobuf/issues/detail?id=242 (No comment was entered for this change.) -- You received this message because you are subscribed to the Google

[protobuf] Re: Issue 196 in protobuf: Python: Ascii output is not assured to be in utf-8

2010-12-06 Thread protobuf
Updates: Status: Fixed Labels: FixedIn-2.4.0 Comment #2 on issue 196 by liuj...@google.com: Python: Ascii output is not assured to be in utf-8 http://code.google.com/p/protobuf/issues/detail?id=196 Now the text_format.PrintMessage has a parameter "as_utf", which

[protobuf] Re: Issue 225 in protobuf: Invalid file descriptor error only with VC++ 2005-2008 (not 2010)

2010-12-06 Thread protobuf
Comment #2 on issue 225 by liuj...@google.com: Invalid file descriptor error only with VC++ 2005-2008 (not 2010) http://code.google.com/p/protobuf/issues/detail?id=225 Hmm, do you mean the question marks or the quotes? -- You received this message because you are subscribed to the Google

[protobuf] Re: Issue 225 in protobuf: Invalid file descriptor error only with VC++ 2005-2008 (not 2010)

2010-12-06 Thread protobuf
Comment #3 on issue 225 by flio...@free.fr: Invalid file descriptor error only with VC++ 2005-2008 (not 2010) http://code.google.com/p/protobuf/issues/detail?id=225 Oups! It's the "question mark" of course. The sign of the Sphinx if you prefer. ;) -- You received this mess

[protobuf] Issue 247 in protobuf: Ability to redirect file output to stdout

2010-12-06 Thread protobuf
Status: New Owner: ken...@google.com Labels: Type-Defect Priority-Medium New issue 247 by compuwarescc: Ability to redirect file output to stdout http://code.google.com/p/protobuf/issues/detail?id=247 Unable to output source to stdout. Being able to receive output to stdout rather than just as

[protobuf] Re: Issue 247 in protobuf: Ability to redirect file output to stdout

2010-12-06 Thread protobuf
Comment #1 on issue 247 by moofish32: Ability to redirect file output to stdout http://code.google.com/p/protobuf/issues/detail?id=247 I think this would be a very beneficial enhancement for the same reason. Solves any concerns over permissions in the unix environment. -- You received

[protobuf] Issue 248 in protobuf: protobuf will not compile without thread library

2010-12-06 Thread protobuf
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 248 by ryan.drake.08: protobuf will not compile without thread library http://code.google.com/p/protobuf/issues/detail?id=248 What steps will reproduce the problem? 1. Compile the library without WIN32 or HAVE_PTHREAD 2

[protobuf] Re: Issue 248 in protobuf: protobuf will not compile without thread library

2010-12-06 Thread protobuf
Comment #1 on issue 248 by ryan.drake.08: protobuf will not compile without thread library http://code.google.com/p/protobuf/issues/detail?id=248 Note, these stubs are enough to compile protobuf-lite. Not sure if more are needed for the full library. -- You received this message because

[protobuf] 4 issues changed in protobuf

2010-12-06 Thread protobuf
Updates: Status: Fixed Labels: FixedIn-2.4.0 Comment by liuj...@google.com: Fixed in r356 Affected issues: issue 165: can not link for mips architecture http://code.google.com/p/protobuf/issues/detail?id=165 issue 211: common.cc calls "abort" on FATAL message

[protobuf] Re: Issue 211 in protobuf: common.cc calls "abort" on FATAL messages

2010-12-06 Thread protobuf
Comment #8 on issue 211 by goo...@thenuthand.com: common.cc calls "abort" on FATAL messages http://code.google.com/p/protobuf/issues/detail?id=211 Looks good. Thank you. -- You received this message because you are subscribed to the Google Groups "Protocol Buffers" gr

[protobuf] Re: Issue 248 in protobuf: protobuf will not compile without thread library

2010-12-07 Thread protobuf
Updates: Status: Accepted Owner: ken...@google.com Comment #2 on issue 248 by ken...@google.com: protobuf will not compile without thread library http://code.google.com/p/protobuf/issues/detail?id=248 Hmm. I don't think we should automatically fall back to thread-ho

[protobuf] Re: Issue 196 in protobuf: Python: Ascii output is not assured to be in utf-8

2010-12-07 Thread protobuf
Updates: Status: Accepted Labels: -FixedIn-2.4.0 Comment #3 on issue 196 by ken...@google.com: Python: Ascii output is not assured to be in utf-8 http://code.google.com/p/protobuf/issues/detail?id=196 Jisi, I'm not convinced that this is fixed. The as_utf parameter s

[protobuf] Re: Issue 247 in protobuf: Ability to redirect file output to stdout

2010-12-07 Thread protobuf
Comment #2 on issue 247 by ken...@google.com: Ability to redirect file output to stdout http://code.google.com/p/protobuf/issues/detail?id=247 What happens if multiple files are generated? For example, if you have "option java_multiple_files = true;" in your .proto file.

[protobuf] 3 issues changed in protobuf

2010-12-07 Thread protobuf
Updates: Status: Fixed Labels: FixedIn-2.4.0 Comment by liuj...@google.com: Fixed in r358 Affected issues: issue 223: Python Package Doesn't Contain Compiler Plugin Module http://code.google.com/p/protobuf/issues/detail?id=223 issue 224: Protobuf installation err

[protobuf] Re: Issue 179 in protobuf: Visual C++ error C1091 when compiling protoc generated code with over 64k descriptor

2010-12-07 Thread protobuf
Updates: Status: New Comment #2 on issue 179 by ken...@google.com: Visual C++ error C1091 when compiling protoc generated code with over 64k descriptor http://code.google.com/p/protobuf/issues/detail?id=179 This proves complicated to fix, because we have lots of code that passes

[protobuf] Re: Issue 210 in protobuf: Java code should detect incompatible runtime library version

2010-12-07 Thread protobuf
Updates: Cc: liuj...@google.com Comment #13 on issue 210 by liuj...@google.com: Java code should detect incompatible runtime library version http://code.google.com/p/protobuf/issues/detail?id=210 Hmm, not very clear about the maven version compatibility... So suppose you have a

[protobuf] Re: Issue 210 in protobuf: Java code should detect incompatible runtime library version

2010-12-07 Thread protobuf
Comment #14 on issue 210 by aantono: Java code should detect incompatible runtime library version http://code.google.com/p/protobuf/issues/detail?id=210 It would if you specify, what Maven calls, a range -> [1.0,2.0), which would basically read that anything inclusive between 1.0 and

[protobuf] Re: Issue 196 in protobuf: Python: Ascii output is not assured to be in utf-8

2010-12-08 Thread protobuf
Comment #4 on issue 196 by liuj...@google.com: Python: Ascii output is not assured to be in utf-8 http://code.google.com/p/protobuf/issues/detail?id=196 I see, this was actually fixed by another CL that always encodes unicode string field to utf8 to fix this bug in our internal branch(s

[protobuf] Re: Issue 179 in protobuf: Visual C++ error C1091 when compiling protoc generated code with over 64k descriptor

2010-12-08 Thread protobuf
Comment #3 on issue 179 by iiro.hie...@gmail.com: Visual C++ error C1091 when compiling protoc generated code with over 64k descriptor http://code.google.com/p/protobuf/issues/detail?id=179 OK, thanks! It might be a good idea to write down this limitation into documentation. -- You

[protobuf] Re: Issue 247 in protobuf: Ability to redirect file output to stdout

2010-12-08 Thread protobuf
Comment #3 on issue 247 by compuwarescc: Ability to redirect file output to stdout http://code.google.com/p/protobuf/issues/detail?id=247 To be honest I had not considered that scenario. Perhaps breaking this into two feature requests would be more appropriate... 1) Command line flag

[protobuf] Re: Issue 248 in protobuf: protobuf will not compile without thread library

2010-12-08 Thread protobuf
Comment #3 on issue 248 by liuj...@google.com: protobuf will not compile without thread library http://code.google.com/p/protobuf/issues/detail?id=248 Hmm, what OS/platform are you using? Common platforms should define HAVE_PTHREAD.. Maybe it's the acx_pthread.m4 problem.. -

[protobuf] Issue 249 in protobuf: Default values are not assumed to be filled in.

2010-12-08 Thread protobuf
Status: New Owner: ken...@google.com Labels: Type-Defect Priority-Medium New issue 249 by osfan6313: Default values are not assumed to be filled in. http://code.google.com/p/protobuf/issues/detail?id=249 What steps will reproduce the problem? 1. Create a protocol buffer file with a required

[protobuf] Re: Issue 249 in protobuf: Default values are not assumed to be filled in.

2010-12-08 Thread protobuf
Updates: Status: WorkingAsIntended Owner: jas...@google.com Comment #1 on issue 249 by jas...@google.com: Default values are not assumed to be filled in. http://code.google.com/p/protobuf/issues/detail?id=249 This is a common point of confusion. required means a field must be

[protobuf] Re: Issue 248 in protobuf: protobuf will not compile without thread library

2010-12-08 Thread protobuf
Comment #4 on issue 248 by ryan.drake.08: protobuf will not compile without thread library http://code.google.com/p/protobuf/issues/detail?id=248 Brew mobile platform, with the RVCT4.0 ARM compiler. I think HAVE_PTHREAD should be around all code blocks that require pthread.h, but I agree

[protobuf] Re: Issue 210 in protobuf: Java code should detect incompatible runtime library version

2010-12-08 Thread protobuf
Comment #15 on issue 210 by ken...@google.com: Java code should detect incompatible runtime library version http://code.google.com/p/protobuf/issues/detail?id=210 Urgh, this is problematic. When people declare a dependency on protocol buffers, I want them to be able to use the version

[protobuf] Re: Issue 248 in protobuf: protobuf will not compile without thread library

2010-12-08 Thread protobuf
Comment #5 on issue 248 by poftwaresatent: protobuf will not compile without thread library http://code.google.com/p/protobuf/issues/detail?id=248 For what it's worth, I'm running into this on OS X 10.6.5 with up-to-date macports and protobuf svn rev 358. The configure script do

[protobuf] Re: Issue 248 in protobuf: protobuf will not compile without thread library

2010-12-08 Thread protobuf
Comment #6 on issue 248 by poftwaresatent: protobuf will not compile without thread library http://code.google.com/p/protobuf/issues/detail?id=248 Looks like r353 broke the m4 code that checks for pthread/sharedlib coexistance on OS X. Somehow m4/acx_pthread.m4 ends up injecting -Wl,-z,foo

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-09 Thread protobuf
Updates: Status: Accepted Owner: liuj...@google.com Labels: -FixedIn-2.4.0 Comment #13 on issue 188 by liuj...@google.com: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/iss

[protobuf] Re: Issue 248 in protobuf: protobuf will not compile without thread library

2010-12-09 Thread protobuf
Comment #7 on issue 248 by liuj...@google.com: protobuf will not compile without thread library http://code.google.com/p/protobuf/issues/detail?id=248 Yes, the r353 fix isn't correct. Rolled it back at r360. -- You received this message because you are subscribed to the Google G

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-09 Thread protobuf
Updates: Status: NeedPatchFromUser Comment #14 on issue 188 by liuj...@google.com: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 Hi, xarthisius.kk, The line above: A

[protobuf] Re: Issue 249 in protobuf: Default values are not assumed to be filled in.

2010-12-09 Thread protobuf
Comment #2 on issue 249 by osfan6313: Default values are not assumed to be filled in. http://code.google.com/p/protobuf/issues/detail?id=249 Ah. Well I haven't seen any information which ever said that; the documentation is rather lacking in that regard. It would be nice to have th

[protobuf] Re: Issue 249 in protobuf: Default values are not assumed to be filled in.

2010-12-09 Thread protobuf
Comment #3 on issue 249 by mdpo...@troilus.org: Default values are not assumed to be filled in. http://code.google.com/p/protobuf/issues/detail?id=249 The behavior and documentation seem clear to me: "required" means the user must set it to something (i.e. it is required).

[protobuf] Re: Issue 249 in protobuf: Default values are not assumed to be filled in.

2010-12-09 Thread protobuf
Comment #4 on issue 249 by osfan6313: Default values are not assumed to be filled in. http://code.google.com/p/protobuf/issues/detail?id=249 We're using protocol buffers to store user settings, so we want some fields to be required(for data validation purposes). However, when we

[protobuf] Re: Issue 249 in protobuf: Default values are not assumed to be filled in.

2010-12-09 Thread protobuf
Comment #5 on issue 249 by jas...@google.com: Default values are not assumed to be filled in. http://code.google.com/p/protobuf/issues/detail?id=249 It sounds like there is still some confusion over the meaning of "set." It is always legal to access a field from a protobuf, whe

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-10 Thread protobuf
Comment #15 on issue 188 by monty.taylor: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 FWIW, we're seeing this in building Drizzle for Ubuntu Natty (and only there

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-11 Thread protobuf
Comment #16 on issue 188 by liuj...@google.com: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 Hmm, could you please share how you fixed this for Drizzle? explicitly adding

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-11 Thread protobuf
Comment #17 on issue 188 by xarthisius.kk: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 How about checking for the situation I've described in c#6 ? Attachments:

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-11 Thread protobuf
Comment #18 on issue 188 by xarthisius.kk: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 Wrong patch... Attachments: protobuf-2.3.0-acx_pthread.patch 2.5 KB -- Yo

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-11 Thread protobuf
Comment #19 on issue 188 by monty.taylor: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 The following patch (which I believe came from Gentoo - http://bugs.gentoo.org/show_

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-11 Thread protobuf
Comment #20 on issue 188 by liuj...@google.com: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 Thanks Xarthisius and Monty. Xarthisius, your patch seems to be examining i

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-11 Thread protobuf
Comment #21 on issue 188 by xarthisius.kk: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 I think if the -nostdlib is actually set in the CFLAGS before running the acx_pthr

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-11 Thread protobuf
Comment #22 on issue 188 by liuj...@google.com: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 I see, so how did you set the nostdlib flag? Is this issue due to the nostdl

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-11 Thread protobuf
Comment #23 on issue 188 by xarthisius.kk: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 I see, so how did you set the nostdlib flag? I'm confused. -nostdlib is used by

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-11 Thread protobuf
Comment #24 on issue 188 by liuj...@google.com: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 So I assumed the --as-needed in LDFLAGS is used in the acx_pthread.m4, if not,

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-11 Thread protobuf
Comment #25 on issue 188 by liuj...@google.com: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 Ah.. Ok, so your fix looks good. Thanks! -- You received this message becau

[protobuf] Re: Issue 196 in protobuf: Python: Ascii output is not assured to be in utf-8

2010-12-14 Thread protobuf
Updates: Status: Fixed Labels: FixedIn-2.4.0 Comment #5 on issue 196 by ken...@google.com: Python: Ascii output is not assured to be in utf-8 http://code.google.com/p/protobuf/issues/detail?id=196 Cool, it sounds like it is indeed fixed. -- You received this message because

[protobuf] Re: Issue 247 in protobuf: Ability to redirect file output to stdout

2010-12-14 Thread protobuf
Updates: Status: Accepted Labels: -Type-Defect Type-Enhancement Comment #4 on issue 247 by ken...@google.com: Ability to redirect file output to stdout http://code.google.com/p/protobuf/issues/detail?id=247 We do already have the ability to output to a jar file. I suppose

[protobuf] Re: Issue 248 in protobuf: protobuf will not compile without thread library

2010-12-14 Thread protobuf
Comment #8 on issue 248 by ken...@google.com: protobuf will not compile without thread library http://code.google.com/p/protobuf/issues/detail?id=248 To be clear, the warning printed by configure (for v2.3.0 and earlier) is bogus. It's a side effect of a deeper bug in the m4 file

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-20 Thread protobuf
Comment #26 on issue 188 by liuj...@google.com: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 Hi, Kacper, Before we can accept this patch, we need you to sign the C

[protobuf] Re: Issue 225 in protobuf: Invalid file descriptor error only with VC++ 2005-2008 (not 2010)

2010-12-20 Thread protobuf
Updates: Status: Fixed Labels: FixedIn-2.4.0 Comment #4 on issue 225 by liuj...@google.com: Invalid file descriptor error only with VC++ 2005-2008 (not 2010) http://code.google.com/p/protobuf/issues/detail?id=225 Fixed in r362 -- You received this message because you are

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-21 Thread protobuf
Comment #27 on issue 188 by xarthisius.kk: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 Sorry but it's getting ridiculous. That electronic form g

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-21 Thread protobuf
Comment #28 on issue 188 by lukejr2005: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 Just BSD license the patch. Never heard of a company that refuses to use BSD lic

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-21 Thread protobuf
Comment #29 on issue 188 by temporal: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 Kacper, Believe me, if we didn't have to ask you to do this, we wouldn't. We

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-21 Thread protobuf
Comment #30 on issue 188 by ken...@google.com: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 (Arg, posted from wrong account again; I always do that when I'm at h

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-22 Thread protobuf
Comment #31 on issue 188 by xarthisius.kk: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 For comparison, the Free Software Foundation has... I've contributed to sever

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-22 Thread protobuf
Comment #32 on issue 188 by ken...@google.com: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 It looks like our CLA form may in fact be broken. :( I'll try to get it

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-22 Thread protobuf
Comment #33 on issue 188 by ken...@google.com: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 The issue should be fixed now. Sounds like it's too late, though. :(

[protobuf] Re: Issue 188 in protobuf: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread

2010-12-28 Thread protobuf
Comment #34 on issue 188 by xarthisius.kk: protobuf fails to link after compiling with LDFLAGS="-Wl,--as-needed" because of missing -lpthread http://code.google.com/p/protobuf/issues/detail?id=188 I've signed the CLA. Best regards, Kacper Kowalik -- You received this message

[protobuf] Re: Issue 60 in protobuf: Support static values in .proto files

2011-01-03 Thread protobuf
Comment #8 on issue 60 by mer...@google.com: Support static values in .proto files http://code.google.com/p/protobuf/issues/detail?id=60 I've been wishing for this once in a while. There's a workaround: Protobuf actually does have a syntax for literal values: field defaults.

<    2   3   4   5   6   7   8   9   10   11   >