Bug #8595: mkmf.rb pkg_config modifies $LDFLAGS

• Status: Closed
• Assignee: nobu (Nobuyoshi Nakada)
• Target version: -
• ruby -v: ruby 2.0.0p247 (2013-06-27 revision 41674) [x86_64-linux]
• Backport: 1.9.3: DONTNEED, 2.0.0: DONE

[sorah (Sorah Fukumori)]

=begin
mkmf.rb's pkg_config modifies $LDFLAGS after 2.0.0 (r35605 ?) like the following:

ruby 1.9.3p392 (2013-02-22 revision 39386) [x86_64-linux]

p $LDFLAGS #=> "-L. -rdynamic -Wl,-export-dynamic"
p pkg_config('libxml-2.0') #=> ["-I/usr/include/libxml2 ", "", "-lxml2 "]
p $LDFLAGS #=> "-L. -rdynamic -Wl,-export-dynamic "

ruby 2.0.0p247 (2013-06-27 revision 41674) [x86_64-linux]

p $LDFLAGS #=> "-L. -fstack-protector -rdynamic -Wl,-export-dynamic"
p pkg_config('libxml-2.0') #=> ["-I/usr/include/libxml2", "", "-lxml2"]
p $LDFLAGS #=> "-lxml2 "

2.0.0 + my patch

p $LDFLAGS #=> "-L. -fstack-protector -rdynamic -Wl,-export-dynamic"
p pkg_config('libxml-2.0') #=> ["-I/usr/include/libxml2", "", "-lxml2"]
p $LDFLAGS #=> "-L. -fstack-protector -rdynamic -Wl,-export-dynamic "

attached patch fixes this to 1.9.3 behaviour.

I'll commit this if there's no problem.

=end

[sorah (Sorah Fukumori)]

• Description updated (diff)
• ruby -v changed from ruby 2.0.0p247 (2013-06-27 revision 41674) [x86_64-darwin12.3.0] to ruby 2.0.0p247 (2013-06-27 revision 41674) [x86_64-linux]

[sorah (Sorah Fukumori)]

• File a.patch a.patch added

oops, sorry, forgot to attach the patch

[sorah (Sorah Fukumori)]

• File b.patch b.patch added

Updating patch because I uploaded older one:

• Don't append libs into $LDFLAGS

[sorah (Sorah Fukumori)]

• Status changed from Open to Closed
• % Done changed from 0 to 100

This issue was solved with changeset r41734.
Shota, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

• lib/mkmf.rb (try_config): Fix to not replace $LDFLAGS with $libs
  (1.9.3 behavior) [ruby-core:55752] [Bug #8595]

[sorah (Sorah Fukumori)]

• Backport changed from 1.9.3: UNKNOWN, 2.0.0: UNKNOWN to 1.9.3: DONTNEED, 2.0.0: REQUIRED

[nagachika (Tomoyuki Chikanaga)]

• Backport changed from 1.9.3: DONTNEED, 2.0.0: REQUIRED to 1.9.3: DONTNEED, 2.0.0: DONE