Thanks Vinod! On Aug 29, 2011, at 12:30 AM, Vinod Kumar Vavilapalli wrote:
> 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 >>> >>
