Skip Menu |

This queue is for tickets about the Acpi-Class CPAN distribution.

Report information
The Basics
Id: 83006
Status: resolved
Priority: 0/
Queue: Acpi-Class

People
Owner: Nobody in particular
Requestors: blue [...] thisisnotmyrealemail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: poor choice of names
If this is only ever going to be specific to Linux then you should name this Linux::ACPI. See https://pause.perl.org/pause/query?ACTION=pause_namingmodules and look at other existing modules for naming conventions.
El Lun Ene 28 12:22:53 2013, blue escribió: Show quoted text
> If this is only ever going to be specific to Linux then you should name > this Linux::ACPI. See > https://pause.perl.org/pause/query?ACTION=pause_namingmodules and look > at other existing modules for naming conventions.
Thanks for your observation! I do very much agree with your comment. Nevertheless, there are some other modules, specific to Linux, with similar names. These are: Acpi::Fan, Acpi::Temperature and Acpi::Battery. These three modules use the deprecated /proc/acpi directory to get the information, this is why I have written this module and used the naming that has already been set. At the same time, I understand that mac os x also stores system information in /sys/class tree and, although I do not have access to a mac os x, future developments can take this into consideration. I am open to suggestions on how to proceed. Cheers!
On Mon Jan 28 19:08:56 2013, MIMOSINET wrote: Show quoted text
> El Lun Ene 28 12:22:53 2013, blue escribió:
> > If this is only ever going to be specific to Linux then you should
name Show quoted text
look Show quoted text
> > at other existing modules for naming conventions.
> > Thanks for your observation! I do very much agree with your comment. > Nevertheless, there are some other modules, specific to Linux, with > similar names. These are: > > Acpi::Fan, Acpi::Temperature and Acpi::Battery. > > These three modules use the deprecated /proc/acpi directory to get the > information, this is why I have written this module and used the
naming Show quoted text
> that has already been set. At the same time, I understand that mac os
x Show quoted text
> also stores system information in /sys/class tree and, although I do
not Show quoted text
> have access to a mac os x, future developments can take this into > consideration. > > I am open to suggestions on how to proceed.
After consideration, the name of the module follows the naming of the already existing modules. The right solution would be to rename all the modules at once. Cheers!