Re: [PATCH] Makefile: fix config cc version check

[Masahiro Yamada]

On Sun, Apr 3, 2022 at 6:11 PM Akemi Yagi <toracat@xxxxxxxxxx> wrote:
>
> On Tue, 08 Mar 2022 11:23:31 +0900, Masahiro Yamada wrote:
>
> > On Tue, Mar 8, 2022 at 7:32 AM Michael S. Tsirkin <mst@xxxxxxxxxx>
> > wrote:
> >>
> >> .config is of the form:
> >> CONFIG_CC_VERSION_TEXT="gcc (GCC) 11.2.1 20220127 (Red Hat 11.2.1-9)"
> >
> >
> > No. This was changed.
> >
> > See this:
> >
> > $ git show 129ab0d2d9f -- Makefile
> >
> >
> >
> >> while CC_VERSION_TEXT is of the form: gcc (GCC) 11.2.1 20220127 (Red
> >> Hat 11.2.1-7)
> >>
> >> thus when comparing these, CONFIG_CC_VERSION_TEXT should not be put in
> >> "", otherwise we get () outside "" which shell then tries to evaluate.
> >>
> >> Signed-off-by: Michael S. Tsirkin <mst@xxxxxxxxxx>
> >> ---
> >>  Makefile | 4 ++--
> >>  1 file changed, 2 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/Makefile b/Makefile index daeb5c88b50b..d5c03c827825
> >> 100644 --- a/Makefile +++ b/Makefile @@ -1714,9 +1714,9 @@ PHONY +=
> >> prepare
> >>  # now expand this into a simple variable to reduce the cost of shell
> >>  evaluations prepare: CC_VERSION_TEXT := $(CC_VERSION_TEXT)
> >>  prepare:
> >> -       @if [ "$(CC_VERSION_TEXT)" != "$(CONFIG_CC_VERSION_TEXT)" ];
> >> then \
> >> +       @if [ "$(CC_VERSION_TEXT)" != $(CONFIG_CC_VERSION_TEXT) ]; then
> >> \
> >>                 echo >&2 "warning: the compiler differs from the one
> >>                 used to build the kernel"; \
> >> -               echo >&2 "  The kernel was built by:
> >> $(CONFIG_CC_VERSION_TEXT)"; \ +               echo >&2 "  The kernel
> >> was built by: "$(CONFIG_CC_VERSION_TEXT)";" \
> >>                 echo >&2 "  You are using:
> >>                 $(CC_VERSION_TEXT)"; \
> >>         fi
> >>
> >> --
> >> MST
>
> I believe the patch proposed by Michael indeed fixes an issue
> introduced by 129ab0d2d9f (Makefile).
>
> Please see the following bug reports:
>
> https://elrepo.org/bugs/view.php?id=1215
> https://elrepo.org/bugs/view.php?id=1214
>
> and this forum post:
>
> https://www.phoronix.com/forums/forum/software/general-linux-open-source/
> 1303986-linux-5-17-rc1-released-a-little-bit-early-but-with-shiny-new-
> features#post1304036
>
> Akemi
>

I left a comment:
https://elrepo.org/bugs/view.php?id=1215


This is a bug in ELRepo.

The upstream kernel is good.

Please do not come back to this over again.




-- 
Best Regards
Masahiro Yamada