Re: [PATCH 1/2] recipes-ti: devtools: add TI XDC tools 3.50.07.20

[Denys Dmytriyenko <denys@ti.com>]

On Tue, Aug 14, 2018 at 02:39:45PM -0400, Denys Dmytriyenko wrote:
> On Tue, Aug 14, 2018 at 10:58:21AM -0400, Zhang, Hao wrote:
> > 
> > 
> > > -----Original Message-----
> > > From: Dmytriyenko, Denys
> > > Sent: Monday, August 13, 2018 8:56 PM
> > > To: Zhang, Hao
> > > Cc: meta-ti@yoctoproject.org
> > > Subject: Re: [meta-ti] [PATCH 1/2] recipes-ti: devtools: add TI XDC tools
> > > 3.50.07.20
> > > 
> > > On Fri, Aug 10, 2018 at 05:54:27AM -0400, hzhang@ti.com wrote:
> > > > From: Hao Zhang <hzhang@ti.com>
> > > >
> > > > Signed-off-by: Hao Zhang <hzhang@ti.com>
> > > > ---
> > > >  recipes-ti/devtools/ti-xdctools_3.50.07.20.bb | 9 +++++++++
> > > >  1 file changed, 9 insertions(+)
> > > >  create mode 100644 recipes-ti/devtools/ti-xdctools_3.50.07.20.bb
> > > 
> > > Why is 3.50.03 being left around?
> > > 
> > Please keep the old version for now, we will remove it later if it is not 
> > needed.
> 
> The general OE convention is to keep only the latest version, unless there are 
> clear uses for older version as well.
> 
> That said, we keep on adding new versions and never remove old ones - 3.23, 
> 3.30, 3.50 and now another minor version of 3.50.
> 
> I'd understand the need for 3.23 and 3.30 if they are API/ABI incompatible 
> with 3.50, but two minor versions of 3.50?
> 
> Please start cleaning those up. Thanks.

And also please make patches as diffs against old versions - use -C/-M flags.


> > > > diff --git a/recipes-ti/devtools/ti-xdctools_3.50.07.20.bb b/recipes-
> > > ti/devtools/ti-xdctools_3.50.07.20.bb
> > > > new file mode 100644
> > > > index 0000000..d45f373
> > > > --- /dev/null
> > > > +++ b/recipes-ti/devtools/ti-xdctools_3.50.07.20.bb
> > > > @@ -0,0 +1,9 @@
> > > > +require recipes-ti/devtools/ti-xdctools-core.inc
> > > > +
> > > > +PV = "3_50_07_20"
> > > > +PR = "r0"
> > > > +
> > > > +LIC_FILES_CHKSUM =
> > > "file://docs/license/xdc/shelf/package.html;beginline=1;endline=140;md5=1
> > > 796f68c3d32d408415f5612f1f60652"
> > > > +
> > > > +SRC_URI[xdczip.md5sum] = "52e4b5ddf2e004c6acff52104840724e"
> > > > +SRC_URI[xdczip.sha256sum] =
> > > "bef631c110ca9243790de06cd51839170d7979b92e01396e4265009776ff7a07
> > > "
> > > > --
> > > > 2.7.4
> > > >
> > > > --
> > > > _______________________________________________
> > > > meta-ti mailing list
> > > > meta-ti@yoctoproject.org
> > > > https://lists.yoctoproject.org/listinfo/meta-ti
> -- 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti