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

High Sierra Testing #26

Closed
1JamesRyan opened this issue Jul 18, 2017 · 8 comments
Closed

High Sierra Testing #26

1JamesRyan opened this issue Jul 18, 2017 · 8 comments

Comments

@1JamesRyan
Copy link

Started testing with High Sierra. Changed Line 134
if not 8 <= int(os_vers()) <= 13:

Changed Line 299
if int(os_vers()) >= 13:

Everything is working as intended. Currently testing on a mac mini with High Sierra Beta 3.

@clburlison
Copy link
Owner

That's good to know. Did you allow pinpoint to enable location services and add python to the allow list? or was this a machine that was upgraded from 10.12?

@rogercrawford
Copy link

Just to add, although you're probably aware. Working with Beta 9, same two lines changed as above. Location Services were already enabled, Python had to be allowed access. Can't get Google lookup to work but Apple lookup is working fine. High Sierra was an upgrade from a clean install of Sierra.

@paul-cossey
Copy link

Does anyone have a way of enabling Pinpoint in location services via a script? My googling's only got me as far as enabling location services but not any Apps.

@rogercrawford
Copy link

rogercrawford commented Nov 21, 2017 via email

@clburlison
Copy link
Owner

@paul-cossey Pinpoint should enable python but currently doesn't due to a bug #24

@rogercrawford the Pinpoint code will do and enable Location Services so the above bash script shouldn't be required.

As for this current issue of High Sierra not working I should have an updated preview build uploaded in a few hours. I might need help testing the preview on 10.12 and earlier.

clburlison added a commit that referenced this issue Nov 21, 2017
* 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
Copy link
Owner

Please test the latest pre-release and provide feedback https://github.com/clburlison/pinpoint/releases/tag/v2.0.0beta1

@rogercrawford
Copy link

rogercrawford commented Nov 21, 2017 via email

@paul-cossey
Copy link

Thanks, guys. I'll give the pre-release a test today ands report back.

:-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants