Re: [Xen-devel] [PATCH] hvmloader: fix building error
On 04/08/2015 04:46 PM, Andrew Cooper wrote: On 08/04/15 08:33, Wen Congyang wrote: On 04/08/2015 03:28 PM, Olaf Hering wrote: On Wed, Apr 08, Wen Congyang wrote: On 04/08/2015 03:21 PM, Olaf Hering wrote: On Wed, Apr 08, Wen Congyang wrote: commit b9245b75 introduces a building error: make[1]: Entering directory `/root/work/xen/tools/firmware/hvmloader' gcc -O1 -fno-omit-frame-pointer -m32 -march=i686 -g -fno-strict-aliasing -std=gnu99 -Wall -Wstrict-prototypes -Wdeclaration-after-statement -O0 -g3 -D__XEN_TOOLS__ -MMD -MF .smbios.o.d -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -fno-optimize-sibling-calls -mno-tls-direct-seg-refs -Werror -fno-stack-protector -fno-exceptions -fno-builtin -msoft-float -I/root/work/xen/tools/firmware/hvmloader/../../../tools/include -DENABLE_ROMBIOS -DENABLE_SEABIOS -D__SMBIOS_DATE__=04/08/2015 -c -o smbios.o smbios.c smbios.c:384:46: error: invalid digit 8 in octal constant smbios.c:792:46: error: invalid digit 8 in octal constant make[1]: *** [smbios.o] Error 1 Oh, I did not spot that because in my builds the octal number was always valid. But you are right, it has to be a string constant. Do you run 'make -C tools/ clean' before building? No, I dont. I do 'git clean -dffx', often. I guess if you do it, you will also find this problem. It will entirely depend on whether you attempt to compile it on an 8th or 9th of a month, or in August or September. Outside of that that window, the resulting number will parse correctly and not fail to compile. Yes, outside of that window, the problem is hidden. For the fix itself, Acked-by: Andrew Cooper andrew.coop...@citrix.com . ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH] hvmloader: fix building error
On 08/04/15 11:46, Ian Jackson wrote: Wen Congyang writes ([Xen-devel] [PATCH] hvmloader: fix building error): commit b9245b75 introduces a building error: ... smbios.c:384:46: error: invalid digit 8 in octal constant smbios.c:792:46: error: invalid digit 8 in octal constant make[1]: *** [smbios.o] Error 1 Thanks for this fix, Acked-by: Ian Jackson ian.jack...@eu.citrix.com Committed-by: Ian Jackson ian.jack...@eu.citrix.com (CC'ing Andrew Cooper who also requested this fix and Jan Beulich who acked and committed b9245b75.) There is an acked-by me on the bottom of the thread. ~Andrew ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH] hvmloader: fix building error
Wen Congyang writes ([Xen-devel] [PATCH] hvmloader: fix building error): commit b9245b75 introduces a building error: ... smbios.c:384:46: error: invalid digit 8 in octal constant smbios.c:792:46: error: invalid digit 8 in octal constant make[1]: *** [smbios.o] Error 1 Thanks for this fix, Acked-by: Ian Jackson ian.jack...@eu.citrix.com Committed-by: Ian Jackson ian.jack...@eu.citrix.com (CC'ing Andrew Cooper who also requested this fix and Jan Beulich who acked and committed b9245b75.) Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH] hvmloader: fix building error
On 04/08/2015 03:28 PM, Olaf Hering wrote: On Wed, Apr 08, Wen Congyang wrote: On 04/08/2015 03:21 PM, Olaf Hering wrote: On Wed, Apr 08, Wen Congyang wrote: commit b9245b75 introduces a building error: make[1]: Entering directory `/root/work/xen/tools/firmware/hvmloader' gcc -O1 -fno-omit-frame-pointer -m32 -march=i686 -g -fno-strict-aliasing -std=gnu99 -Wall -Wstrict-prototypes -Wdeclaration-after-statement -O0 -g3 -D__XEN_TOOLS__ -MMD -MF .smbios.o.d -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -fno-optimize-sibling-calls -mno-tls-direct-seg-refs -Werror -fno-stack-protector -fno-exceptions -fno-builtin -msoft-float -I/root/work/xen/tools/firmware/hvmloader/../../../tools/include -DENABLE_ROMBIOS -DENABLE_SEABIOS -D__SMBIOS_DATE__=04/08/2015 -c -o smbios.o smbios.c smbios.c:384:46: error: invalid digit 8 in octal constant smbios.c:792:46: error: invalid digit 8 in octal constant make[1]: *** [smbios.o] Error 1 Oh, I did not spot that because in my builds the octal number was always valid. But you are right, it has to be a string constant. Do you run 'make -C tools/ clean' before building? No, I dont. I do 'git clean -dffx', often. I guess if you do it, you will also find this problem. Thanks Wen Congyang Olaf . ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH] hvmloader: fix building error
On Wed, Apr 08, Wen Congyang wrote: On 04/08/2015 03:21 PM, Olaf Hering wrote: On Wed, Apr 08, Wen Congyang wrote: commit b9245b75 introduces a building error: make[1]: Entering directory `/root/work/xen/tools/firmware/hvmloader' gcc -O1 -fno-omit-frame-pointer -m32 -march=i686 -g -fno-strict-aliasing -std=gnu99 -Wall -Wstrict-prototypes -Wdeclaration-after-statement -O0 -g3 -D__XEN_TOOLS__ -MMD -MF .smbios.o.d -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -fno-optimize-sibling-calls -mno-tls-direct-seg-refs -Werror -fno-stack-protector -fno-exceptions -fno-builtin -msoft-float -I/root/work/xen/tools/firmware/hvmloader/../../../tools/include -DENABLE_ROMBIOS -DENABLE_SEABIOS -D__SMBIOS_DATE__=04/08/2015 -c -o smbios.o smbios.c smbios.c:384:46: error: invalid digit 8 in octal constant smbios.c:792:46: error: invalid digit 8 in octal constant make[1]: *** [smbios.o] Error 1 Oh, I did not spot that because in my builds the octal number was always valid. But you are right, it has to be a string constant. Do you run 'make -C tools/ clean' before building? No, I dont. I do 'git clean -dffx', often. Olaf ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH] hvmloader: fix building error
On 04/08/2015 03:21 PM, Olaf Hering wrote: On Wed, Apr 08, Wen Congyang wrote: commit b9245b75 introduces a building error: make[1]: Entering directory `/root/work/xen/tools/firmware/hvmloader' gcc -O1 -fno-omit-frame-pointer -m32 -march=i686 -g -fno-strict-aliasing -std=gnu99 -Wall -Wstrict-prototypes -Wdeclaration-after-statement -O0 -g3 -D__XEN_TOOLS__ -MMD -MF .smbios.o.d -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -fno-optimize-sibling-calls -mno-tls-direct-seg-refs -Werror -fno-stack-protector -fno-exceptions -fno-builtin -msoft-float -I/root/work/xen/tools/firmware/hvmloader/../../../tools/include -DENABLE_ROMBIOS -DENABLE_SEABIOS -D__SMBIOS_DATE__=04/08/2015 -c -o smbios.o smbios.c smbios.c:384:46: error: invalid digit 8 in octal constant smbios.c:792:46: error: invalid digit 8 in octal constant make[1]: *** [smbios.o] Error 1 Oh, I did not spot that because in my builds the octal number was always valid. But you are right, it has to be a string constant. Do you run 'make -C tools/ clean' before building? Thanks Wen Congyang Olaf ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH] hvmloader: fix building error
On 08/04/15 08:33, Wen Congyang wrote: On 04/08/2015 03:28 PM, Olaf Hering wrote: On Wed, Apr 08, Wen Congyang wrote: On 04/08/2015 03:21 PM, Olaf Hering wrote: On Wed, Apr 08, Wen Congyang wrote: commit b9245b75 introduces a building error: make[1]: Entering directory `/root/work/xen/tools/firmware/hvmloader' gcc -O1 -fno-omit-frame-pointer -m32 -march=i686 -g -fno-strict-aliasing -std=gnu99 -Wall -Wstrict-prototypes -Wdeclaration-after-statement -O0 -g3 -D__XEN_TOOLS__ -MMD -MF .smbios.o.d -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -fno-optimize-sibling-calls -mno-tls-direct-seg-refs -Werror -fno-stack-protector -fno-exceptions -fno-builtin -msoft-float -I/root/work/xen/tools/firmware/hvmloader/../../../tools/include -DENABLE_ROMBIOS -DENABLE_SEABIOS -D__SMBIOS_DATE__=04/08/2015 -c -o smbios.o smbios.c smbios.c:384:46: error: invalid digit 8 in octal constant smbios.c:792:46: error: invalid digit 8 in octal constant make[1]: *** [smbios.o] Error 1 Oh, I did not spot that because in my builds the octal number was always valid. But you are right, it has to be a string constant. Do you run 'make -C tools/ clean' before building? No, I dont. I do 'git clean -dffx', often. I guess if you do it, you will also find this problem. It will entirely depend on whether you attempt to compile it on an 8th or 9th of a month, or in August or September. Outside of that that window, the resulting number will parse correctly and not fail to compile. For the fix itself, Acked-by: Andrew Cooper andrew.coop...@citrix.com ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel