On Thu, Sep 08, 2016 at 09:22:28AM -0600, Todd C. Miller wrote:
> On Thu, 08 Sep 2016 10:08:23 -0400, Anthony Coulter wrote:
> 
> > We can fix either the manual or ksh itself; this diff takes the latter
> > approach. It is tempting to do this with "typeset -ir PPID" but that
> > actually doesn't work:
> > 
> >     $ FOO=123
> >     $ typeset -ir FOO
> >     ksh: FOO: is read only
> 
> Perhaps we should fix the above issue as well, other ksh versions
> don't throw an error there.  Below is a simple fix, though a bit
> gross due to the magic 0x4.

Your fix looks correct to me and accomplishes the desired effect. Should
we perhaps introduce

#define NO_RO_CHECK 0x4

and replace the 5 occurrences of the magic number in var.c?

Reply via email to