Skip to content

Fix for arm64 runner#392

Merged
hannesa2 merged 1 commit intomasterfrom
arm-runner-fix
Mar 12, 2023
Merged

Fix for arm64 runner#392
hannesa2 merged 1 commit intomasterfrom
arm-runner-fix

Conversation

@hannesa2
Copy link
Copy Markdown
Contributor

@hannesa2 hannesa2 commented Mar 12, 2023

master is broken on arm64 runner
#391

@hannesa2
Copy link
Copy Markdown
Contributor Author

I run into this on arm64 runner

image

@hannesa2
Copy link
Copy Markdown
Contributor Author

hannesa2 commented Mar 12, 2023

Even in terminal the --output doesn't work anymore

image

@hannesa2 hannesa2 merged commit e0d192b into master Mar 12, 2023
@hannesa2 hannesa2 deleted the arm-runner-fix branch March 12, 2023 11:14
@ADTC
Copy link
Copy Markdown
Contributor

ADTC commented Mar 12, 2023

@hannesa2 It should be --output=$PP_PATH. Notice the = (equal sign) instead of (space). You can see in the man doc.

It's usually the case for -- (double dash) options in Unix, while space works with - (single dash) options. I haven't seen an exception before. Surprised it worked before with the space.

-o is a shortcut and is intended to work with space instead of equal sign.

@ADTC
Copy link
Copy Markdown
Contributor

ADTC commented Mar 12, 2023

@hannesa2 BTW, shouldn't we fix this in BuildRelease.yml also?

@hannesa2
Copy link
Copy Markdown
Contributor Author

@hannesa2 BTW, shouldn't we fix this in BuildRelease.yml also?

sure !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants