Use non-deprecated paths for battery stuff #24

Closed
svenstaro opened this Issue Jun 8, 2012 · 8 comments

Comments

Projects
None yet
5 participants
@svenstaro
Contributor

svenstaro commented Jun 8, 2012

--battery and --power fail due to "Module dstat_power failed to load. ([Errno 2] No such file or directory: '/proc/acpi/battery/')" which sucks since this dir is /sys/class/power_supply/BAT1 in modern Linux systems.

@svenstaro

This comment has been minimized.

Show comment Hide comment
@svenstaro

svenstaro Jun 8, 2012

Contributor

You should probably just check both paths.

Contributor

svenstaro commented Jun 8, 2012

You should probably just check both paths.

@olek

This comment has been minimized.

Show comment Hide comment
@olek

olek Sep 3, 2012

Same issue here with kernel 3.4.10. Would be nice to have dstat working properly with new kernel.

olek commented Sep 3, 2012

Same issue here with kernel 3.4.10. Would be nice to have dstat working properly with new kernel.

@dagwieers

This comment has been minimized.

Show comment Hide comment
@dagwieers

dagwieers Nov 20, 2012

Owner

Patches are accepted ;-)

Owner

dagwieers commented Nov 20, 2012

Patches are accepted ;-)

@svenstaro

This comment has been minimized.

Show comment Hide comment
@svenstaro

svenstaro Dec 6, 2012

Contributor

Well then, why don't you merge it?

Contributor

svenstaro commented Dec 6, 2012

Well then, why don't you merge it?

@aggroskater

This comment has been minimized.

Show comment Hide comment
@aggroskater

aggroskater Jan 10, 2013

Seconding request for a merge.

Seconding request for a merge.

@svenstaro

This comment has been minimized.

Show comment Hide comment
@svenstaro

svenstaro Feb 26, 2013

Contributor

Yo, merge this please.

Contributor

svenstaro commented Feb 26, 2013

Yo, merge this please.

@svenstaro

This comment has been minimized.

Show comment Hide comment
@svenstaro

svenstaro Sep 13, 2013

Contributor

A few months have gone by and no merge. This is trivial to fix and merge. :/

Contributor

svenstaro commented Sep 13, 2013

A few months have gone by and no merge. This is trivial to fix and merge. :/

@dagwieers dagwieers closed this Sep 13, 2013

@dagwieers dagwieers added the bug label Mar 23, 2016

@dagwieers dagwieers added this to the 0.7.3 milestone May 16, 2016

@q2dg

This comment has been minimized.

Show comment Hide comment
@q2dg

q2dg Aug 31, 2017

I have dstat 0.7.3 installed from official packages in Fedora 26 and I still suffer this error.
I've seen https://github.com/dagwieers/dstat/blob/master/ChangeLog and this request isn't listed on it as an improvement.
Please, it's really annoying

q2dg commented Aug 31, 2017

I have dstat 0.7.3 installed from official packages in Fedora 26 and I still suffer this error.
I've seen https://github.com/dagwieers/dstat/blob/master/ChangeLog and this request isn't listed on it as an improvement.
Please, it's really annoying

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment