Skip to content

Qooxdoo arm#90

Merged
wvangeit merged 8 commits intoITISFoundation:masterfrom
wvangeit:qooxdoo_arm
Oct 16, 2025
Merged

Qooxdoo arm#90
wvangeit merged 8 commits intoITISFoundation:masterfrom
wvangeit:qooxdoo_arm

Conversation

@wvangeit
Copy link
Copy Markdown
Contributor

@wvangeit wvangeit commented Oct 15, 2025

Add ability to compile the qooxdoo-kit image on ARM processors.

Adds a workflow to test the build (on intel and arm worker). Only building for now, not deploying.

@wvangeit wvangeit self-assigned this Oct 15, 2025
@wvangeit wvangeit added the enhancement New feature or request label Oct 15, 2025
@wvangeit wvangeit added this to the Imparable milestone Oct 15, 2025
@wvangeit wvangeit marked this pull request as ready for review October 15, 2025 12:55
@wvangeit wvangeit marked this pull request as draft October 15, 2025 13:00
@wvangeit wvangeit marked this pull request as ready for review October 15, 2025 13:26
Copy link
Copy Markdown
Member

@pcrespov pcrespov left a comment

Choose a reason for hiding this comment

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

qooxdoo leg! ;-)
thx

@wvangeit wvangeit merged commit ad329b4 into ITISFoundation:master Oct 16, 2025
2 checks passed
@wvangeit wvangeit deleted the qooxdoo_arm branch October 16, 2025 07:47
Copy link
Copy Markdown
Member

@sanderegg sanderegg left a comment

Choose a reason for hiding this comment

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

can we discuss a bit this?
there are a few things I already went (painfully) through ;) no need you have to do the same.

su - node -c "npm install"
RUN su - node -c "npm install npm@latest"

RUN if [ "$(uname -m)" = "aarch64" ] || [ "$(uname -m)" = "arm64" ]; then export NPM_OPTIONS="--build-from-source"; else export NPM_OPTIONS=""; fi
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

for info there are special docker ENVs for these things

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants