* [PATCH] xen: typo: use ' as apostrophe in grant_table.h
@ 2016-02-24 17:19 Dario Faggioli
2016-06-09 10:29 ` George Dunlap
0 siblings, 1 reply; 2+ messages in thread
From: Dario Faggioli @ 2016-02-24 17:19 UTC (permalink / raw)
To: xen-devel; +Cc: Anthony Perard, Ian Jackson, Ian Campbell, Jan Beulich
If grep 2.23 is installed, build fails like this:
...
mkdir -p compat
grep -v 'DEFINE_XEN_GUEST_HANDLE(long)' public/grant_table.h | \
python /home/SOURCES/xen/xen/xen.git/xen/tools/compat-build-source.py >compat/grant_table.c.new
mv -f compat/grant_table.c.new compat/grant_table.c
gcc ... -o compat/grant_table.i compat/grant_table.c
compat/grant_table.c:33:1: error: unterminated comment
/*
^
compat/grant_table.c:28:0: error: unterminated #ifndef
#ifndef __XEN_PUBLIC_GRANT_TABLE_H__
^
Makefile:62: recipe for target 'compat/grant_table.i' failed
make[3]: *** [compat/grant_table.i] Error 1
rm compat/grant_table.c
make[3]: Leaving directory '/home/SOURCES/xen/xen/xen.git/xen/include'
...
This is because grant_table.h contains this (note the
apostrophe): "granter’s memory", and `grep -v', in version
2.23, stops processing the file (while, for instance,
until 2.22, this was not happening).
Although the above behavior is likely an issue in grep,
(https://debbugs.gnu.org/cgi/bugreport.cgi?bug=22461)
I think we better switch to using " ' " in that line
anyway, as we do basically everywhere else (even in
the same file).
Signed-off-by: Dario Faggioli <dario.faggioli@citrix.com>
---
Cc: Ian Campbell <ian.campbell@citrix.com>
Cc: Ian Jackson <ian.jackson@eu.citrix.com>
Cc: Jan Beulich <jbeulich@suse.com>
Cc: Anthony Perard <anthony.perard@citrix.com>
---
xen/include/public/grant_table.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/xen/include/public/grant_table.h b/xen/include/public/grant_table.h
index e9393fd..e5f04ec 100644
--- a/xen/include/public/grant_table.h
+++ b/xen/include/public/grant_table.h
@@ -43,7 +43,7 @@
* table are identified by grant references. A grant reference is an
* integer, which indexes into the grant table. It acts as a
* capability which the grantee can use to perform operations on the
- * granter’s memory.
+ * granter's memory.
*
* This capability-based system allows shared-memory communications
* between unprivileged domains. A grant reference also encapsulates
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] xen: typo: use ' as apostrophe in grant_table.h
2016-02-24 17:19 [PATCH] xen: typo: use ' as apostrophe in grant_table.h Dario Faggioli
@ 2016-06-09 10:29 ` George Dunlap
0 siblings, 0 replies; 2+ messages in thread
From: George Dunlap @ 2016-06-09 10:29 UTC (permalink / raw)
To: Dario Faggioli
Cc: Anthony Perard, xen-devel, Ian Jackson, Ian Campbell, Jan Beulich
On Wed, Feb 24, 2016 at 5:19 PM, Dario Faggioli
<dario.faggioli@citrix.com> wrote:
> If grep 2.23 is installed, build fails like this:
> ...
> mkdir -p compat
> grep -v 'DEFINE_XEN_GUEST_HANDLE(long)' public/grant_table.h | \
> python /home/SOURCES/xen/xen/xen.git/xen/tools/compat-build-source.py >compat/grant_table.c.new
> mv -f compat/grant_table.c.new compat/grant_table.c
> gcc ... -o compat/grant_table.i compat/grant_table.c
> compat/grant_table.c:33:1: error: unterminated comment
> /*
> ^
> compat/grant_table.c:28:0: error: unterminated #ifndef
> #ifndef __XEN_PUBLIC_GRANT_TABLE_H__
> ^
> Makefile:62: recipe for target 'compat/grant_table.i' failed
> make[3]: *** [compat/grant_table.i] Error 1
> rm compat/grant_table.c
> make[3]: Leaving directory '/home/SOURCES/xen/xen/xen.git/xen/include'
> ...
>
> This is because grant_table.h contains this (note the
> apostrophe): "granter’s memory", and `grep -v', in version
> 2.23, stops processing the file (while, for instance,
> until 2.22, this was not happening).
>
> Although the above behavior is likely an issue in grep,
> (https://debbugs.gnu.org/cgi/bugreport.cgi?bug=22461)
> I think we better switch to using " ' " in that line
> anyway, as we do basically everywhere else (even in
> the same file).
>
> Signed-off-by: Dario Faggioli <dario.faggioli@citrix.com>
...And this needs to be backported, as Xen-4.6 currently won't build
on systems which exhibit this bug (such as Ubuntu Xenial).
-George
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-06-09 10:30 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-24 17:19 [PATCH] xen: typo: use ' as apostrophe in grant_table.h Dario Faggioli
2016-06-09 10:29 ` George Dunlap
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).