All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Richard Purdie" <richard.purdie@linuxfoundation.org>
To: Konrad Weihmann <kweihmann@outlook.com>,
	Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [OE-core] [RFC PATCH 14/14] layer.conf: Extend recipes not to install without explict dependencies
Date: Mon, 18 Oct 2021 22:08:02 +0100	[thread overview]
Message-ID: <74058fe34e8fcdff0337ec54674fbac5b02c559a.camel@linuxfoundation.org> (raw)
In-Reply-To: <AM9PR09MB46425E114838BDB1CB772C89A8BC9@AM9PR09MB4642.eurprd09.prod.outlook.com>

On Mon, 2021-10-18 at 21:07 +0200, Konrad Weihmann wrote:
> Just as an idea: can those knowingly breaking changes be announced 
> upfront somehow? maybe in the architecture list?!? (and then applied on 
> a flag day)

This one was announced on the mailing list in patch form, in the weekly tech
calls and I think in the weekly status emails too. It was announced enough that
I had feedback saying "not in the release". I listened to the feedback and
deferred it until we started merging new development for the next release. So it
was actually pending for a while.

> I know we all got a bit of a high workload, so it might be better to 
> have breaking changes not hit "uncontrolled" into master.
> 
> I agree that master is a development branch, but the number of breaking 
> changes that hit me by surprise this year is astonishingly high.

I think the problem is that for a period, active development of these kinds of
changes stagnated. That is good and bad, it gave stability but it also meant we
didn't tackle and fix some of the hard issues we need to fix for the project to
move forward. 

For the record the bigger changes were discussed on the architecture list so
those shouldn't have surprised anyone yet they still did :(.

> That doesn't mean, that I don't appreciate the actual change made, but 
> that way it has been applied (and the breaking changes before) makes me 
> rethink the way I do integrate my non-core layer in the future, while I 
> personally would like to remain on this bleeding-edge model that worked 
> well for a long time.

I think the question is more how long these things are breaking for. If things
were broken and then spent weeks languishing in that state, there would be cause
for concern. I'm not aware of that happening and most things do seem to be
getting resolved quickly. Those that aren't are a warning sign as it means
they're not being actively maintained and you need to ask other questions in
that case.

There is a balance here between being able to move the project forward and
drowning under the shear weight of "before changing X you need to write a
proposal, wait X weeks, get TSC approval, ensure it was in X weekly status
reports, talked about in X weekly meetings" and so on. I actually keep asking
myself, "why do I bother trying to change anything as I generally just get
complaints about the work it causes people?". I get really depressed at the
complaints I see and if I feel like this, it must be a real barrier to others
too and it will stop things changing. I don't go out with the intent of making
things difficult, I actually want to see the project move forward technically
not stagnate which I think it is at serious risk of. Do we really want to put
high barriers in place to development?

I will raise this with the TSC as a concern so they can review things.

Cheers,

Richard


  reply	other threads:[~2021-10-18 21:08 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-20 12:46 [PATCH 01/14] vim: Backport fix for CVE-2021-3770 Richard Purdie
2021-09-20 12:46 ` [PATCH 02/14] libgcrypt: Upgrade 1.9.3 -> 1.9.4 Richard Purdie
2021-09-20 12:46 ` [PATCH 03/14] sqlite3: Exclude CVE-2021-36690 from cve checks Richard Purdie
2021-09-20 12:46 ` [PATCH 04/14] recipes: Add missing pkgconfig inherit Richard Purdie
2021-09-20 12:46 ` [PATCH 05/14] lttng-tools: Add missing DEPENDS on bison-native Richard Purdie
2022-01-18 20:48   ` [OE-core] " Denys Dmytriyenko
2021-09-20 12:46 ` [PATCH 06/14] image/qemu: Add explict depends for qemu-helper addto_recipe_sysroot task Richard Purdie
2021-09-20 12:46 ` [PATCH 07/14] staging: Mark deploy an sstate task Richard Purdie
2021-09-20 12:46 ` [PATCH 08/14] sstate: Ensure deploy tasks don't pull in toolchains Richard Purdie
2021-09-20 12:46 ` [PATCH 09/14] sstate: Avoid deploy_source_date_epoch sstate when unneeded Richard Purdie
2021-09-20 12:46 ` [RFC PATCH 10/14] package_ipk/deb/rpm: Drop recursive do_build task dependencies Richard Purdie
2021-09-23 21:41   ` [OE-core] " Peter Kjellerstedt
2021-09-23 21:58     ` Richard Purdie
2021-09-24  4:50     ` Khem Raj
2021-09-24  7:58       ` Martin Jansa
2021-09-24  8:30         ` Richard Purdie
2021-09-24 17:20         ` Khem Raj
2021-09-20 12:46 ` [RFC PATCH 11/14] populate_sdk_base/images: Drop use of 'meta' class and hence do_build dependencies Richard Purdie
2021-10-27  2:43   ` [OE-core] " ChenQi
2021-11-02 13:06     ` Richard Purdie
2021-09-20 12:46 ` [PATCH 12/14] buildtools-tarball/uninative-tarball/meta-ide-support: Drop useless meta class Richard Purdie
2021-09-20 12:46 ` [PATCH 13/14] meta: Drop useless class Richard Purdie
2021-09-20 12:46 ` [RFC PATCH 14/14] layer.conf: Extend recipes not to install without explict dependencies Richard Purdie
     [not found] ` <16A68880435BB472.28512@lists.openembedded.org>
2021-09-20 12:48   ` [OE-core] " Richard Purdie
2021-09-20 13:34     ` Joshua Watt
2021-09-21  4:21       ` Khem Raj
2021-10-01 14:17         ` Martin Jansa
2021-10-17 23:50           ` Andreas Müller
2021-10-18 14:12             ` Martin Jansa
2021-10-18 14:29               ` Richard Purdie
2021-10-18 16:50               ` Andreas Müller
2021-10-18 16:59               ` Andreas Müller
2021-10-18 19:07                 ` Konrad Weihmann
2021-10-18 21:08                   ` Richard Purdie [this message]
     [not found] ` <16A6887F33E2E04C.31899@lists.openembedded.org>
2021-09-20 12:51   ` [OE-core] [RFC PATCH 11/14] populate_sdk_base/images: Drop use of 'meta' class and hence do_build dependencies Richard Purdie
2021-09-20 16:32     ` Khem Raj
2021-09-20 20:02       ` Richard Purdie

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=74058fe34e8fcdff0337ec54674fbac5b02c559a.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=kweihmann@outlook.com \
    --cc=openembedded-core@lists.openembedded.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.