On Mon, Oct 24, 2022 at 7:46 PM Xiangyu Chen <[email protected]> wrote:
>
>
> On 10/25/22 00:30, Peter Kjellerstedt wrote:
> > [Please note: This e-mail is from an EXTERNAL e-mail address]
> >
> >> -----Original Message-----
> >> From: [email protected] 
> >> <[email protected]> On Behalf Of Xiangyu Chen
> >> Sent: den 24 oktober 2022 11:57
> >> To: [email protected]
> >> Subject: [oe] [meta-oe][PATCH] ipmitool: add default iana enterprise 
> >> numbers database
> >>
> >> When using default configuration without add IANA_ENTERPRISE_NUMBERS and
> >> SRC_URI in a bbappend, the ipmitool would report an error as below:
> >>
> >> "
> >> root@intel-x86-64:~# ipmitool -I open bmc getenables
> >> IANA PEN registry open failed: No such file or directory
> >> "
> >>
> >> Add a default iana enterprise numbers database to avoid the error
> >> happen if using ipmitool default bb configuration.
> >>
> >> Signed-off-by: Xiangyu Chen <[email protected]>
> >> ---
> >>   .../ipmitool/ipmitool/iana-enterprise-numbers | 237682 +++++++++++++++
> >>   .../ipmitool/ipmitool_1.8.19.bb               |      6 +-
> >>   2 files changed, 237686 insertions(+), 2 deletions(-)
> >>   create mode 100644 
> >> meta-oe/recipes-kernel/ipmitool/ipmitool/iana-enterprise-numbers
> >>
> >> diff --git 
> >> a/meta-oe/recipes-kernel/ipmitool/ipmitool/iana-enterprise-numbers 
> >> b/meta-oe/recipes-kernel/ipmitool/ipmitool/iana-enterprise-numbers
> >> new file mode 100644
> >> index 000000000..f8c328796
> >> --- /dev/null
> > [cut 237k+ lines... ]
> >
> > For the benefit of all members of this list, do not send a 237k+
> > lines patch as a mail! The resulting mail was 20 MB. It took my
> > Outlook client over two hours to create this response (where I
> > then had to remove most of those 237k+ lines again)...
> >
> > Instead push the change to some public repository. For
> > OpenEmbedded you should be able to use the github repository and
> > create a merge request that way (and let us hope that Khem does
> > not have automatic tooling that sends all merge requests as
> > patches to the mailing list...)

Regardless, I would not like to have binary blobs in git metadata.
Please make it a separate downloadable artifact
hosted somewhere reliable.

>
> sorry about that, i'll pay attention to check the size next time.
>
> I've created a merge request on github
> (https://github.com/openembedded/meta-openembedded/pull/629)
>
> >> +++ b/meta-oe/recipes-kernel/ipmitool/ipmitool/iana-enterprise-numbers
> >> diff --git a/meta-oe/recipes-kernel/ipmitool/ipmitool_1.8.19.bb 
> >> b/meta-oe/recipes-kernel/ipmitool/ipmitool_1.8.19.bb
> >> index 0a600e23b..e166ea213 100644
> >> --- a/meta-oe/recipes-kernel/ipmitool/ipmitool_1.8.19.bb
> >> +++ b/meta-oe/recipes-kernel/ipmitool/ipmitool_1.8.19.bb
> >> @@ -27,9 +27,11 @@ SRC_URI = 
> >> "git://github.com/ipmitool/ipmitool;protocol=https;branch=master \
> >>              
> >> file://0001-ipmi_fru.c-Provide-missing-function-declarations.patch \
> >>              
> >> file://0001-configure-Remove-the-logic-to-download-IANA-PEN-data.patch \
> >>              "
> >> -IANA_ENTERPRISE_NUMBERS ?= ""
> >>
> >> -# Add these via bbappend if this database is needed by the system
> >> +# Default IANA enterprise database version is 2022-10-24
> >> +IANA_ENTERPRISE_NUMBERS ?= "file://iana-enterprise-numbers"
> > If you compress the file with xz, the size is reduced to a third.
> > Bitbake will then uncompress it automatically during do_unpack().
> Sure, I used a xz instead of the pure text file in latest version merge
> request.
> >
> >> +
> >> +# Add these via bbappend if the latest version database is needed by the 
> >> system
> >>   #IANA_ENTERPRISE_NUMBERS ?= 
> >> "http://www.iana.org/assignments/enterprise-numbers;name=iana-enterprise-numbers;downloadfilename=iana-enterprise-numbers";
> > Change "?=" to "=", or the suggestion to add it as a bbappend will
> > not work.
> I have changed ?= to = in latest version merge request.
> >>   #SRC_URI[iana-enterprise-numbers.sha256sum] = 
> >> "cdd97fc08325667434b805eb589104ae63f7a9eb720ecea73cb55110b383934c"
> >>
> >> --
> >> 2.34.1
> > //Peter
> >
>
> 
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#99301): 
https://lists.openembedded.org/g/openembedded-devel/message/99301
Mute This Topic: https://lists.openembedded.org/mt/94530972/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to