[go: up one dir, main page]

Wrong shebang in sfntest/size.sh

The shebang is #!/bin/sh, however a lot debian-based machines do not symlink /bin/sh to /bin/bash. And dash provides /bin/sh (read somewhere they do this as dash is faster to start then bash, improving system startup).

I'm using the latest commit (7c05d96ba4cced6d3e13b4f7d5b50837497eb910).

Changing the shebang to #!/bin/bash fixes the issue. (Issues are on lines 5, 14 and 17. As dash complains).

Without the change:

./size.sh: 5: function: not found
readelf: Warning: Nothing to do.
Usage: readelf <option(s)> elf-file(s)
[TRIMMED]
  -v --version           Display the version number of readelf
Simple renderer: ./size.sh: 14: getsize: not found
Normal renderer: ./size.sh: 17: getsize: not found
make: *** [Makefile:26: size] Error 127

With changing the shebang to #!/bin/bash:

Simple renderer: 1157 bytes
Normal renderer: 29731 bytes