[Git][NTPsec/ntpsec][master] 2 commits: sys.platform is "sunos5" on Solaris, not "Solaris"
Matt Selsky
gitlab at mg.gitlab.com
Fri Dec 30 16:05:16 UTC 2016
Matt Selsky pushed to branch master at NTPsec / ntpsec
Commits:
301cf5d2 by Matt Selsky at 2016-12-30T04:44:38-05:00
sys.platform is "sunos5" on Solaris, not "Solaris"
- - - - -
9d6027b1 by Matt Selsky at 2016-12-30T04:50:15-05:00
No need to import readline in ntpq itself. Cmd already does it for us.
- - - - -
2 changed files:
- ntpclients/ntpq
- wafhelpers/configure.py
Changes:
=====================================
ntpclients/ntpq
=====================================
--- a/ntpclients/ntpq
+++ b/ntpclients/ntpq
@@ -25,13 +25,6 @@ except ImportError as e:
sys.stderr.write("%s\n" % e)
sys.exit(1)
-# This import only works on Unixes. The intention is to enable
-# Ctrl-P, Ctrl-N, and friends in Cmd.
-try:
- import readline
-except ImportError:
- pass
-
version = ntp.util.stdversion()
# General notes on Python 2/3 compatibility:
=====================================
wafhelpers/configure.py
=====================================
--- a/wafhelpers/configure.py
+++ b/wafhelpers/configure.py
@@ -382,7 +382,7 @@ def cmd_configure(ctx, config):
ctx.define("HAVE_PPSAPI", 1, comment="Enable the PPS API")
# Check for Solaris capabilities
- if ctx.get_define("HAVE_PRIV_H") and sys.platform == "Solaris":
+ if ctx.get_define("HAVE_PRIV_H") and sys.platform.startswith("sunos"):
ctx.define("HAVE_SOLARIS_PRIVS", 1, comment="Enable Solaris Privileges (Solaris only)")
from wafhelpers.check_sockaddr import check_sockaddr
View it on GitLab: https://gitlab.com/NTPsec/ntpsec/compare/1d834187d4fceb8f10e1e2a82a788d099cc4b4e2...9d6027b15eab3fa3aaa5f430d7fa02e00ee03a70
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ntpsec.org/pipermail/vc/attachments/20161230/bf70e6e0/attachment.html>
More information about the vc
mailing list