resmo commented on PR #13832: URL: https://github.com/apache/cloudstack/pull/13832#issuecomment-5290121955
> Without getting into the merits of the programming language/script to use, the idea behind using bash for this extension was to showcase that any executable can be used. This is why Proxmox extension was created in bash and HyperV extension was created in Python. I would still prefer it in bash or maybe something other than python. Thanks @shwstppr for explanation, just to make it clear, I don't say your work or the show case is bad or wrong. Showcases are good to show a case but I don't think that we should expand the scope of programming languages in apache/cloudstack just to "prove" we can do that. This is "software" of course you can do many things. I don't know many people writing error prove bash and the ones do, avoid bash by any chance: maintability and extensibilty, integration, error handling and testabilty is way better in any other language. Large parts of apache/cloudstack is in java or python. These are the langauges the community knows best. If we don't have a very, very good reason to not using one of these, use one of these. (I burnt my fingers once created a project (gh/git-ftp/git-ftp) in bash and learned the hard way how easy it is to write "good looking but wrong" code in bash.) -- 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]
