-
Notifications
You must be signed in to change notification settings - Fork 0
Merge bitcoin/bitcoin#28452: Do not use std::vector = {} to release memory #932
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
e8f7232
94a949a
c78607e
7454cf9
b418728
b4871f5
f02283d
26cd950
024f985
a30e6d6
c3dd813
ab7169d
28aed29
01bc738
5d4ef25
f682f33
6176d99
06a3efe
9dda95e
1551739
b957ec0
b158b0a
92ec61f
1d6b227
2ad06fc
634d616
b0ec83b
3cfc0d5
73809c1
2209779
242cbc6
eda39ea
916b16f
9de051b
ba84f27
b5e48b2
e8d4dff
54a273f
e79fe14
53646bc
1977c70
3931770
552f1c5
bd21777
43d194c
41d99b3
06d3a92
e7815ec
adab39e
5e610cc
df35941
c84e6f7
43306ea
4b02880
664f6dc
4102d65
681a930
6559000
3691d34
18c9fe5
28011dd
155435c
8ca78db
2be3206
a15317b
0481dc6
a9f35e5
a60f5cb
fc5b0b7
ce4c9b1
1aa5116
0a16334
d54bdf7
9379b1e
57a5d50
af0e1e1
429cc06
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,7 @@ | ||
| [main] | ||
| host = https://www.transifex.com | ||
|
|
||
| [dash.dash_ents] | ||
| [o:dash:p:dash:r:dash_ents] | ||
| file_filter = src/qt/locale/dash_<lang>.ts | ||
| source_file = src/qt/locale/dash_en.xlf | ||
| source_lang = en |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -55,7 +55,7 @@ export BOOST_TEST_RANDOM=${BOOST_TEST_RANDOM:-1} | |
| export DEBIAN_FRONTEND=noninteractive | ||
| export HOST_CACHE_DIR=${HOST_CACHE_DIR:-$BASE_ROOT_DIR/ci-cache-$BUILD_TARGET} | ||
| export CACHE_DIR=${CACHE_DIR:-$HOST_CACHE_DIR} | ||
| export CCACHE_SIZE=${CCACHE_SIZE:-100M} | ||
| export CCACHE_MAXSIZE=${CCACHE_MAXSIZE:-100M} | ||
| export CCACHE_TEMPDIR=${CCACHE_TEMPDIR:-/tmp/.ccache-temp} | ||
| export CCACHE_COMPRESS=${CCACHE_COMPRESS:-1} | ||
|
Comment on lines
57
to
60
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Renamed variable needs follow-up clean-up
#!/usr/bin/env bash
# Verify no stale reference remains
rg --fixed-strings 'CCACHE_SIZE' -n🤖 Prompt for AI Agents |
||
| # The cache dir. | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -97,7 +97,7 @@ if [ "$CI_OS_NAME" == "macos" ]; then | |||||||||||||||
| echo "Number of CPUs: $(sysctl -n hw.logicalcpu)" | ||||||||||||||||
| else | ||||||||||||||||
| CI_EXEC free -m -h | ||||||||||||||||
| CI_EXEC echo "Number of CPUs \(nproc\):" \$\(nproc\) | ||||||||||||||||
| CI_EXEC echo "Number of CPUs (nproc): $(nproc)" | ||||||||||||||||
| CI_EXEC echo "$(lscpu | grep Endian)" | ||||||||||||||||
|
Comment on lines
99
to
101
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
-CI_EXEC echo "Number of CPUs (nproc): $(nproc)"
+CI_EXEC bash -c 'echo "Number of CPUs (nproc): $(nproc)"'This ensures the 📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||
| fi | ||||||||||||||||
| CI_EXEC echo "Free disk space:" | ||||||||||||||||
|
|
||||||||||||||||
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
Pinning
lief==0.13.1may break reproducibility on future CI imagesliefoften publishes manylinux wheels only for a subset of Python versions and glibc combos.Consider:
--no-binary :all:compile path if wheels disappear.contrib/cilike other toolchain pins.This prevents sudden CI breakage when upstream yanks/relocates binaries.
#!/bin/bash
set -e
echo "=== Searching for all pip3 install commands in the repository ==="
rg "pip3 install" -n .
echo "=== Checking for any use of --no-binary in CI or install scripts ==="
rg "--no-binary" -n .
echo "=== Looking for a contrib/ci directory (cached wheels area) ==="
find . -type d | grep -E "contrib/ci" || echo "No contrib/ci directory found"
🏁 Script executed:
Length of output: 320993
Ensure reproducible LIEF installation in CI
File: ci/lint/04_install.sh (around line 38)
The current line pins LIEF to 0.13.1 but provides no rationale, no build-from-source fallback, and no cached wheel if PyPI wheels vanish. To avoid future CI failures, please:
lief==0.13.1is chosen (e.g. last version with manylinux wheels for our Python/glibc matrix).contrib/ci/directory (as with other toolchain pins) and prefer it before falling back to PyPI.These steps will guard against upstream wheel removals and keep CI builds stable.
🤖 Prompt for AI Agents