fix-shebang duplication

osrf / rosbook

Example code to accompany the book Programming Robots with ROS

Apache License 2.0

484 stars 236 forks source link

fix-shebang duplication #26

Closed hunterino closed 2 years ago

hunterino commented 7 years ago

Removed duplicate Shebang

gbiggs commented 7 years ago

I suspect that these are actually intentional duplications to:

Make the scripts executable as-is (because a shebang must be the first line)
Ensure the shebang appears in the code included in the book (which needs to be after the #BEGIN ... line)

gerkey commented 7 years ago

Yeah, as @gbiggs said, we're trying to make the code in this repo do double-duty: be runnable by users but also be parseable by the asciidoc typesetting system used by O'Reilly to produce the book. I don't think that we can get away with removing the duplicate shebang lines. But we'll take a look; we're about to embark on a second printing that will give us a chance to incorporate improvements.