Skip to content

[AURON #1476] Remove extra parenthesis in assignment causing invalid Docker build context#1477

Merged
merrily01 merged 3 commits intoapache:masterfrom
merrily01:hotfix-1459
Oct 20, 2025
Merged

[AURON #1476] Remove extra parenthesis in assignment causing invalid Docker build context#1477
merrily01 merged 3 commits intoapache:masterfrom
merrily01:hotfix-1459

Conversation

@merrily01
Copy link
Member

@merrily01 merrily01 commented Oct 20, 2025 •
edited
Loading

Which issue does this PR close?

Closes #1476.

Rationale for this change

Compilation issue caused by a typo that was accidentally introduced. Sorry for the inconvenience

What changes are included in this PR?

Are there any user-facing changes?

How was this patch tested?

@github-actions github-actions bot removed the infra label Oct 20, 2025
@merrily01 merrily01 marked this pull request as ready for review October 20, 2025 11:45
@merrily01
Copy link
Member Author

merrily01 commented Oct 20, 2025

Compilation issue caused by a typo that was accidentally introduced.
Has been verified via GA, https://github.com/apache/auron/actions/runs/18645920516

PTAL @wForget @guixiaowen @Tartarus0zm , sorry for the inconvenience.

@cxzl25
Copy link
Contributor

cxzl25 commented Oct 20, 2025

I recommend adding auron-build.sh to build-ce7-releases.yml paths.

auron/.github/workflows/build-ce7-releases.yml

Line 31 in 9e9dc61

- '.github/workflows/build-ce7-releases.yml'

@guixiaowen
Copy link
Contributor

guixiaowen commented Oct 20, 2025

Compilation issue caused by a typo that was accidentally introduced. Has been verified via GA, https://github.com/apache/auron/actions/runs/18645920516

PTAL @wForget @guixiaowen @Tartarus0zm , sorry for the inconvenience.

@merrily01 LGTM

@merrily01
Copy link
Member Author

merrily01 commented Oct 20, 2025

I recommend adding auron-build.sh to build-ce7-releases.yml paths.

auron/.github/workflows/build-ce7-releases.yml

Line 31 in 9e9dc61

- '.github/workflows/build-ce7-releases.yml'

Thanks for the suggestion! Since this is a critical bug fix, I'd like to merge this first and work on it afterwards in a separate issue

@merrily01 merrily01 merged commit 37a1e02 into apache:master Oct 20, 2025
85 checks passed
@merrily01
Copy link
Member Author

merrily01 commented Oct 20, 2025

Thanks again @wForget @cxzl25 @guixiaowen

guixiaowen pushed a commit to guixiaowen/blaze that referenced this pull request Oct 20, 2025
@merrily01
[AURON apache#1476] Remove extra parenthesis in assignment causing in…
a4bfab1 
…valid Docker build context (apache#1477)

* [AURON apache#1476] Remove extra parenthesis in assignment causing invalid Docker build context

* Trigger a GA testing

* Revert "Trigger a GA testing"
@merrily01 merrily01 mentioned this pull request Oct 27, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cxzl25 cxzl25 cxzl25 approved these changes

@wForget wForget wForget approved these changes

Assignees

No one assigned

Labels

build

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Hotfix for AURON #1459

4 participants

@merrily01 @cxzl25 @guixiaowen @wForget