micheleciona / aceracpi

Automatically exported from code.google.com/p/aceracpi
0 stars 0 forks source link

compilation failure on debian 4 latest #36

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. tar xvfj acer_acpi-0.11.1.tar.bz2
2. cd acer_acpi-0.11.1
3. make

What is the expected output? What do you see instead?
I get a compilation error:
/usr/src/acer_acpi-0.11.1/wmi-acer.c: In function âwmi_parse_guidâ:
/usr/src/acer_acpi-0.11.1/wmi-acer.c:220: error: âfalseâ undeclared (first
use in this function)
/usr/src/acer_acpi-0.11.1/wmi-acer.c:220: error: (Each undeclared
identifier is reported only once
/usr/src/acer_acpi-0.11.1/wmi-acer.c:220: error: for each function it
appears in.)
/usr/src/acer_acpi-0.11.1/wmi-acer.c:230: error: âtrueâ undeclared (first
use in this function)
make[2]: *** [/usr/src/acer_acpi-0.11.1/wmi-acer.o] Error 1
make[1]: *** [_module_/usr/src/acer_acpi-0.11.1] Error 2
make[1]: Leaving directory `/usr/src/linux-headers-2.6.18-6-686'
make: *** [wmi-acer.ko] Error 2

What version of acer_acpi are you using? On which distribution and kernel
version?

Debian 4 latest

Please provide any additional information below.

Patch for wmi-acer.c that works:
--- wmi-acer.c  2008-03-09 15:06:01.000000000 +0200
+++ wmi-acer.c_org      2008-03-09 15:05:36.000000000 +0200
@@ -55,11 +55,8 @@
 /* Workaround needed for older kernels */
 #ifndef bool
 #define bool int
-#endif
-
-#ifndef true
-#define true    1
-#define false   0
+#define true   1
+#define false  0
 #endif

 MODULE_AUTHOR("Carlos Corbacho");

Original issue reported on code.google.com by akostadi...@gmail.com on 9 Mar 2008 at 1:08

GoogleCodeExporter commented 8 years ago
That patch doesn't apply here - you seem to have diff'ed against your another 
modification to the file you've made, rather than the original.

You are right, however, that there is a bug, and I've fixed up the include/ 
define 
mess on this in Git now, so this will be fixed in the next release (0.11.2, 
whenever I get round to pushing it out, which will probably be sooner rather 
than 
later).

Original comment by cathec...@gmail.com on 9 Mar 2008 at 1:32