[U-Boot] [STATUS] "Quality" of patches / testing.
Anton Staaf
robotboy at google.com
Tue Oct 18 19:16:35 CEST 2011
On Mon, Oct 17, 2011 at 11:23 PM, Wolfgang Denk <wd at denx.de> wrote:
> Hi all,
>
> the patches that have been submitted for this release turn out to of
> of shockingly bad quality. About every other batch of patches I apply
> will break building not only for a single board r a few boards, but
> for large numbers of boards, including whole processor families and
> even whole architectures.
>
> Not to mention the huge number of patches that have to be rejected
> because they raise tons or errors and warnings from checkpatch.pl
>
>
> The time I have available is limited even without such avoidable
> problems, and I really, really rather spend it on constructive work
> instead of continuously running git bisect to track down the culprits.
>
> What's even worse is that it appears that I am the only person in this
> whole community who runs any build tests. Or how comes that it's
> always me who detects such build breakages?
>
>
> Note # 3 at http://www.denx.de/wiki/U-Boot/Patches says:
>
> Before sending the patch, you must run the MAKEALL script on
> your patched source tree and make sure that no errors or
> warnings are reported for any of the boards. Well, at least
> not any more warnings than without your patch.
>
> It is strongly recommended to verify that out-of-tree building
> (with "-O" make option resp. "BUILD_DIR" environment variable)
> is still working. For example, run:
>
> $ BUILD_DIR=/tmp/u-boot-build ./MAKEALL
>
> Why is nobody doing this?
I would like to start a thread addressing this question. I don't think
the people submitting are running MAKEALL because it has a very high
barrier to entry. In particular I spent a few days trying to get as
many architecture toolchains up and running as I could so that I could
run MAKEALL to fully test my builds. The result was frustrating. I
was only able to get ARM to build (Our local toolchain works, but I also
got the ELDK toolchain to work for me). I was not able to get the ELDK
toolchain for PowerPC to work however.
-Anton
Ahh, looks like Mike agrees...
> I cannot continue like that. We need to find ways to improve the
> quality of the submitted patches, and to distribute the work load for
> testing.
>
> I need your help.
>
>
> Thanks.
>
> Wolfgang Denk
>
> --
> DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> There are no data that cannot be plotted on a straight line if the
> axis are chosen correctly.
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
More information about the U-Boot
mailing list