Skip to content

Commit

Permalink
pybind: Remove unneeded suppression of #warnings (Resolves RobotLoc…
Browse files Browse the repository at this point in the history
…omotion#8437)

This was done before because our fork of `pybind11` included NumPy headers
to support dtype=object, but did not add the directive to not use
deprecated features. They should not be necessary now.
  • Loading branch information
EricCousineau-TRI committed Apr 13, 2018
1 parent 750ea03 commit 81f94c1
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions tools/skylark/pybind.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ _PY_VERSION = "2.7"
def _drake_pybind_cc_binary(
name,
srcs = [],
copts = [],
deps = [],
visibility = None,
testonly = None):
Expand All @@ -38,12 +37,6 @@ def _drake_pybind_cc_binary(
name = name,
# This is how you tell Bazel to link in a shared library.
srcs = srcs + ["//tools/install/libdrake:libdrake.so"],
# These copts are per pybind11 deficiencies.
copts = [
"-Wno-#warnings",
"-Wno-cpp",
"-Wno-unknown-warning-option",
] + copts,
# This is how you tell Bazel to create a shared library.
linkshared = 1,
linkstatic = 1,
Expand All @@ -66,7 +59,6 @@ def drake_pybind_library(
name,
cc_srcs = [],
cc_deps = [],
copts = [],
cc_so_name = None,
package_info = None,
py_srcs = [],
Expand Down

0 comments on commit 81f94c1

Please sign in to comment.