ardupilot icon indicating copy to clipboard operation
ardupilot copied to clipboard

Filter: Fix gnuplot shebang lines

Open cclauss opened this issue 5 months ago • 1 comments

https://github.com/ArduPilot/ardupilot/pull/30351#pullrequestreview-2928927779

The shebang lines are actually incorrect in the .gnu files

% git grep gnuplot

Tools/Replay/plotit.sh:gnuplot _plot.gnu  # Not a shebang line.
libraries/Filter/tests/plot_harmonics.gnu:#!/usr/bin/gnuplot -persist
libraries/Filter/tests/plot_harmonictest2.gnu:#!/usr/bin/gnuplot -persist
libraries/Filter/tests/plot_harmonictest3.gnu:#!/usr/bin/gnuplot -persist
libraries/Filter/tests/plot_harmonictest4.gnu:#!/usr/bin/gnuplot -persist
libraries/Filter/tests/plot_harmonictest5.gnu:#!/usr/bin/gnuplot -persist

% git diff # In 5 files.

- #!/usr/bin/gnuplot -persist
+ #!/usr/bin/env -S gnuplot -persist

cclauss avatar Jun 15 '25 09:06 cclauss

Can a maintainer please rerun the failing test?

#!/usr/bin/env -S gnuplot -persist

cclauss avatar Jun 15 '25 20:06 cclauss

Can a maintainer please rerun the failing test?

#!/usr/bin/env -S gnuplot -persist

I've re-tested and things are now working.

Please avoid pushing the testing burden onto other people. It's usually more work than the initial patches.

peterbarker avatar Jun 20 '25 01:06 peterbarker

Thanks for your patience and the reviews!

Please avoid pushing the testing burden onto other people. It's usually more work than the initial patches.

Agreed. Sorry. brew install gnuplot would have installed a lot of extra gorp onto my Mac, which I wanted to avoid. I should have tested this in a Docker container.

On the bright side, I now have that #!/usr/bin/env -S syntax firmly planted in my head and am using it in other testing.

  • https://github.com/ArduPilot/MAVProxy/pull/1588#issuecomment-2986540996

cclauss avatar Jun 20 '25 03:06 cclauss