diff mbox series

build: correct gas --noexecstack check

Message ID e5b54663-77c3-5be1-ea54-3f18246bf77f@suse.com (mailing list archive)
State New, archived
Headers show
Series build: correct gas --noexecstack check | expand

Commit Message

Jan Beulich July 12, 2023, 2:23 p.m. UTC 
The check was missing an escape for the inner $, thus breaking things
in the unlikely event that the underlying assembler doesn't support this
option.

Fixes: 62d22296a95d ("build: silence GNU ld warning about executable stacks")
Signed-off-by: Jan Beulich <jbeulich@suse.com>

Comments

Anthony PERARD Aug. 11, 2023, 2:31 p.m. UTC | #1 
On Wed, Jul 12, 2023 at 04:23:22PM +0200, Jan Beulich wrote:
> The check was missing an escape for the inner $, thus breaking things
> in the unlikely event that the underlying assembler doesn't support this
> option.
> 
> Fixes: 62d22296a95d ("build: silence GNU ld warning about executable stacks")
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Reviewed-by: Anthony PERARD <anthony.perard@citrix.com>

Thanks,
diff mbox series

Patch

--- a/xen/Makefile
+++ b/xen/Makefile
@@ -397,7 +397,7 @@  endif
 
 AFLAGS += -D__ASSEMBLY__
 
-$(call cc-option-add,AFLAGS,CC,-Wa$(comma)--noexecstack)
+$(call cc-option-add,AFLAGS,CC,-Wa$$(comma)--noexecstack)
 
 LDFLAGS-$(call ld-option,--warn-rwx-segments) += --no-warn-rwx-segments