[Buildroot] [PATCH 1/1] package/makedumpfile: fix build for powerpc target

Alexander Egorenkov egorenar-dev at posteo.net
Thu Sep 10 07:08:47 UTC 2020


> Is there an autobuilder failure for this? If so, there should be a link
> to it in your commit log. See for example this commit log:
>
>   https://git.buildroot.org/buildroot/commit/?id=a85ae8b297fd0526f57fe99d73087dd840ea6219
>

Fixed.

>
> I'm not sure the best solution here is to patch makedumpfile. Indeed,
> Buildroot's makedumpfile.mk is passing TARGET=$(BR2_ARCH), but there is
> nothing that "forces" makedumpfile to understand the BR2_ARCH values.
>
> So instead, I think we should have some logic in makedumpfile.mk that
> calculates the correct TARGET values depending on the CPU architecture.
> Something along the lines of:
>
> ifeq ($(BR2_aarch64)$(BR2_aarch64_be),y)
> MAKEDUMPFILE_TARGET = ...
> else ifeq ($(BR2_arm)$(BR2_armeb),y)
> MAKEDUMPFILE_TARGET = ...
> else ifeq ...
> ...
> endif
>
> Perhaps if for some architectures the BR2_ARCH value is OK, you could
> finish this if sequence by:
>
> else
> MAKEDUMPFILE_TARGET = $(BR2_ARCH)
> endif
>
> and then of course, pass TARGET=$(MAKEDUMPFILE_TARGET)


Fixed. I find your suggestion a lot cleaner and like it better.

The new version of patch is on the way. 

Thanks for your constructive criticism
Regards
Alex



More information about the buildroot mailing list