Skip to content

Commit ae43961

Browse files
authored
unvendor wrapt (#1678)
While investigating #1676, I noticed that wrapt now builds binary wheels for a plethora of platforms. We originally opted to vendor wrapt so we could build binary wheels ourselves. This came with the added complexity of compiling/packaging C code. Unvendoring wrapt removes all that complexity, and we benefit of binary wheels for a larger set of platforms than what we provided ourselves.
1 parent 153fdd8 commit ae43961

File tree

18 files changed

+18
-4587
lines changed

18 files changed

+18
-4587
lines changed

Jenkinsfile

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -139,12 +139,10 @@ pipeline {
139139
deleteDir()
140140
unstash 'source'
141141
dir("${BASE_DIR}"){
142-
sh script: 'pip3 install --user cibuildwheel', label: "Installing cibuildwheel"
143-
sh script: 'mkdir wheelhouse', label: "creating wheelhouse"
144-
// skip pypy builds with CIBW_SKIP=pp*
145-
sh script: 'CIBW_SKIP="pp* cp27* cp35*" CIBW_ARCHS_LINUX="x86_64 aarch64" cibuildwheel --platform linux --output-dir wheelhouse; ls -l wheelhouse'
142+
sh script: 'pip3 install --user wheel', label: "Installing wheel"
143+
sh script: 'python3 setup.py bdist_wheel', label: "Building universal wheel"
146144
}
147-
stash allowEmpty: true, name: 'packages', includes: "${BASE_DIR}/wheelhouse/*.whl,${BASE_DIR}/dist/*.tar.gz", useDefaultExcludes: false
145+
stash allowEmpty: true, name: 'packages', includes: "${BASE_DIR}/dist/*.whl,${BASE_DIR}/dist/*.tar.gz", useDefaultExcludes: false
148146
}
149147
}
150148
}

elasticapm/contrib/django/middleware/__init__.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,14 +36,15 @@
3636
from types import FunctionType
3737
from typing import Optional
3838

39+
import wrapt
3940
from django.apps import apps
4041
from django.conf import settings as django_settings
4142
from django.http import HttpRequest, HttpResponse
4243

4344
import elasticapm
4445
from elasticapm.conf import constants
4546
from elasticapm.contrib.django.client import client, get_client
46-
from elasticapm.utils import build_name_with_http_method_prefix, get_name_from_func, wrapt
47+
from elasticapm.utils import build_name_with_http_method_prefix, get_name_from_func
4748

4849
try:
4950
from importlib import import_module

elasticapm/handlers/logging.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,11 @@
3636
import traceback
3737
import warnings
3838

39+
import wrapt
40+
3941
from elasticapm import get_client
4042
from elasticapm.base import Client
4143
from elasticapm.traces import execution_context
42-
from elasticapm.utils import wrapt
4344
from elasticapm.utils.stacks import iter_stack_frames
4445

4546

elasticapm/instrumentation/packages/base.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,9 @@
3131
import functools
3232
import os
3333

34+
import wrapt
35+
3436
from elasticapm.traces import execution_context
35-
from elasticapm.utils import wrapt
3637
from elasticapm.utils.logging import get_logger
3738

3839
logger = get_logger("elasticapm.instrument")

elasticapm/instrumentation/packages/dbapi2.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,10 @@
3535

3636
import re
3737

38+
import wrapt
39+
3840
from elasticapm.instrumentation.packages.base import AbstractInstrumentedModule
3941
from elasticapm.traces import capture_span
40-
from elasticapm.utils import wrapt
4142
from elasticapm.utils.encoding import force_text, shorten
4243

4344

elasticapm/utils/wrapt/LICENSE

Lines changed: 0 additions & 24 deletions
This file was deleted.

elasticapm/utils/wrapt/__init__.py

Lines changed: 0 additions & 19 deletions
This file was deleted.

0 commit comments

Comments
 (0)