Re: [Buildroot] [PATCH 1/3] configs/armadeus_apf27: add hashes for Linux/U-Boot

["Yann E. MORIN" <yann.morin.1998@free.fr>]

Julien, All,

On 2024-05-05 22:07 +0200, julien.boibessot@armadeus.com spake thusly:
> Le 2024-05-05 10:38, Yann E. MORIN a écrit :
> > Julien, All,
> > On 2024-05-03 17:48 +0200, Julien BOIBESSOT spake thusly:
> > > And enable BR2_DOWNLOAD_FORCE_CHECK_HASHES. As asked By Romain.
> >     $ ./utils/docker-run make check-package
> >     configs/armadeus_apf27_defconfig:0: lib_defconfig.ForceCheckHash
> > was expected to fail, did you fix the file and forget to update
> > .checkpackageignore?
[--SNIP--]
> Thanks for the fix ! I also saw the messages when running "make
> check-package" but didn't know if it was my job or not to fix
> .checkpackageignore file.

Entries in .checkpackageignore means that those files are exempleted
from the associated tests. Presumably, those files pre-existed
check-package, and thus we did not want to fail for those, so we
excluded them.

So, when a file is fixed, the corresponding exclusion must be removed.

In this case, you fixed the lib_defconfig.ForceCheckHash test, so now it
no longer fails, but it was expected to fail. so the correct solution is
indeed for you to state "yes, I fixed it, so it is no longer excluded!".

> Is there any documentation on what I was supposed to do ?

Err... Joker.

Thanks!

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot