Skip to content

fix: place S/4HANA 2025 SAPCAR in downloads - #119

Merged
devanshjainms merged 1 commit into
mainfrom
dev/devanshjain/fix980b5b03
Jul 25, 2026
Merged

fix: place S/4HANA 2025 SAPCAR in downloads#119
devanshjainms merged 1 commit into
mainfrom
dev/devanshjain/fix980b5b03

Conversation

@devanshjainms

Copy link
Copy Markdown
Contributor

Summary

  • place the S/4HANA 2025 SAPCAR executable in the downloads directory
  • align the BOM destination with the installer command /usr/sap/install/downloads/SAPCAR
  • retain the executable filename and 0755 permissions added previously

Validation

  • parsed SAP/S4_2025_SPS08ms/S4_2025_SPS08ms.yaml with PyYAML
  • ran git diff --check

Operational note

Existing consolidated BOMs in SAP Library storage must be refreshed before deployments consume this updated path.

Align the BOM with the installer path used to invoke SAPCAR during media extraction.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: d4830c26-a648-44e7-aa1b-299ea5a060c6
Copilot AI review requested due to automatic review settings July 25, 2026 00:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the S/4HANA 2025 SPS08 BOM material definition so the SAPCAR executable is placed under the downloads directory, aligning the BOM output with the installer’s expected /usr/sap/install/downloads/SAPCAR path while keeping the existing filename and executable permissions behavior.

Changes:

  • Change SAPCAR material path from download_basket to downloads in the S/4HANA 2025 SPS08 BOM.
  • Retain existing SAPCAR target filename (SAPCAR.EXE) and 0755 permissions settings.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread SAP/S4_2025_SPS08ms/S4_2025_SPS08ms.yaml
@devanshjainms
devanshjainms merged commit 34b16b1 into main Jul 25, 2026
2 checks passed
@devanshjainms
devanshjainms deleted the dev/devanshjain/fix980b5b03 branch July 28, 2026 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants