[protobuf] Re: Issue 187 in protobuf: Command-line argument to override the optimize_for option

2014-11-05 Thread protobuf
Comment #14 on issue 187 by austin.l...@gmail.com: Command-line argument to override the optimize_for option https://code.google.com/p/protobuf/issues/detail?id=187 I was able to use Keaton's plugin as a starting point and am using it now. I didn't know that the protobuf compiler was that

[protobuf] Re: Issue 187 in protobuf: Command-line argument to override the optimize_for option

2010-07-28 Thread protobuf
Comment #11 on issue 187 by ken...@google.com: Command-line argument to override the optimize_for option http://code.google.com/p/protobuf/issues/detail?id=187 Hi all, Attached is a little protoc plugin written in Python which converts all the inputs to LITE_RUNTIME -- including renaming

[protobuf] Re: Issue 187 in protobuf: Command-line argument to override the optimize_for option

2010-07-28 Thread protobuf
Comment #12 on issue 187 by ken...@google.com: Command-line argument to override the optimize_for option http://code.google.com/p/protobuf/issues/detail?id=187 s/code code/core code/ s/retains/remains/ *sigh* -- You received this message because you are subscribed to the Google Groups

[protobuf] Re: Issue 187 in protobuf: Command-line argument to override the optimize_for option

2010-07-20 Thread protobuf
Comment #7 on issue 187 by ken...@google.com: Command-line argument to override the optimize_for option http://code.google.com/p/protobuf/issues/detail?id=187 If you just want to append a string to the file, you can easily do that like: cp foo.proto foo2.proto echo 'option

[protobuf] Re: Issue 187 in protobuf: Command-line argument to override the optimize_for option

2010-07-20 Thread protobuf
Comment #8 on issue 187 by dima: Command-line argument to override the optimize_for option http://code.google.com/p/protobuf/issues/detail?id=187 Now suppose foo.proto is imported by a few other .proto files... You'll have to modify each one of them, too. Every other compiler - gcc,

[protobuf] Re: Issue 187 in protobuf: Command-line argument to override the optimize_for option

2010-07-20 Thread protobuf
Comment #9 on issue 187 by ken...@google.com: Command-line argument to override the optimize_for option http://code.google.com/p/protobuf/issues/detail?id=187 I agree that protoc should provide something. I was merely pointing out that eshober's suggestion can trivially be implemented

[protobuf] Re: Issue 187 in protobuf: Command-line argument to override the optimize_for option

2010-07-20 Thread protobuf
Comment #10 on issue 187 by esho...@fusionio.com: Command-line argument to override the optimize_for option http://code.google.com/p/protobuf/issues/detail?id=187 Admittedly, my suggestion above does not handle imports and that makes it mostly unusable whether done in or outside of protoc.

[protobuf] Re: Issue 187 in protobuf: Command-line argument to override the optimize_for option

2010-07-16 Thread protobuf
Comment #6 on issue 187 by esho...@fusionio.com: Command-line argument to override the optimize_for option http://code.google.com/p/protobuf/issues/detail?id=187 What about an even more generic implementation where you can optionally poke arbitrary strings into the file from the commandline?

[protobuf] Re: Issue 187 in protobuf: Command-line argument to override the optimize_for option

2010-07-13 Thread protobuf
Comment #5 on issue 187 by esho...@fusionio.com: Command-line argument to override the optimize_for option http://code.google.com/p/protobuf/issues/detail?id=187 We also need this feature. We're sharing .proto files between applications. One of the apps has a requirement to be optimized

[protobuf] Re: Issue 187 in protobuf: Command-line argument to override the optimize_for option

2010-05-17 Thread protobuf
Comment #4 on issue 187 by ken...@google.com: Command-line argument to override the optimize_for option http://code.google.com/p/protobuf/issues/detail?id=187 I agree, we should be able to override options on the command-line. The only problem is that it's unclear how far this support

[protobuf] Re: Issue 187 in protobuf: Command-line argument to override the optimize_for option

2010-05-13 Thread protobuf
Comment #3 on issue 187 by mike.weller: Command-line argument to override the optimize_for option http://code.google.com/p/protobuf/issues/detail?id=187 This is something I've always thought was needed. It makes much more sense to me to define these kinds of options on the command line so

[protobuf] Re: Issue 187 in protobuf: Command-line argument to override the optimize_for option

2010-05-12 Thread protobuf
Comment #1 on issue 187 by dima: Command-line argument to override the optimize_for option http://code.google.com/p/protobuf/issues/detail?id=187 Actually, the patch is wrong... It overrides optimize_for if it's set in .proto files, but does not override the default value. -- You

[protobuf] Re: Issue 187 in protobuf: Command-line argument to override the optimize_for option

2010-05-12 Thread protobuf
Comment #2 on issue 187 by n...@chromium.org: Command-line argument to override the optimize_for option http://code.google.com/p/protobuf/issues/detail?id=187 Would it be much more difficult to support arbitrary option overrides on the command line? -- You received this message because