Skip to content

Commit

Permalink
use perl instead of sed when generating hashes
Browse files Browse the repository at this point in the history
sed is not portable - some platforms (OSX) do not behave like others
  • Loading branch information
MonsieurNicolas committed Feb 24, 2023
1 parent 80c6341 commit b49c6bd
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion hash-xdrs.sh
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ namespace stellar {
extern const std::vector<std::pair<std::filesystem::path, std::string>> XDR_FILES_SHA256 = {
EOF

sha256sum -b $1/xdr/*.x | grep -v Stellar-internal | sed -e 's/\([a-f0-9]\+\)[ \*]*\(.*\)/{"\2", "\1"},/'
sha256sum -b $1/xdr/*.x | grep -v Stellar-internal | perl -pe 's/([a-f0-9]+)[ \*]+(.*)/{"$2", "$1"},/'

echo '{"", ""}};'
echo '}'

5 comments on commit b49c6bd

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from sisuresh
at MonsieurNicolas@b49c6bd

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging MonsieurNicolas/stellar-core/fixupXDRHash = b49c6bd into auto

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MonsieurNicolas/stellar-core/fixupXDRHash = b49c6bd merged ok, testing candidate = 7b0f492

@latobarita
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 7b0f492

Please sign in to comment.