Skip to content
This repository has been archived by the owner on Jun 9, 2018. It is now read-only.

New LS Enable method #25

Closed
clburlison opened this issue Apr 12, 2017 · 0 comments
Closed

New LS Enable method #25

clburlison opened this issue Apr 12, 2017 · 0 comments
Assignees
@clburlison
Labels
enhancement

Comments

@clburlison
Copy link
Owner

Use the following

sudo -u _locationd /usr/bin/defaults -currentHost write com.apple.locationd LocationServicesEnabled -bool TRUE

in place of

pinpoint/pkgroot/Library/Application Support/pinpoint/bin/pinpoint

Lines 291 to 294 in a98b680

cmd = ['/usr/bin/defaults', 'write', das_plist,
'LocationServicesEnabled', '-int', '1']
subprocess.check_output(cmd)
os.chown(das_plist, 205, 205)

This removes the need to kill the locationd daemon. It might make other things place nicer as well.
@clburlison clburlison self-assigned this Apr 12, 2017
clburlison added a commit that referenced this issue Nov 21, 2017
@clburlison
feat: Support 10.13
7d54c81 
* This is a major refactor to properly support enabling python on
10.12.4+.
* Removed pylint to use flake8.
* Dropped support for operation systems before 10.12.

Fix: #26
Fix: #25
Fix: #24
@clburlison clburlison mentioned this issue Nov 21, 2017
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@clburlison clburlison

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@clburlison