Updated http://wiki.apache.org/hadoop/DevelopingOnTrunkAfter279Merge and filed MAPREDUCE-2901.
+Vinod On Fri, Aug 19, 2011 at 12:22 AM, Alejandro Abdelnur <[email protected]>wrote: > protoc is invoked from an antrun plugin configuration. > > we could check using ant tasks that protoc is avail, and fail with a > message > if not. > > Thxs. > > Alejandro > > On Thu, Aug 18, 2011 at 11:44 AM, Eli Collins <[email protected]> wrote: > > > On Thu, Aug 18, 2011 at 11:39 AM, Alejandro Abdelnur <[email protected]> > > wrote: > > > IMO, committing generated code is not a good idea. I'd rather put the > > burden > > > on developers of installing protoc (they have to do it for Java, > Maven, > > > Forrest, and all the autoconf stuff if compiling native). > > > > > > I would document protoc as required tool. > > > > > > > Alejandro - is there an easy way we can make maven spit out a nicer > > error message (eg check for protoc on the path and print a message if > > it's not present)? > > > > (Thank you Mahadev btw for pointing out the fix!) > > > > Thanks, > > Eli > > >
