Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.debian.kernel > #57644

Re: [PATCH] kbuild: Do not use hyphen in exported variable name

From Ben Hutchings <ben@decadent.org.uk>
Newsgroups linux.debian.kernel
Subject Re: [PATCH] kbuild: Do not use hyphen in exported variable name
Date 2017-04-23 09:30 +0200
Message-ID <tzk81-5ta-7@gated-at.bofh.it> (permalink)
References <txpOy-84k-1@gated-at.bofh.it> <tzjOG-5m6-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

On Sun, 2017-04-23 at 15:47 +0900, Masahiro Yamada wrote:
[...]
> I tested dtbs_install once again by myself, but
> dtbinst-root is exported to the sub make
> and the vendor directories are created correctly.
> 
> 
> I checked the debian's forum you gave
> > References: https://bugs.debian.org/833561
> 
> In there, you mentioned:
> "This looks like a bug in make, but we can at least work around it by
> using a non-hyphenated variable name."
> 
> 
> Does this issue happen on a specific Make version?
> 
> I tested GNU make 3.81, 3.82, 4.0, 4.1, 4.2,
> but I was not hit by the problem.

I don't think this is make version dependent.  I can't reproduce the
issue today with make 4.1.  But I would have been using the same
version in August when I wrote that.

What more can I say?  Clearly the hyphenated variable gets passed to
the sub-make in most cases.  But it's not totally reliable because last
year it wasn't working for us.

> In the last post in the thread, you concluded:
> "We believe that the bug you reported is fixed in the latest version of
> linux, which is due to be installed in the Debian FTP archive."

I didn't write that, it's a standard message generated for bugs marked
as closed in a package changelog. :-)

> If so, why is this patch here?
> How is the dtbs_install procedure different in the Debian package?

This is the patch I applied to the package.

Ben.

-- 
Ben Hutchings
I haven't lost my mind; it's backed up on tape somewhere.

Back to linux.debian.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] kbuild: Do not use hyphen in exported variable name Ben Hutchings <ben@decadent.org.uk> - 2017-04-18 03:10 +0200
  Re: [PATCH] kbuild: Do not use hyphen in exported variable name Sam Ravnborg <sam@ravnborg.org> - 2017-04-18 07:10 +0200
  Re: [PATCH] kbuild: Do not use hyphen in exported variable name Masahiro Yamada <yamada.masahiro@socionext.com> - 2017-04-23 09:10 +0200
    Re: [PATCH] kbuild: Do not use hyphen in exported variable name Ben Hutchings <ben@decadent.org.uk> - 2017-04-23 09:30 +0200
      Re: [PATCH] kbuild: Do not use hyphen in exported variable name Masahiro Yamada <yamada.masahiro@socionext.com> - 2017-04-30 16:40 +0200
        Re: [PATCH] kbuild: Do not use hyphen in exported variable name Ben Hutchings <ben@decadent.org.uk> - 2017-04-30 16:50 +0200
          Re: [PATCH] kbuild: Do not use hyphen in exported variable name Masahiro Yamada <yamada.masahiro@socionext.com> - 2017-05-03 07:10 +0200
          Re: [PATCH] kbuild: Do not use hyphen in exported variable name Ben Hutchings <ben@decadent.org.uk> - 2017-08-19 03:20 +0200
            Re: [PATCH] kbuild: Do not use hyphen in exported variable name Masahiro Yamada <yamada.masahiro@socionext.com> - 2017-08-19 20:10 +0200
              Re: [PATCH] kbuild: Do not use hyphen in exported variable name Ben Hutchings <ben@decadent.org.uk> - 2017-08-19 22:20 +0200

csiph-web