From: "Fredrik Gustafsson" <fredrik.gustafsson@axis.com>
To: Paul Barker <pbarker@konsulko.com>
Cc: openembedded-core <openembedded-core@lists.openembedded.org>,
tools-cfpbuild-internal <tools-cfpbuild-internal@axis.com>
Subject: Re: [OE-core] Add package managers as a plugin
Date: Fri, 3 Jul 2020 11:45:13 +0000 [thread overview]
Message-ID: <1593776713224.44705@axis.com> (raw)
In-Reply-To: <CAM9ZRVtfoy12++04DPYrOk_89n8FsmwRuGWkWFa2O6q4a9Zjnw@mail.gmail.com>
________________________________________
From: Paul Barker <pbarker@konsulko.com>
Sent: Thursday, July 2, 2020 10:28 PM
To: Fredrik Gustafsson
Cc: openembedded-core; tools-cfpbuild-internal
Subject: Re: [OE-core] Add package managers as a plugin
On Thu, 2 Jul 2020 at 20:29, Fredrik Gustafsson
<fredrik.gustafsson@axis.com> wrote:
>
> OE-core today has three different package managers, the well-known formats deb
> and rpm is supported as well as ipkg that is good for embedded devices.
>
> When building and having a good cache hit, a significant amount of time is
> spent in the phase of generating a rootfs, which is really about the
> performance of the package manager. To save build time and also get a
> package manager that is suitanle for use on targets where flash memory is a
> concern, support for apk is suggested.
>
> However, it might or might not be what's wanted for OE-core since it increases
> the test matrix. Therefore I will refactor the package management code to
> allow a layer to add a new package manager without editing the meta layer.
>
> This refactor will be divided into multiple patch series to be easier to review,
> this is the second patch series.
Sorry to be a pain but I think this series suffers from the opposite
problem to the previous version - it's only a partial move of some of
the code. I'd much rather review a series which does the complete
refactor of the existing code into separate modules without making any
functional changes. I'll let others give their opinions as well, I'm
happy to review this one if we do want to look at this piecemeal.
--
Paul Barker
Konsulko Group
No problems, as said before I will do my best to make this easier to review.
I've sent a new patch series.
/Fredrik
next prev parent reply other threads:[~2020-07-03 11:45 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-02 19:29 Add package managers as a plugin Fredrik Gustafsson
2020-07-02 19:29 ` [PATCH v3 1/3] Move rpm manifest to its own subdir Fredrik Gustafsson
2020-07-02 19:29 ` [PATCH v3 2/3] Move ipk " Fredrik Gustafsson
2020-07-02 19:29 ` [PATCH v3 3/3] Move deb " Fredrik Gustafsson
2020-07-02 19:32 ` ✗ patchtest: failure for "[v3] Move rpm manifest to its ..." and 2 more Patchwork
2020-07-02 19:34 ` Fredrik Gustafsson
2020-07-03 10:51 ` [OE-core] " Richard Purdie
2020-07-02 20:28 ` [OE-core] Add package managers as a plugin Paul Barker
2020-07-03 11:45 ` Fredrik Gustafsson [this message]
-- strict thread matches above, loose matches on Subject: below --
2020-06-25 10:21 Fredrik Gustafsson
2020-06-25 10:40 ` [OE-core] " Paul Barker
2020-06-30 18:39 ` Fredrik Gustafsson
2020-06-25 10:13 Fredrik Gustafsson
2020-06-25 10:24 ` [OE-core] " Alexander Kanavin
2020-06-30 15:04 ` Fredrik Gustafsson
2020-06-30 15:43 ` Alexander Kanavin
2020-06-30 18:46 ` Fredrik Gustafsson
2020-06-26 10:52 ` Ross Burton
2020-06-27 4:59 ` Tim Orling
2020-06-30 15:36 ` Khem Raj
2020-06-30 19:01 ` Fredrik Gustafsson
2020-06-30 21:54 ` Martin Jansa
2020-06-30 22:12 ` Fredrik Gustafsson
2020-07-01 19:12 ` Alejandro del Castillo
2020-06-30 23:15 ` Andre McCurdy
2020-07-01 0:16 ` Martin Jansa
2020-06-23 11:13 Fredrik Gustafsson
2020-06-23 18:50 ` [OE-core] " Denys Dmytriyenko
2020-06-30 15:15 ` Fredrik Gustafsson
2020-06-24 15:18 ` Alex Stewart
2020-06-30 18:38 ` Fredrik Gustafsson
2020-06-30 20:22 ` Alex Stewart
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=1593776713224.44705@axis.com \
--to=fredrik.gustafsson@axis.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=pbarker@konsulko.com \
--cc=tools-cfpbuild-internal@axis.com \
/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