Hi Chris,

Thank you very much for the detailed explanation. :)

Best regards,
Ha Thach


> -----Original Message-----
> From: Christopher Collins [mailto:[email protected]]
> Sent: Wednesday, July 20, 2016 3:05 AM
> To: [email protected]
> Subject: Re: newtmgr - go install failed
> 
> On Wed, Jul 20, 2016 at 02:43:53AM +0700, hathach wrote:
> > Hi,
> >
> > I get into an issue with newtmgr installation (go install) with go
> > 1.6.3 on Windows & Ubuntu and OSX.
> >
> > # mynewt.apache.org/newt/newtmgr/cli
> > go\src\mynewt.apache.org\newt\newtmgr\cli\commands.go:45: cmd.Help
> > undefined (type *cobra.Command has no field or method Help)
> > go\src\mynewt.apache.org\newt\newtmgr\cli\connprofile.go:125: cmd.Help
> > undefined (type *cobra.Command has no field or method Help)
> > go\src\mynewt.apache.org\newt\newtmgr\cli\image.go:404: cmd.Help
> > undefined (type *cobra.Command has no field or method Help)
> > go\src\mynewt.apache.org\newt\newtmgr\cli\logs.go:196: cmd.Help
> > undefined (type *cobra.Command has no field or method Help)
> > go\src\mynewt.apache.org\newt\newtmgr\cli\usage.go:38: cmd.Help
> > undefined (type *cobra.Command has no field or method Help)
> >
> > I had to manually comment out those lines cmd.Help() to get the
> > installation going. Is there any chance that you guy miss a file or
> > two. I may miss something as well.
> 
> Hi Ha,
> 
> Thanks for the heads up.  It looks like the Cobra library was updated
upstream
> in a backwards incompatible way.  We should be using Go's vendoring
support
> for newtmgr to prevent this issue from occurring.
> 
> I just merged a pull request into the 'develop' branch to bring the
newtmgr
> code up to date with the latest Cobra.  So, everything in the newt repo
should
> now build if you use the develop branch Unfortunately,
> 0.9.0 remains broken, and I don't think there is anything that can be done
to fix
> it.  We will need to make sure a proper fix (i.e.,
> vendoring) goes into 0.10.0 so that this doesn't happen again.
> 
> Thanks,
> Chris

Reply via email to