Skip to content

Commit

Permalink
Revert "Fix flags for dev build (elastic#31955)" (elastic#32250)
Browse files Browse the repository at this point in the history
This reverts commit a787169.
  • Loading branch information
cmacknz authored Jul 7, 2022
1 parent def59d4 commit ba5d0cb
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion CHANGELOG-developer.next.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@ The list below covers the major changes between 7.0.0-rc2 and main only.
- Avoid panicking in `add_fields` processor when input event.Fields is a nil map. {pull}28219[28219]
- Drop event batch when get HTTP status 413 from Elasticsearch to avoid infinite loop {issue}14350[14350] {pull}29368[29368]
- Allow to use metricbeat for named mssql instances. {issue}24076[24076] {pull}30859[30859]
- Setting DEV=true when running `mage build` now correctly generates binaries without optimisations and with debug symbols {pull}31955[31955]

==== Added

Expand Down
2 changes: 1 addition & 1 deletion dev-tools/mage/build.go
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ func DefaultBuildArgs() BuildArgs {

if DevBuild {
// Disable optimizations (-N) and inlining (-l) for debugging.
args.ExtraFlags = append(args.ExtraFlags, `-gcflags=all=-N -l`)
args.ExtraFlags = append(args.ExtraFlags, `-gcflags`, `"all=-N -l"`)
} else {
// Strip all debug symbols from binary (does not affect Go stack traces).
args.LDFlags = append(args.LDFlags, "-s")
Expand Down

0 comments on commit ba5d0cb

Please sign in to comment.