[Bug 1798397] Re: too eager expansion in substitution

2018-10-20 Thread Thorsten Glaser 
Found to be not a bug:

$ i=42; : ${var#${q[i=777]}}; echo $i; var=meow; : ${var#${q[i=777]}};
echo $i

must return 777 twice, because the pattern must always be expanded
before looking at $var.

$ i=42; : ${var+${q[i=777]}}; echo $i; var=meow; : ${var+${q[i=777]}};
echo $i

already behaves correctly (42 then 777).

** Changed in: mksh
   Status: Triaged => Invalid

-- 
You received this bug notification because you are a member of mksh
Mailing List, which is subscribed to mksh.
Matching subscriptions: mkshlist-to-mksh-bugmail
https://bugs.launchpad.net/bugs/1798397

Title:
  too eager expansion in substitution

Status in mksh:
  Invalid

Bug description:
  “If word is not needed, it shall not be expanded.” (POSIX)

  00:52 < izabera> i=42; : ${var#${q[i=777]}}; echo $i; var=meow; : 
${var#${q[i=777]}}; echo $i
  00:52 < izabera> prints 42 777 in bash and ksh93
  00:52 < izabera> and 777 777 in mkmsh

  The #/##/%/%% of course need to expand word first.

  Thanks to jilles for the heads-up.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mksh/+bug/1798397/+subscriptions

[Bug 1779179] Re: nōn-ASCII heredoc separators

2018-10-20 Thread Thorsten Glaser 
** Changed in: mksh
   Status: Triaged => Fix Committed

-- 
You received this bug notification because you are a member of mksh
Mailing List, which is subscribed to mksh.
Matching subscriptions: mkshlist-to-mksh-bugmail
https://bugs.launchpad.net/bugs/1779179

Title:
  nōn-ASCII heredoc separators

Status in mksh:
  Fix Committed

Bug description:
  tglase@tglase-nb:~ $ mksh -c $':

[Bug 1783355] Re: here document + pipe inside COMSUB broken

2018-10-20 Thread Thorsten Glaser 
The entire tree printing code is currently not suited for that, a major
rewrite is needed.

-- 
You received this bug notification because you are a member of mksh
Mailing List, which is subscribed to mksh.
Matching subscriptions: mkshlist-to-mksh-bugmail
https://bugs.launchpad.net/bugs/1783355

Title:
  here document + pipe inside COMSUB broken

Status in mksh:
  Triaged

Bug description:
  tglase@tglase:~ $ mksh x
  Foo
  BlA9907BAr
  try2
  x: syntax error: unexpected '|'
  1|tglase@tglase:~ $ bash x
  Foo
  BlA9910BAr
  try2
  
  tglase@tglase:~ $ ksh93 x
  Foo
  BlA9917BAr
  try2
  

To manage notifications about this bug go to:
https://bugs.launchpad.net/mksh/+bug/1783355/+subscriptions

[Bug 1719035] Re: mksh.htm "Patching" section should mention KSH_VERSIONNAME_VENDOR_EXT

2018-10-20 Thread Thorsten Glaser 
** Changed in: mksh
   Status: New => Fix Released

-- 
You received this bug notification because you are a member of mksh
Mailing List, which is subscribed to mksh.
Matching subscriptions: mkshlist-to-mksh-bugmail
https://bugs.launchpad.net/bugs/1719035

Title:
  mksh.htm "Patching" section should mention KSH_VERSIONNAME_VENDOR_EXT

Status in mksh:
  Fix Released

Bug description:
  currently there's a dead link to debian and a suggestion of keeping a
  manual local patch that's likely to get lost. overriding
  KSH_VERSIONNAME_VENDOR_EXT seems more robust.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mksh/+bug/1719035/+subscriptions