Skip to content

Commit

Permalink
Fix drcachesim installation missing drconfiglib
Browse files Browse the repository at this point in the history
The drcachesim installation accidentally installed drinjectlib twice rather
than installing it and drconfiglib.

Review-URL: https://codereview.appspot.com/319280043
  • Loading branch information
derekbruening committed Feb 3, 2017
1 parent ad88186 commit 7f9bb9e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions clients/drcachesim/CMakeLists.txt
@@ -1,5 +1,5 @@
# **********************************************************
# Copyright (c) 2015-2016 Google, Inc. All rights reserved.
# Copyright (c) 2015-2017 Google, Inc. All rights reserved.
# **********************************************************

# Redistribution and use in source and binary forms, with or without
Expand Down Expand Up @@ -221,7 +221,7 @@ if (WIN32)
# drcachesim needs these dlls (i#1737 would eliminate this)
get_target_property(injectlib_loc drinjectlib LOCATION${location_suffix})
DR_install(FILES "${injectlib_loc}" DESTINATION "${INSTALL_CLIENTS_BIN}")
get_target_property(configlib_loc drinjectlib LOCATION${location_suffix})
get_target_property(configlib_loc drconfiglib LOCATION${location_suffix})
DR_install(FILES "${configlib_loc}" DESTINATION "${INSTALL_CLIENTS_BIN}")
add_custom_command(TARGET drcachesim POST_BUILD
COMMAND ${CMAKE_COMMAND} ARGS -E copy ${DR_LIBRARY_BASE_DIRECTORY}/drinjectlib.dll
Expand Down

0 comments on commit 7f9bb9e

Please sign in to comment.