Skip to content

Commit

Permalink
Bug 1523312 - do not export msgpack, as it has no callers out side of…
Browse files Browse the repository at this point in the history
… libxul r=dmajor

Includes a drive-by fix for two unused `DEFINES`.

Differential Revision: https://phabricator.services.mozilla.com/D17859

--HG--
extra : moz-landing-system : lando
  • Loading branch information
rhelmer committed Feb 27, 2019
1 parent 228ecaf commit 30e3d84
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion third_party/msgpack/moz.build
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
# License, v. 2.0. If a copy of the MPL was not distributed with this
# file, You can obtain one at http://mozilla.org/MPL/2.0/.

DEFINES['MSGPACK_BUILD_LIBRARY'] = True
DEFINES['MSGPACK_DLLEXPORT'] = ''

LOCAL_INCLUDES += [
'include',
Expand Down
2 changes: 0 additions & 2 deletions third_party/prio/moz.build
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@
# License, v. 2.0. If a copy of the MPL was not distributed with this
# file, You can obtain one at http://mozilla.org/MPL/2.0/.

DEFINES['PRIO_BUILD_LIBRARY'] = True

LOCAL_INCLUDES += [
'/security/nss/lib/freebl/mpi',
'/third_party/msgpack/include',
Expand Down

0 comments on commit 30e3d84

Please sign in to comment.