Skip to content

Conversation

@darkbasic
Copy link

I've packaged this for Gentoo Linux and I'm using it on ppc64le. Works well and I'd like add explicit support in the Makefile so the binary won't end up into the amd64 folder.

@mmaietta
Copy link
Collaborator

Coming late to the party here, but would you still like this implemented? If so, we'll also need to edit the package.json to include these files in the npm package

app-builder/package.json

Lines 33 to 46 in df4f272

"executableFiles": [
"linux/arm/app-builder",
"linux/arm64/app-builder",
"linux/ia32/app-builder",
"linux/loong64/app-builder",
"linux/riscv64/app-builder",
"linux/x64/app-builder",
"mac/app-builder",
"mac/app-builder_amd64",
"mac/app-builder_arm64",
"win/arm64/app-builder.exe",
"win/ia32/app-builder.exe",
"win/x64/app-builder.exe"
]

@changeset-bot
Copy link

changeset-bot bot commented Sep 15, 2024

⚠️ No Changeset found

Latest commit: f37cf0a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@darkbasic
Copy link
Author

@mmaietta thanks, I'm still interested to see this implemented. I have applied the changes you suggested among other changes I've copied from the riscv PR but I haven't been able to test a new build on ppc64 yet.

@mmaietta
Copy link
Collaborator

Hi @darkbasic , have you had a chance to test this yet by chance? Just wanted to check in 🙂

@darkbasic
Copy link
Author

Hi @mmaietta , no I'm sorry I still didn't have the chance to test it :(

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