Le 3 mars 08 à 12:26, Quentin Mathé a écrit : > Le 25 févr. 08 à 12:28, Lennart Melzer a écrit : > >> Hi there, >> >> I replaced my former code with the implementation of libacpi, which >> takes the same steps (reads some files, splits lines by spaces, >> converts values) for me. I also cleaned up the APM code so that both >> ways (APM and ACPI) call a seperate function to return the status >> from the current percentage of battery level >> >> As I'm not yet an etoile-developer I attach the changed code here >> for you to review. If there are no blatant mistakes, I can commit it >> to the trunk. >> >> For this code to compile/link I installed libacpi/libacpi-dev (or >> the equivalent package on other linux Distributions, I' m using >> Debian right now) and added -lacpi to the SystemConfig Makefile. >> >> Concerning the mentioned problem of the location of acpi >> information: Iibacpi hardcodes the location /proc/acpi and uses >> that. Maybe one has to override that behaviour when using customized >> kernels. If anyone is having problems with that behaviour, please >> tell me. > > Hi Lennart, > > I just reviewed your patch. I haven't tested it yet, but it looks good > so I'll commit it. > If you are interested to work on SystemConfig, let me know :-)
I finally applied your patch. Sorry for the delay. > Author: qmathe > Date: Tue Mar 25 10:23:26 2008 > New Revision: 2970 > > URL: http://svn.gna.org/viewcvs/etoile?rev=2970&view=rev > Log: > Applied a patch by Lennart Melzer that implements ACPI support in > SCPower by relying on libacpi. The formatting and the code > organization of the patch have been reworked a bit. ACPI support is > turned off by default for now. > Author: qmathe > Date: Tue Mar 25 10:23:32 2008 > New Revision: 2971 > > URL: http://svn.gna.org/viewcvs/etoile?rev=2971&view=rev > Log: > Necessary changes to compile and link with ACPI support. ACPI is > still turned off by default. Made this choice, mostly because > libacpi package isn't yet available on Ubuntu (next Ubuntu release > Hardy Heron will include it though). Listed all authors in README To compile SystemConfig with ACPI support, type: make LIBACPI=yes (explained in INSTALL file) Cheers, Quentin. . _______________________________________________ Etoile-dev mailing list [email protected] https://mail.gna.org/listinfo/etoile-dev
