-
Notifications
You must be signed in to change notification settings - Fork 152
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Analyze changes made after last stable release (ksh93u+) #1116
Comments
FWIW, Any introduction of a new builtin is strictly speaking not backward compatible since the new builtin may have different behavior than the platform's external command of the same name. Which is why @siteshwar removed a few new builtins such as |
We should revert this change:
|
Could this change cause issues with compatibility ?
|
Another experimental change is:
and
|
This is a backward incompatible change too. But it looks like a sane change that can be easily worked around in scripts by moving namespace definitions outside functions. I will keep it. |
This flag was marked as experimental and should not be included in next release. Related: #1116
This was an experimental feature that was added after last stable release. Disable it. Related: #1116
ksh93u+
was last stable release that was embraced by various distributions. We should analyze all the changes made after this release. Any backward incompatible changes should be reverted. If there is a new feature that's worth mentioning, it should be mentioned inCHANGELOG.md
file. Changes can be found here.The text was updated successfully, but these errors were encountered: