Skip to content

Commit c0efaa5

Browse files
committed
Merge #187: ci: have bash scripts explicitly opt out of locale dependence.
3a6db38 ci: rename configs to .bash (Sjors Provoost) 401e0ce ci: add copyright to bash scripts (Sjors Provoost) e956467 ci: export LC_ALL (Sjors Provoost) Pull request description: Prevents linter issues like reported here: Sjors/bitcoin#90 (comment) But probably also just a good idea. Also make the use of `#!/usr/bin/env bash` consistent and added a copyright header (mostly for aesthetic reasons). ACKs for top commit: ryanofsky: Code review ACK 3a6db38. Thanks! Tree-SHA512: de4a3fe62126f0f37e9b33c6e443ee07b2968ac8df18b7d91ce482ce63aeab896b216a7efcc638945ef4f83b4429471433c3515b5bbe744fb238bec08792f7af
2 parents 8954cc0 + 3a6db38 commit c0efaa5

File tree

7 files changed

+18
-4
lines changed

7 files changed

+18
-4
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,5 +25,5 @@ jobs:
2525

2626
- name: Run CI script
2727
env:
28-
CI_CONFIG: ci/configs/${{ matrix.config }}.sh
28+
CI_CONFIG: ci/configs/${{ matrix.config }}.bash
2929
run: ci/scripts/run.sh

ci/README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,9 @@ All CI is just bash and nix.
1616
To run jobs locally:
1717

1818
```bash
19-
CI_CONFIG=ci/configs/default.sh ci/scripts/run.sh
20-
CI_CONFIG=ci/configs/llvm.sh ci/scripts/run.sh
21-
CI_CONFIG=ci/configs/gnu32.sh ci/scripts/run.sh
19+
CI_CONFIG=ci/configs/default.bash ci/scripts/run.sh
20+
CI_CONFIG=ci/configs/llvm.bash ci/scripts/run.sh
21+
CI_CONFIG=ci/configs/gnu32.bash ci/scripts/run.sh
2222
```
2323

2424
By default CI jobs will reuse their build directories. `CI_CLEAN=1` can be specified to delete them before running instead.
File renamed without changes.
File renamed without changes.
File renamed without changes.

ci/scripts/ci.sh

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,11 @@
11
#!/usr/bin/env bash
2+
#
3+
# Copyright (c) The Bitcoin Core developers
4+
# Distributed under the MIT software license, see the accompanying
5+
# file COPYING or http://www.opensource.org/licenses/mit-license.php.
6+
7+
export LC_ALL=C.UTF-8
8+
29
set -o errexit -o nounset -o pipefail -o xtrace
310

411
[ "${CI_CONFIG+x}" ] && source "$CI_CONFIG"

ci/scripts/run.sh

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,11 @@
11
#!/usr/bin/env bash
2+
#
3+
# Copyright (c) The Bitcoin Core developers
4+
# Distributed under the MIT software license, see the accompanying
5+
# file COPYING or http://www.opensource.org/licenses/mit-license.php.
6+
7+
export LC_ALL=C.UTF-8
8+
29
set -o errexit -o nounset -o pipefail -o xtrace
310

411
[ "${CI_CONFIG+x}" ] && source "$CI_CONFIG"

0 commit comments

Comments
 (0)