-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
ref: revise setup.py, remove stray license headers #16174
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
Changes from all commits
5fcc6a6
2a1eaea
6196ed7
609fcee
dcf9ff6
4240972
1947af3
2077e6a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,36 +1,13 @@ | ||
| #!/usr/bin/env python | ||
| """ | ||
| Sentry | ||
| ====== | ||
|
|
||
| Sentry is a realtime event logging and aggregation platform. It specializes | ||
| in monitoring errors and extracting all the information needed to do a proper | ||
| post-mortem without any of the hassle of the standard user feedback loop. | ||
|
|
||
| Sentry is a Server | ||
| ------------------ | ||
|
|
||
| The Sentry package, at its core, is just a simple server and web UI. It will | ||
| handle authentication clients (such as `the Python one | ||
| <https://github.com/getsentry/sentry-python>`_) | ||
| and all of the logic behind storage and aggregation. | ||
|
|
||
| That said, Sentry is not limited to Python. The primary implementation is in | ||
| Python, but it contains a full API for sending events from any language, in | ||
| any application. | ||
|
|
||
| :copyright: (c) 2011-2014 by the Sentry Team, see AUTHORS for more details. | ||
| :license: BSD, see LICENSE for more details. | ||
| """ | ||
| from __future__ import absolute_import | ||
|
|
||
| # if sys.version_info[:2] != (2, 7): | ||
| # print 'Error: Sentry requires Python 2.7' | ||
| # sys.exit(1) | ||
| import sys | ||
|
|
||
| if sys.version_info[:2] != (2, 7): | ||
| sys.exit("Error: Sentry requires Python 2.7.") | ||
|
|
||
| import os | ||
| import os.path | ||
| import sys | ||
|
|
||
| from distutils.command.build import build as BuildCommand | ||
| from setuptools import setup, find_packages | ||
|
|
@@ -39,7 +16,8 @@ | |
|
|
||
| ROOT = os.path.realpath(os.path.join(os.path.dirname(sys.modules["__main__"].__file__))) | ||
|
|
||
| # Add Sentry to path so we can import distutils | ||
| # add sentry to path so we can import distutils | ||
| # XXX: consequentially, this means sentry must be pip installed with --no-use-pep517 | ||
joshuarli marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| sys.path.insert(0, os.path.join(ROOT, "src")) | ||
|
|
||
| from sentry.utils.distutils import ( | ||
|
|
@@ -48,23 +26,9 @@ | |
| BuildJsSdkRegistryCommand, | ||
| ) | ||
|
|
||
| # The version of sentry | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
| VERSION = "10.0.0.dev0" | ||
|
|
||
| # Hack to prevent stupid "TypeError: 'NoneType' object is not callable" error | ||
| # in multiprocessing/util.py _exit_function when running `python | ||
| # setup.py test` (see | ||
| # http://www.eby-sarna.com/pipermail/peak/2010-May/003357.html) | ||
| for m in ("multiprocessing", "billiard"): | ||
| try: | ||
| __import__(m) | ||
| except ImportError: | ||
| pass | ||
|
|
||
| IS_LIGHT_BUILD = os.environ.get("SENTRY_LIGHT_BUILD") == "1" | ||
|
|
||
| # we use pip requirements files to improve Docker layer caching | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think this may stay? At least until we move to poetry?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The future is mysterious and hopefully bright! |
||
|
|
||
|
|
||
| def get_requirements(env): | ||
| with open(u"requirements-{}.txt".format(env)) as fp: | ||
|
|
@@ -134,7 +98,7 @@ def run(self): | |
| packages=find_packages("src"), | ||
| zip_safe=False, | ||
| install_requires=install_requires, | ||
| extras_require={"dev": dev_requires, "postgres": []}, | ||
| extras_require={"dev": dev_requires}, | ||
| cmdclass=cmdclass, | ||
| license="BSL-1.1", | ||
| include_package_data=True, | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,12 +1,3 @@ | ||
| """ | ||
| sentry.conf.urls | ||
| ~~~~~~~~~~~~~~~~ | ||
|
|
||
| These are additional urls used by the Sentry-provided web server | ||
|
|
||
| :copyright: (c) 2012 by the Sentry Team, see AUTHORS for more details. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Let me quote the PR title:
This file says
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. scope creep gets the best of us, haha |
||
| :license: BSD, see LICENSE for more details. | ||
| """ | ||
| from __future__ import absolute_import | ||
|
|
||
| from django.conf import settings | ||
|
|
||

There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😿
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😿