Skip to content

Commit b498426

Browse files
committed
Apply fixes
1 parent 3a1d751 commit b498426

File tree

2 files changed

+6
-17
lines changed

2 files changed

+6
-17
lines changed

bin/doc-tools.js

100644100755
File mode changed.

tools/property-extractor/Makefile

Lines changed: 6 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,5 @@
11
.PHONY: build venv clean redpanda-git treesitter generate-docs check
22

3-
# --- Main build: venv, fetch code, build parser, extract & docgen ---
4-
build: venv redpanda-git treesitter
5-
@echo "🔧 Building with Redpanda tag: $(TAG)"
6-
@mkdir -p $(TOOL_ROOT)/gen
7-
@cd $(TOOL_ROOT) && \
8-
$(PYTHON) -W ignore::FutureWarning property_extractor.py \
9-
--recursive \
10-
--path $(REDPANDA_SRC) \
11-
--output gen/properties-output.json
12-
@echo "✅ Cluster properties JSON generated at $(TOOL_ROOT)/gen/properties-output.json"
13-
@$(MAKE) generate-docs
14-
153
# Default tag (can be overridden via `make TAG=v25.1.1`)
164
TAG ?= dev
175

@@ -51,7 +39,7 @@ build: venv redpanda-git treesitter
5139
exit 1; \
5240
fi
5341
@cd $(TOOL_ROOT) && \
54-
$(PYTHON) -W ignore::FutureWarning property_extractor.py \
42+
$(PYTHON) -W ignore property_extractor.py \
5543
--recursive \
5644
--path $(REDPANDA_SRC) \
5745
--output gen/properties-output.json \
@@ -67,11 +55,12 @@ venv: $(TOOL_ROOT)/requirements.txt
6755
@if [ ! -d "$(VENV)" ]; then \
6856
echo "🐍 Creating virtual environment in $(VENV)..."; \
6957
python3 -m venv $(VENV); \
70-
$(VENV)/bin/pip install --upgrade pip --quiet; \
71-
$(VENV)/bin/pip install --no-cache-dir -r $<; \
7258
else \
7359
echo "🐍 Virtual environment already exists at $(VENV)"; \
74-
fi
60+
fi; \
61+
echo "🔄 Upgrading pip and installing requirements..."; \
62+
$(VENV)/bin/pip install --upgrade pip --quiet; \
63+
$(VENV)/bin/pip install --no-cache-dir -r $<;
7564

7665
# --- Clean out all generated state ---
7766
clean:
@@ -107,7 +96,7 @@ treesitter:
10796
git fetch --tags -q && \
10897
git checkout -q v0.20.5
10998
@echo "🔧 Generating parser in $(TREESITTER_DIR)"
110-
@cd "$(TREESITTER_DIR)" && npm install --silent && $(TREE_SITTER) generate
99+
@cd "$(TREESITTER_DIR)" && export CFLAGS="-Wno-unused-but-set-variable" && npm install --silent && export CFLAGS="-Wno-unused-but-set-variable" && $(TREE_SITTER) generate
111100

112101
# --- Install Node.js dependencies for Handlebars ---
113102
node-deps:

0 commit comments

Comments
 (0)