Skip to content
This repository was archived by the owner on Mar 26, 2022. It is now read-only.

Support for High Sierra Caching Service#50

Open
darkmind-io wants to merge 7 commits intoerikng:masterfrom
darkmind-io:master
Open

Support for High Sierra Caching Service#50
darkmind-io wants to merge 7 commits intoerikng:masterfrom
darkmind-io:master

Conversation

@darkmind-io
Copy link
Copy Markdown

Configured to support High Sierra and Unified Logging. Removed Server.app references.

Had to disable all configuration commands, but added in the Unified logging
Spelling
Removed some debugging print statements and added Language to the Config check
@MarcProe
Copy link
Copy Markdown

MarcProe commented Jan 29, 2018

Merged this on my branch, works. Thanks!

@erikng
Copy link
Copy Markdown
Owner

erikng commented Apr 7, 2018

@darkmind-io can you please look at #47

@bedris
Copy link
Copy Markdown

bedris commented Oct 22, 2018

I am getting the following error and not sure where it's coming from.
Traceback (most recent call last):
File "/users/name/cacher.py", line 843, in
main()
File "/users/name/cacher.py", line 831, in main
cacherdata = cacher(rawLog.readlines(), targetDate, friendlyNames)
File "/users/namecacher.py", line 291, in cacher
totalbytesserved.append(bytes_served)
UnboundLocalError: local variable 'bytes_served' referenced before assignment

@patgmac
Copy link
Copy Markdown

patgmac commented Dec 11, 2018

This PR has been working well for me. Any plans to merge?

@erikng
Copy link
Copy Markdown
Owner

erikng commented Dec 11, 2018

It has conflicts so I can't. If the author wants to update this or of if someone wants to send a PR for the fixes, I can merge it.

I don't have caching servers at my company or time to test and work on this, so help is appreciated.

@macshome
Copy link
Copy Markdown

macshome commented Feb 8, 2019

Do you want help testing and merging this?

@erikng
Copy link
Copy Markdown
Owner

erikng commented Feb 8, 2019

Absolutely @macshome - surprised you're interested in this project but I will
Absolutely take your help!

@macshome
Copy link
Copy Markdown

Cool. I’ll look to find some time to take a look.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants