Skip to content

fix: eic-spack: dd4hep: backport "OSError raised by getpass.getuser()" patch#330

Merged
wdconinc merged 1 commit into
masterfrom
dd4hep-getpass-getuser
Jul 2, 2026
Merged

fix: eic-spack: dd4hep: backport "OSError raised by getpass.getuser()" patch#330
wdconinc merged 1 commit into
masterfrom
dd4hep-getpass-getuser

Conversation

@wdconinc

@wdconinc wdconinc commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Briefly, what does this PR introduce? Please link to any relevant presentations or discussions.

This PR updates eic-spack to the commit with the DD4hep backport of the "OSError raised by getpass.getuser()" patch.

What is the urgency of this PR?

  • High (please describe reason below)
  • Medium
  • Low

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue #__)
  • Optimization (issue #__)
  • Updated documentation
  • other: __

Please check if any of the following apply

  • This PR introduces breaking changes. Please describe changes users need to make below.
  • This PR changes default behavior. Please describe changes below.
  • AI was used in preparing this PR. Please describe usage below.

@wdconinc wdconinc requested review from a team, Copilot and veprbl July 1, 2026 16:24
@wdconinc wdconinc enabled auto-merge (squash) July 1, 2026 16:24

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

This PR updates the pinned eic/eic-spack reference used by the container build pipeline to a newer commit that includes the DD4hep backport fixing the OSError raised by getpass.getuser() issue.

Changes:

  • Bump EICSPACK_VERSION in eic-spack.sh from 01aaad210b... to b1ebb8295d... to pull in the DD4hep backport.

@wdconinc wdconinc merged commit 994de48 into master Jul 2, 2026
85 checks passed
@wdconinc wdconinc deleted the dd4hep-getpass-getuser branch July 2, 2026 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants