Skip to content
This repository has been archived by the owner on Feb 14, 2020. It is now read-only.

Performance improvements #34

Merged
merged 10 commits into from
Nov 26, 2015

Conversation

childnode
Copy link
Contributor

  1. ioreg -l is very time expensive, better use ioreg -c AppleSmartBattery -r
    see also discussion in http://hints.macworld.com/article.php?story=20100130123935998
  2. disable package manager queries per default

to sum up: startup is reduced by 500ms

@childnode
Copy link
Contributor Author

for testing:

  -bash:$ brew unlink archey && brew install -v https://raw.githubusercontent.com/childnodeContribArch/homebrew/performance/Library/Formula/archey.rb

@childnode childnode closed this Nov 5, 2015
@childnode childnode reopened this Nov 5, 2015
obihann pushed a commit that referenced this pull request Nov 26, 2015
@obihann obihann merged commit 0f9a6ab into obihann:develop Nov 26, 2015
@obihann obihann mentioned this pull request Nov 26, 2015
@obihann obihann self-assigned this Nov 26, 2015
@obihann obihann modified the milestone: 1.5.2 Nov 26, 2015
@obihann
Copy link
Owner

obihann commented Dec 1, 2015

@childnode So it seems that #38 is caused by this, though ioreg -c AppleSmartBattery -r is faster it has broken the test that we do to ensure their is a real battery status, since machines like Mac Pro's don't have batteries. I'm going to see what I can do so that we hopefully can keep this improvement.

childnode added a commit to childnodeContribArch/archey-osx that referenced this pull request Dec 2, 2015
…conditional else case if battery max value is not found resulted in ? instead of NULL
This was referenced Dec 2, 2015
@obihann obihann mentioned this pull request Feb 25, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants