Bug #1876

ksh93 almost certainly doesn't really need to be DWARF

Added by Rich Lowe over 2 years ago. Updated over 2 years ago.

Status:Resolved Start date:2011-12-10
Priority:Normal Due date:
Assignee:Rich Lowe % Done:

100%

Category:lib - userland libraries Spent time: -
Target version:-
Difficulty:Bite-size Tags:

Description

usr/src/Makefile.ast forces ksh93 to use DWARF because "We need this for C99/VLA support", experimentally, that's not true, and allowing it to default (to stabs, right now), worked fine.

This should be verified.

I should perhaps explain that we currently use stabs, always, when building with Sun Studio not because we are intransigent, stuck in the past, or ornery, but because Sun Studio generates broken DWARF, and thus using it arbitrarily means that you're placing a bet that the DWARF it generates for you will be ok. Even should you win that bet, there's a good chance the first person to lose it will be pretty angry.

History

Updated by Rich Lowe over 2 years ago

Further research has discovered two bugs that may have necessitated this, one of which is long since fixed, the other of which applies to a compiler which we do not use, and didn't exist at the time the comment was written (it has, also, since been fixed).

Updated by Rich Lowe over 2 years ago

  • Assignee set to Rich Lowe
  • Difficulty changed from Medium to Bite-size
  • Tags deleted (needs-triage)

Updated by Rich Lowe over 2 years ago

  • Status changed from New to In Progress

Updated by Yuri Pankov over 2 years ago

We almost certainly don't need ksh93, wonder if smartos did already drop it.

Updated by Rich Lowe over 2 years ago

They didn't, and actually removing it is a lot harder than you're giving credit to.

Moving it to nexenta's illumos-userland tree, and oi-build/illumos-extra for the rest of us seems saner. But that's another bug, feel free to file it.

Updated by Rich Lowe over 2 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

Resolved in r13541 7320fae78e0e

Also available in: Atom PDF