pan3793 opened a new pull request, #57851:
URL: https://github.com/apache/spark/pull/57851

   ### What changes were proposed in this pull request?
   
   `dev/test-dependencies.sh` now writes a header at the top of each manifest 
it generates (`dev/deps/spark-deps-*`):
   
   ```
   #
   # Generated by `dev/test-dependencies.sh --replace-manifest`.
   # Do not edit manually.
   #
   ```
   
   The same header is prepended to the committed 
`dev/deps/spark-deps-hadoop-3-hive-2.3` so the manifest matches generator 
output and the self-check (`dev/test-dependencies.sh` without 
`--replace-manifest`) stays green.
   
   Only POSIX-portable utilities are used in the new code path (`printf 
'%s\n'`, `cat`, `rm`); the existing `awk` already uses only POSIX-standard 
functions, so the script remains compatible across macOS (BSD awk) and Linux 
(gawk).
   
   ### Why are the changes needed?
   
   The manifest is machine-generated and regenerated by 
`dev/test-dependencies.sh --replace-manifest`; hand edits get overwritten on 
the next regeneration. A header stating this prevents accidental manual edits 
to a checked-in generated file.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   - `bash -n dev/test-dependencies.sh`
   - Verified byte-for-byte that the header the script emits matches the header 
prepended to the committed manifest, so the manifest self-check still passes.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: GLM 5.2
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to