Comments (6)
please pull jelmd@bcf9fb4
from ast.
I vote no to merging the suggested fix.
Why is ksh93 still using the libast language translation functions? Rather than continuing to maintain and fix that code we should switch to the standard locale functions such as gettext()
and the idiom of printf(_("please localize me\n"));
. This would be another huge step in slimming down the ksh code base that has to be maintained by eliminating the need to maintain the AST message catalog code.
from ast.
Well, I guess, the initial idea was to not depend on non-libc/ non-ast libs and provide base for $'....' handling. Anyway, if you vote against merge, you should provide an alternative to fix the bug or tell, what needs to be fixed in the fix ;-)
from ast.
I did provide an alternative solution, jelmd. To wit: use the reasonably standard gettext()
API and related tools. Also, your problem statement was just
NLSPATH resolver bug needs to be fixed.
What bug? How can someone reproduce the problem? Why didn't you add a unit test?
And this comment in your fix suggests to me the fix is not reliable:
/* 'ksh93_tm' hopefully does not resolve to a regular executable file */
I realize English is not your native language but you continue to be willfully obtuse and obnoxious in your comments. Not just in this project bug also here: tensorflow/tensorflow#13550. Most people are going to ignore you because of your abrasive style of interaction.
from ast.
Again, why do you read the subjects, only and not the problem description before making unqualified comments again and again?
The bug description as well as test case are included in the issue comment 1!
Wrt. "fix is not reliable": Can you read the issue description first and than ask, what you not understand?
I think it is pretty well explained, what happens and at least a ksh developer, who knows, how ksh's localization currently works, should understand it (even if it is not native/perfect English). And if so, one would realize, that the fix tries to circumvent the problem but can't fix it for all corner cases (here if a 'ksh93_tm' binary can be found via PATH), because this would break ksh behavior.
Wrt. to the last paragraph: plz do not talk about stuff you do not understand! Picking up some stuff w/o the context is a really bad behavior!
from ast.
We're going to drop use of the libast locale code as soon as possible. This will make ksh
behavior consistent with other programs on a given platform. So there isn't much point merging a change for this obscure problem.
from ast.
Related Issues (20)
- Function scoped variable should retain its last value in while-loop body HOT 3
- printf %q broken with size specifier HOT 7
- 2020.0.0: test suite is failing HOT 8
- ksh: command substitution doesn't include the output of non-waited-for child processes HOT 1
- write package as an rpm (/bin/pacakge make) HOT 6
- Please fix invalid use of memccpy() on master HOT 5
- MacOS compiling libast
- ${$} throwing syntax error when use in cat << EOF on ksh93 93u+ . HOT 1
- Use of >/dev/null 2>&1 in called function causes "Abort(coredump)" HOT 4
- The subshell does not recognize the path path.
- Process substitution includes extraneous spaces in file descriptor HOT 1
- `a=()` should make `a` an empty array but doesn't HOT 2
- $EDITOR
- man page uses incorrect pipe character, "⎪" xE28EAA instead of "|" x7C HOT 1
- Please consider relicensing AST under the EPL-2.0 with the GPL compatibility clause
- ksh93u+ dumps core when fds >32 HOT 1
- building with Cygwin instead of NMAKE HOT 2
- The release page for the latest version of ksh boggles HOT 1
- undefined reference to `__dso_handle' HOT 3
- support HISTCONTROL=ignoredups HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from ast.