From: Florian Weimer <fweimer@redhat.com>
To: Shahab Vahedi <Shahab.Vahedi@synopsys.com>
Cc: libc-alpha@sourceware.org,
Claudiu Zissulescu <Claudiu.Zissulescu@synopsys.com>,
arcml <linux-snps-arc@lists.infradead.org>
Subject: Re: [PATCH v3 1/2] scripts: Add "|" operator support to glibcpp's parsing
Date: Sun, 27 Nov 2022 19:21:49 +0100 [thread overview]
Message-ID: <87h6yk1cn6.fsf@oldenburg.str.redhat.com> (raw)
In-Reply-To: <20221127173825.3106-2-shahab@synopsys.com> (Shahab Vahedi's message of "Sun, 27 Nov 2022 18:38:24 +0100")
* Shahab Vahedi:
> From the tests point of view, this is a necessary step for another
> patch [1] and allows parsing macros such as "#define A | B". Without
> it, a few tests [2] choke when the other patch [1] is applied:
>
> /src/glibc/scripts/../elf/elf.h:4167: error: uninterpretable macro
> token sequence: ( EF_ARC_MACH_MSK | EF_ARC_OSABI_MSK )
> Traceback (most recent call last):
> File "/src/glibc/elf/tst-glibcelf.py", line 23, in <module>
> import glibcelf
> File "/src/glibc/scripts/glibcelf.py", line 226, in <module>
> _elf_h = _parse_elf_h()
> ^^^^^^^^^^^^^^
> File "/src/glibc/scripts/glibcelf.py", line 223, in _parse_elf_h
> raise IOError('parse error in elf.h')
> OSError: parse error in elf.h
>
> [1] ARC: update definitions in elf/elf.h
> https://sourceware.org/pipermail/libc-alpha/2022-November/143503.html
>
> [2]
> tst-glibcelf, tst-relro-ldso, and tst-relro-libc
>
> Signed-off-by: Shahab Vahedi <shahab@synopsys.com>
This okay and can go in separately, thanks.
Reviewed-by: Florian Weimer <fweimer@redhat.com>
Florian
_______________________________________________
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc
next prev parent reply other threads:[~2022-11-27 18:22 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20221118131542.23188-1-shahab@synopsys.com>
2022-11-19 0:34 ` [PATCH] ARC: update definitions in elf/elf.h Vineet Gupta
2022-11-21 13:07 ` Shahab Vahedi
2022-11-21 13:06 ` [PATCH v2] " Shahab Vahedi
2022-11-21 14:30 ` Adhemerval Zanella Netto
2022-11-23 4:13 ` Vineet Gupta
2022-11-27 17:38 ` [PATCH v3 0/2] Update ARC " Shahab Vahedi
2022-11-27 17:38 ` [PATCH v3 1/2] scripts: Add "|" operator support to glibcpp's parsing Shahab Vahedi
2022-11-27 18:21 ` Florian Weimer [this message]
2022-11-30 2:11 ` Vineet Gupta
2022-11-27 17:38 ` [PATCH v3 2/2] ARC: update definitions in elf/elf.h Shahab Vahedi
2022-11-30 2:11 ` Vineet Gupta
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87h6yk1cn6.fsf@oldenburg.str.redhat.com \
--to=fweimer@redhat.com \
--cc=Claudiu.Zissulescu@synopsys.com \
--cc=Shahab.Vahedi@synopsys.com \
--cc=libc-alpha@sourceware.org \
--cc=linux-snps-arc@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.