8851 2015-12-22 19:35:14 +0000 Add bashism checks for shell script functions 2017-12-06 16:51:21 +0000 1 1 7 Build System, Metadata & Runtime OE-Core core 2.0 All Multiple RESOLVED FIXED Medium+ enhancement 2.2 M4 1 bluelightning ross.burton jose.perez.carranza junhuix.zhang meta.mr.watcher meta.watcher richard.purdie --- --- Yes (doc changes required) oldest_to_newest 57437 0 bluelightning 2015-12-22 19:35:14 +0000 Seeing as we often have the issue of bashisms turning up in shell functions and therefore failing on Ubuntu, we should add some kind of check for these if it's practical to do so. Presumably this would use checkbashisms. I don't know if it will be practical to do this as a normal QA check; if we have to do it externally that would be acceptable. 57628 1 richard.purdie 2016-01-07 15:47:38 +0000 Since we parse the shell functions, could the parser check for these? 57901 2 bluelightning 2016-01-17 22:21:51 +0000 *** Bug 8932 has been marked as a duplicate of this bug. *** 58878 3 ross.burton 2016-02-11 12:07:40 +0000 checkbashisms is 26k of perl, so I don't fancy reimplementing that in Python to be honest! 58879 4 ross.burton 2016-02-11 12:08:49 +0000 I'm thinking a bit of tooling to extract all the shell scripts from the metadata and run it through checkbashisms would be more practical that verifying on every parse. 58937 5 ross.burton 2016-02-11 23:26:47 +0000 Well I've made a prototype and it bloody well works! checkbashism is far too slow to do on parse. 66310 6 ross.burton 2016-09-19 12:42:14 +0000 Merge in oe-core d77fe838ab7631a19e90ff4226f0712e54aa4e22.