XD-DENG opened a new pull request, #63233:
URL: https://github.com/apache/airflow/pull/63233
The error message in `install_from_external_spec()` referenced
`${INSTALLATION_METHOD}`, but the actual variable is
`${AIRFLOW_INSTALLATION_METHOD}`. No variable called `INSTALLATION_METHOD`
exists anywhere in the codebase — it seems a typo introduced in a1717a652b.
Since the Docker scripts run with `set -u`, hitting this error path (by
passing an unsupported installation method) would crash with an "unbound
variable" bash error instead of printing the intended user-friendly message.
The fix is applied to the canonical script and both inlined Dockerfile
copies:
- `scripts/docker/install_airflow_when_building_images.sh`
- `Dockerfile`
- `Dockerfile.ci`
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes — Claude Code (Claude Opus 4.6)
Generated-by: Claude Code (Claude Opus 4.6) following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
