Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Log old python #54

Merged
merged 6 commits into from Oct 14, 2013
Merged
Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Prev

handle lack of json

  • Loading branch information
Mike Blume
Mike Blume committed Oct 10, 2013
commit aeca45ef57acdb39793a95a79e3fce0c5b8c9a5d
@@ -29,7 +29,10 @@
import urllib.request as urllib_request
except ImportError:
import urllib2 as urllib_request
import json
try:
import json
except ImportError:
json = None
import uuid
import base64
import socket
@@ -1465,6 +1468,13 @@ def main():
try:
printMessage("Starting")
options = parse_options()
if json is None:
version = get_python_version_string()
log_msg('''{"python_version": "%s", "subdomain": "%s"}''' %
(version, options.subdomain))

This comment has been minimized.

@MichaelBlume

MichaelBlume Oct 10, 2013
Author Contributor

have to generate json manually here because json is not in Python 2.5 stdlib

printMessage(STR_PYTHON_FAIL_MESSAGE %
(version, MINIMUM_SUPPORTED_PYTHON_VERSION))
sys.exit(-1)
global LOGGLY_QA
LOGGLY_QA = os.environ.get('LOGGLY_QA', '').split()
log({"status":"start", "args": vars(options)})
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.