Skip to content

Commit

Permalink
Merge pull request #38 from JuliaRobotics/build-release
Browse files Browse the repository at this point in the history
build LCM in release mode
  • Loading branch information
rdeits committed Apr 26, 2018
2 parents ef88f4c + 4877e9c commit a72f203
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions deps/build.jl
Expand Up @@ -39,18 +39,21 @@ end

prefix = joinpath(BinDeps.depsdir(lcm), "usr")

lcm_cmake_arguments = String[]
lcm_cmake_arguments = [
"-DCMAKE_BUILD_TYPE=Release",
"-DLCM_ENABLE_TESTS:BOOL=OFF",
"-DLCM_ENABLE_EXAMPLES:BOOL=OFF"
]

@static if Compat.Sys.isapple()
if Pkg.installed("Homebrew") === nothing
error("Homebrew package not installed, please run Pkg.add(\"Homebrew\")")
end
using Homebrew
provides(Homebrew.HB, "glib", glib, os=:Darwin)
push!(lcm_cmake_arguments,
"-DCMAKE_LIBRARY_PATH=$(joinpath(Pkg.dir("Homebrew"), "deps", "usr", "lib"))")
push!(lcm_cmake_arguments,
"-DCMAKE_LIBRARY_PATH=$(joinpath(Pkg.dir("Homebrew"), "deps", "usr", "lib"))",

This comment has been minimized.

Copy link
@quinnj

quinnj Aug 2, 2018

Note that Pkg.dir(mod::String) is deprecated; see JuliaLang/julia#28310 for pathof(mod::Module). You'll need to bump your Julia requires to julia 0.7-rc1

This comment has been minimized.

Copy link
@rdeits

rdeits Aug 2, 2018

Author Collaborator

Thanks! #48

"-DCMAKE_INCLUDE_PATH=$(joinpath(Pkg.dir("Homebrew"), "deps", "usr", "include"))")

end

provides(Yum,
Expand Down

0 comments on commit a72f203

Please sign in to comment.