Skip to content

Commit

Permalink
Only setup native debugging symbols by default
Browse files Browse the repository at this point in the history
Only add -ggdb3 when we build a debug version to provide additional
debug information.
  • Loading branch information
dbussink committed Jul 9, 2012
1 parent 4085f11 commit 3564fa1
Show file tree
Hide file tree
Showing 7 changed files with 19 additions and 14 deletions.
11 changes: 9 additions & 2 deletions lib/ext/melbourne/Rakefile
Original file line number Diff line number Diff line change
Expand Up @@ -61,8 +61,15 @@ when ["build:build"]
end
else
puts "Building bootstrap Melbourne for Rubinius"
add_cflag "-ggdb3 -O2"
add_cxxflag "-ggdb3 -O2"
add_cflag "-g"
add_cxxflag "-g"
if ENV['DEV']
add_cflag "-O0 -ggdb3"
add_cxxflag "-O0 -ggdb3"
else
add_cflag "-O2"
add_cxxflag "-O2"
end
add_include_dir RbConfig::CONFIG["rubyhdrdir"]
end
end
Expand Down
2 changes: 0 additions & 2 deletions lib/ext/melbourne/extconf.rb
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@ def add_lib(name)
#have_library("ptr_array")
#have_library("cchash")

$CFLAGS += " -ggdb3"

# Courtesy of EventMachine. Thank you EventMachine and tmm1 !
case RUBY_PLATFORM
when /mswin32/, /mingw32/, /bccwin32/
Expand Down
4 changes: 2 additions & 2 deletions lib/rbconfig.rb
Original file line number Diff line number Diff line change
Expand Up @@ -121,10 +121,10 @@ module RbConfig
# since we hardcode using gcc, and this flag is only
# used by mkmf to compile extensions, be sure PIC is in
# there
CONFIG["CFLAGS"] = "-ggdb3 -fPIC"
CONFIG["CFLAGS"] = "-g -fPIC"
CONFIG["LDFLAGS"] = ""
if ENV['DEV']
CONFIG["CFLAGS"] << " -O0"
CONFIG["CFLAGS"] << " -O0 -ggdb3"
else
CONFIG["CFLAGS"] << " -O2"
end
Expand Down
4 changes: 2 additions & 2 deletions rakelib/blueprint.rb
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
gcc.cflags << "-mno-omit-leaf-frame-pointer"

gcc.cflags << "-Wno-unused-function"
gcc.cflags << "-g -ggdb3 -Werror"
gcc.cflags << "-g -Werror"
gcc.cflags << "-DRBX_PROFILER"
gcc.cflags << "-D__STDC_LIMIT_MACROS -D__STDC_CONSTANT_MACROS"
gcc.cflags << "-D_LARGEFILE_SOURCE"
Expand All @@ -25,7 +25,7 @@
gcc.cflags << Rubinius::BUILD_CONFIG[:user_cflags]

if ENV['DEV']
gcc.cflags << "-O0"
gcc.cflags << "-O0 -ggdb3"
gcc.mtime_only = true
else
gcc.cflags << "-O2"
Expand Down
8 changes: 4 additions & 4 deletions rakelib/ext_helper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -138,11 +138,11 @@ def include19_dir
end

def add_rbx_capi
add_cflag "-g -ggdb3"
add_cxxflag "-g -ggdb3"
add_cflag "-g"
add_cxxflag "-g"
if ENV['DEV']
add_cflag "-O0"
add_cxxflag "-O0"
add_cflag "-O0 -ggdb3"
add_cxxflag "-O0 -ggdb3"
else
add_cflag "-O2"
add_cxxflag "-O2"
Expand Down
2 changes: 1 addition & 1 deletion vendor/libgdtoa/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ MARCH=$(shell uname -m)
OSVER=$(shell uname -r)

WARNINGS = -Wall
DEBUG = -g -ggdb3
DEBUG = -g

CFLAGS = $(WARNINGS) $(DEBUG) -fno-strict-aliasing
NAME=libgdtoa
Expand Down
2 changes: 1 addition & 1 deletion vendor/libtommath/makefile
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ endif
ifndef IGNORE_SPEED

#for speed
CFLAGS += -O2 -funroll-loops -ggdb3
CFLAGS += -O2 -funroll-loops -g

#for size
#CFLAGS += -Os
Expand Down

0 comments on commit 3564fa1

Please sign in to comment.