lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <YAxRbe4DmEbM0feC@HP> Date: Sat, 23 Jan 2021 17:40:13 +0100 From: Robert Karszniewicz <avoidr@...teo.de> To: Masahiro Yamada <masahiroy@...nel.org> Cc: Robert Karszniewicz <r.karszniewicz@...tec.de>, Kees Cook <keescook@...omium.org>, Michal Marek <michal.lkml@...kovi.net>, Jonathan Corbet <corbet@....net>, Philipp Rudo <prudo@...ux.ibm.com>, linux-hardening@...r.kernel.org, Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>, "open list:DOCUMENTATION" <linux-doc@...r.kernel.org> Subject: Re: [PATCH] Documentation/Kbuild: Remove references to gcc-plugin.sh On Sat, Jan 23, 2021 at 10:17:59PM +0900, Masahiro Yamada wrote: > On Sat, Jan 23, 2021 at 3:19 AM Robert Karszniewicz > <r.karszniewicz@...tec.de> wrote: > > > > gcc-plugin.sh has been removed in commit > > 1e860048c53e ("gcc-plugins: simplify GCC plugin-dev capability test"). > > > > Signed-off-by: Robert Karszniewicz <r.karszniewicz@...tec.de> > > --- > > Documentation/kbuild/gcc-plugins.rst | 6 ------ > > scripts/dummy-tools/gcc | 15 --------------- > > 2 files changed, 21 deletions(-) > > > > [...] > > diff --git a/scripts/dummy-tools/gcc b/scripts/dummy-tools/gcc > > index 33487e99d83e..a484de576e6f 100755 > > --- a/scripts/dummy-tools/gcc > > +++ b/scripts/dummy-tools/gcc > > @@ -74,18 +74,3 @@ if arg_contain -S "$@"; then > > exit 0 > > fi > > fi > > - > > -# For scripts/gcc-plugin.sh > > -if arg_contain -print-file-name=plugin "$@"; then > > - plugin_dir=$(mktemp -d) > > - > > - sed -n 's/.*#include "\(.*\)"/\1/p' $(dirname $0)/../gcc-plugins/gcc-common.h | > > - while read header > > - do > > - mkdir -p $plugin_dir/include/$(dirname $header) > > - touch $plugin_dir/include/$header > > - done > > - > > - echo $plugin_dir > > - exit 0 > > -fi > > > Well, it is wrong to remove this hunk entirely. > It would break the ability to enable GCC_PLUGINS. > > I will simplify this part. > https://lore.kernel.org/patchwork/patch/1369968/ > > > I took the only change to Documentation/kbuild/gcc-plugins.rst > from your patch but this document contains more stale info. > > https://lore.kernel.org/patchwork/patch/1370008/ I see, thanks for letting me know, Robert
Powered by blists - more mailing lists