Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Gary Bisson <gary.bisson@boundarydevices.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2 2/2] package/mfgtools: drop package
Date: Tue, 14 Apr 2020 08:46:24 +0200	[thread overview]
Message-ID: <20200414064624.GA277362@p1g2> (raw)
In-Reply-To: <3e5e773be990132bab2d0067c5e2d69d02c8fe31.camel@embedded.rocks>

Hi Jorg,

On Mon, Apr 13, 2020 at 10:24:06PM +0200, J?rg Krause wrote:
> Hi Gary,
> 
> On Mon, 2020-02-10 at 17:26 +0100, Gary Bisson wrote:
> > Hi Jorge,
> > 
> > On Thu, Jan 09, 2020 at 08:10:20PM +0100, J?rg Krause wrote:
> > > As suggested in [1] the package mfgtools is dropped.
> > > 
> > > NXP did replaced the old mfgtools with the version number 0.2
> > > enterily with the uuu (Universal Update Utility) which is somehow
> > > named mfgtools 3.0 although the version scheme for the uuu tool is
> > > 1.xx.yyy.
> > > 
> > > As the old mfgtools scripts are not compatible with the new uuu
> > > tool and as imx-uuu goes hand-in-hand with imx-uuc, which we ship
> > > for the target, the mfgtools package is dropped.
> > 
> > I know I'm the one who pushed for that commit to happen but now want to
> > mitigate the claim above.
> 
> Thanks for your feedback and sorry for my late reply.
> 
> > True uuu depends on imx-uuc and mfgtools (1st of its name) too. BUT by
> > looking at imx-uuc, the two use cases are separated into 2 different
> > files:
> > - uu.c: daemon that uses utp protocol to communicate with host
> > - ufb.c: daemon that uses fastboot protocol to communicate with host
> 
> In fact, I am using mfgtools without imx-uuc, but with U-Boot.
> 
> > So I think it should be safe to have both host clients co-existing for
> > now. Maybe uu.c will be removed at some point in the future but I guess
> > NXP will leave it there for a while.
> 
> So, lets just update mfgtools to latest version (1.3.154), right?

No, mfgtools 1.x.y really is 'uuu', not mfgtools any more.

So since 'mfgtools' doesn't have any more build issues, I'd say we keep
it as-is for people that use it. So the idea is _not_ to drop it.

Then just have your uuu addition patch, so basically both tools
co-exist in Buildroot (as they serve different purposes).

Regards,
Gary

  reply	other threads:[~2020-04-14  6:46 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-01-09 19:10 [Buildroot] [PATCH v2 1/2] package/imx-uuu: new host package Jörg Krause
2020-01-09 19:10 ` [Buildroot] [PATCH v2 2/2] package/mfgtools: drop package Jörg Krause
2020-02-10 16:26   ` Gary Bisson
2020-04-13 20:24     ` Jörg Krause
2020-04-14  6:46       ` Gary Bisson [this message]
2020-04-14  7:47         ` Jörg Krause
2020-04-14  8:14           ` Gary Bisson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200414064624.GA277362@p1g2 \
    --to=gary.bisson@boundarydevices.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox