Skip to content

Commit

Permalink
remove -i option from sed (#1389)
Browse files Browse the repository at this point in the history
# Description

@swift-nav/devinfra

cargo release uses a pre-commit hook to update the sbp2json input to
point to the latest sbp release in crates.io

However, the sed version uses the `-i` flag which is not POSIX defined
and will give different results with gnu sed vs BSD sed.

Now this should be good to release from OSX or linux
  • Loading branch information
pcrumley authored Dec 8, 2023
1 parent 3d8c0be commit d7b3b44
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion rust/sbp2json/release-hook.sh
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
#!/usr/bin/env bash

sed -i -e "s@^.*TODO.*replace.*sbp.*\$@version = \"$NEW_VERSION\"@" Cargo.toml
sed -e "s@^.*TODO.*replace.*sbp.*\$@version = \"$NEW_VERSION\"@" Cargo.toml > Cargo.toml.new
mv Cargo.toml.new Cargo.toml

0 comments on commit d7b3b44

Please sign in to comment.