From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yann E. MORIN Date: Mon, 5 Jun 2017 18:42:40 +0200 Subject: [Buildroot] [PATCH] linux: fix name of config variable In-Reply-To: <20170605115409.5e7c55d8@free-electrons.com> References: <20170604094839.23541-1-yann.morin.1998@free.fr> <20170605115409.5e7c55d8@free-electrons.com> Message-ID: <20170605164240.GC3944@scaer> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Thomas, All, On 2017-06-05 11:54 +0200, Thomas Petazzoni spake thusly: > On Sun, 4 Jun 2017 11:48:39 +0200, Yann E. MORIN wrote: > > In 4c10eedc1 (systemd: enable required kernel features), we added > > setting a few required kernel features to ensure systemd works. > > > > However, there was a typo for one of the variables: CONFIG_TMPFS_XATTR > > was written as CONFIG_TMPFS_POSIX_XATTR, which does not exist (and never > > ever existed, at least not since 2.6.12)... > > > > Fixes #9926. > > The patch looks good, but I'm not sure why you say it fixes #9926. It > has been noticed as part of the #9926 discussion, but to me it doesn't > fix this bug. > > Could you clarify? OK, I mis-read the comment in #9926. I does not fix it. Still, the patch is meaningful on its own. Can you just drop the "Fixes #9926" part before applying, please? ;-) Regards, Yann E. MORIN. -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------'