From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Otavio Salvador <otavio.salvador@ossystems.com.br>
Cc: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 00/44] Replace Smart package manager with DNF package manager
Date: Tue, 14 Mar 2017 17:35:24 +0000 [thread overview]
Message-ID: <1489512924.13980.54.camel@linuxfoundation.org> (raw)
In-Reply-To: <CAP9ODKr_wVYKauiRWu9310DJ3zPkDQ7iuMeFAiN3J7rx2q76VQ@mail.gmail.com>
On Tue, 2017-03-14 at 14:17 -0300, Otavio Salvador wrote:
> On Tue, Mar 14, 2017 at 2:14 PM, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> >
> > On Tue, 2017-03-14 at 13:48 -0300, Otavio Salvador wrote:
> > >
> > > On Tue, Mar 14, 2017 at 12:14 PM, Richard Purdie
> > > <richard.purdie@linuxfoundation.org> wrote:
> > > ...
> > > >
> > > >
> > > > Despite this I merged it on the basis that I'd rather have this
> > > > in
> > > > pyro
> > > > rather than waiting another 6 months to get this in and having
> > > > to
> > > > tell
> > > > people we were changing soon and having to deal with any more
> > > > smart
> > > > issues.
> > > ...
> > >
> > > This kind of change should have been merged in November, not less
> > > of
> > > two months of release. One more time heavy changes been commited
> > > late
> > > on release.
> > In this case the patches have been talked about and in review on
> > the
> > mailing list for months so I really don't think anyone can be
> > surprised. It was also a stated release objective to do something
> > about
> > smart.
> >
> > Also, nobody replied to the status reports where I discussed
> > merging or
> > not merging either so I could only assume nobody had opinions.
> I remember Mark sending a reply with some considerations and to be
> honest, I did not anticipate it was being considered such late on
> release.
FWIW I do agree it is late in the cycle, almost too late and if things
hadn't been delayed for other reasons it wouldn't have made it.
That said, I do believe we'd potentially hurt OE more by waiting given
the stage the patchset had got to rather than merging it now and
dealing with whatever we need to deal with.
Its a judgement call, I have to make them on a regular basis. Some you
never need to notice, some like this one are controversial and can be
argued both ways. I can think of at least one in the past that went
badly wrong.
So I appreciate the concern and I do have some of my own but I have now
made the decision and we'll now get to see how that works out. It
wasn't an easy one...
Cheers,
Richard
next prev parent reply other threads:[~2017-03-14 17:35 UTC|newest]
Thread overview: 75+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-10 11:23 [PATCH 00/44] Replace Smart package manager with DNF package manager Alexander Kanavin
2017-03-10 11:23 ` [PATCH 01/44] package.bbclass: fix locations of debugedit and rpmdeps utilities Alexander Kanavin
2017-03-10 11:23 ` [PATCH 02/44] libsolv: enable rpm support Alexander Kanavin
2017-03-10 11:23 ` [PATCH 03/44] rpmresolve: remove the recipe Alexander Kanavin
2017-03-10 11:23 ` [PATCH 04/44] createrepo-c: add a recipe Alexander Kanavin
2017-03-10 11:23 ` [PATCH 05/44] createrepo: remove the recipe Alexander Kanavin
2017-03-10 11:23 ` [PATCH 06/44] rpm: add a 4.x recipe Alexander Kanavin
2017-03-10 11:23 ` [PATCH 07/44] rpm: remove 5.x recipe Alexander Kanavin
2017-03-10 11:23 ` [PATCH 08/44] python-pygpgme: add a recipe Alexander Kanavin
2017-03-10 11:23 ` [PATCH 09/44] python-iniparse: " Alexander Kanavin
2017-03-10 11:23 ` [PATCH 10/44] python-backports-lzma: " Alexander Kanavin
2017-03-10 11:23 ` [PATCH 11/44] librepo: " Alexander Kanavin
2017-03-10 11:23 ` [PATCH 12/44] libcomps: " Alexander Kanavin
2017-03-10 11:23 ` [PATCH 13/44] libdnf: " Alexander Kanavin
2017-03-10 11:23 ` [PATCH 14/44] dnf: " Alexander Kanavin
2017-03-10 11:23 ` [PATCH 15/44] python-smartpm: remove the recipe Alexander Kanavin
2017-03-10 11:23 ` [PATCH 16/44] db: remove the 6.x recipe Alexander Kanavin
2017-03-10 11:23 ` [PATCH 17/44] package_rpm.bbclass: make architecture-independent .rpm packages "noarch" instead of "all" Alexander Kanavin
2017-03-10 11:23 ` [PATCH 18/44] package_manager.py: improve the API for insert_feed_uris() Alexander Kanavin
2017-03-10 11:23 ` [PATCH 19/44] package_rpm.bbclass: correctly map RRECOMMENDS and RSUGGESTS to rpm tags Alexander Kanavin
2017-03-10 16:41 ` Mark Hatle
2017-03-10 11:23 ` [PATCH 20/44] nativesdk-packagegroup-sdk-host: replace smartpm with dnf Alexander Kanavin
2017-03-10 11:24 ` [PATCH 21/44] rootfs_rpm.bbclass: migrate image creation to dnf Alexander Kanavin
2017-03-13 14:25 ` Burton, Ross
2017-03-13 14:52 ` Alexander Kanavin
2017-03-13 15:04 ` Alexander Kanavin
2017-03-13 15:07 ` Alexander Kanavin
2017-03-10 11:24 ` [PATCH 22/44] scripts/rpm2cpio.sh: replace 5.x version with 4.x version Alexander Kanavin
2017-03-10 17:26 ` Mark Hatle
2017-03-14 12:44 ` Burton, Ross
2017-03-10 11:24 ` [PATCH 23/44] scripts/oe-setup-rpmrepo: replace createrepo with createrepo_c Alexander Kanavin
2017-03-10 11:24 ` [PATCH 24/44] package_rpm.bbclass: replace createrepo reference " Alexander Kanavin
2017-03-10 11:24 ` [PATCH 25/44] package_rpm.bbclass: no need to specify the magic file location Alexander Kanavin
2017-03-10 11:24 ` [PATCH 26/44] testimage.bbclass: fix runtime test for rpm, port smart tests to dnf Alexander Kanavin
2017-03-10 11:24 ` [PATCH 27/44] package_rpm.bbclass: instruct rpm to not remove the buildroot directory Alexander Kanavin
2017-03-10 11:24 ` [PATCH 28/44] selftest: do not perform a full build in test_continue Alexander Kanavin
2017-03-10 11:24 ` [PATCH 29/44] selftest: update output checks in incremental image test to match what dnf prints Alexander Kanavin
2017-03-10 11:24 ` [PATCH 30/44] byacc: remove the recipe Alexander Kanavin
2017-03-10 16:37 ` Mark Hatle
2017-03-13 13:18 ` Alexander Kanavin
2017-03-13 14:02 ` Mark Hatle
2017-03-10 11:24 ` [PATCH 31/44] gpg_sign.py: fix signing of rpm files using gpg Alexander Kanavin
2017-03-10 11:24 ` [PATCH 32/44] oe-selftest: no need to parse gpg output to determine the name of the signing key Alexander Kanavin
2017-03-10 11:24 ` [PATCH 33/44] run-postinsts: simplify the logic of whether to install it to images Alexander Kanavin
2017-03-10 11:24 ` [PATCH 34/44] package_rpm.bbclass: add a /bin/sh Provides for nativesdk- packages Alexander Kanavin
2017-03-10 16:54 ` Mark Hatle
2017-03-13 14:21 ` Alexander Kanavin
2017-03-13 14:27 ` Mark Hatle
2017-03-10 11:24 ` [PATCH 35/44] security_flags.inc: update to reflect smart->dnf transition Alexander Kanavin
2017-03-10 11:24 ` [PATCH 36/44] package_rpm.bbclass: do not set the OS dynamically when building rpms Alexander Kanavin
2017-03-10 16:57 ` Mark Hatle
2017-03-13 14:24 ` Alexander Kanavin
2017-03-13 14:29 ` Mark Hatle
2017-03-10 11:24 ` [PATCH 37/44] nspr: do not use shared library symbol resolution with musl Alexander Kanavin
2017-03-10 11:24 ` [PATCH 38/44] package_rpm.bbclass: do not strip multilib prefixes from package names, do not add multilib prefix to package arch Alexander Kanavin
2017-03-10 17:04 ` Mark Hatle
2017-03-13 14:35 ` Alexander Kanavin
2017-03-10 11:24 ` [PATCH 39/44] package.bbclass: don't exclude package groups from runtime mapping Alexander Kanavin
2017-03-10 17:12 ` Mark Hatle
2017-03-10 11:24 ` [PATCH 40/44] glibc: do the multilib_header magic also for bits/long-double.h Alexander Kanavin
2017-03-10 11:24 ` [PATCH 41/44] bash: package bashbug separately Alexander Kanavin
2017-03-10 17:13 ` Mark Hatle
2017-03-13 14:39 ` Alexander Kanavin
2017-03-10 11:24 ` [PATCH 42/44] db: do the multilib_header processing for db.h Alexander Kanavin
2017-03-10 11:24 ` [PATCH 43/44] base-files: do not write the current date into /etc/issue and /etc/issue.net Alexander Kanavin
2017-03-11 11:22 ` Richard Purdie
2017-03-11 16:22 ` Richard Purdie
2017-03-10 11:24 ` [PATCH 44/44] gpgme: fix python module build and installation Alexander Kanavin
2017-03-14 15:14 ` [PATCH 00/44] Replace Smart package manager with DNF package manager Richard Purdie
2017-03-14 16:48 ` Otavio Salvador
2017-03-14 17:14 ` Richard Purdie
2017-03-14 17:17 ` Otavio Salvador
2017-03-14 17:35 ` Richard Purdie [this message]
2017-03-14 21:12 ` Robert P. J. Day
2017-03-15 9:04 ` Alexander Kanavin
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=1489512924.13980.54.camel@linuxfoundation.org \
--to=richard.purdie@linuxfoundation.org \
--cc=openembedded-core@lists.openembedded.org \
--cc=otavio.salvador@ossystems.com.br \
/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