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, hzh...@ti.com wrote:
> > > > From: Hao Zhang
> > > >
> > > > Signed-off-by: Hao Zhang
> > > > ---
> > > > 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 000..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
--
___
meta-ti mailing list
meta-ti@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-ti
