acassis commented on PR #3557: URL: https://github.com/apache/nuttx-apps/pull/3557#issuecomment-4876741295
> > Thanks for the review @ThePassionate host build tool tools/mkpasswd.c (in the NuttX PR) will still keep a standalone PBKDF2 implementation since it runs on the build machine, not on the target, Right? > > I'll remove apps/crypto/pbkdf2 and have fsutils/passwd use kernel PBKDF2 via /dev/crypto. The host build tool tools/mkpasswd.c (nuttx PR) will keep a standalone implementation since it runs on the build machine. > > it's more simple to convert host tool from c to python, since we don't need compile the tool and could support more OS. @xiaoxiang781216 the problem with Python it because it is a moving target, we don't know when will be the next version transition that will break everything (like 2 to 3). So if we can keep it in plain and old C it is safe. But I'm not against using Python, I just suggest avoid depending on external libraries, this way it will survive longer and we will avoid listening complains from news users because NuttX is not working -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
