Skip to content

Quickfix x86 builds#2129

Open
0x41c wants to merge 1 commit into
ish-app:masterfrom
0x41c:master
Open

Quickfix x86 builds#2129
0x41c wants to merge 1 commit into
ish-app:masterfrom
0x41c:master

Conversation

@0x41c
Copy link
Copy Markdown
Contributor

@0x41c 0x41c commented Jun 5, 2023

A small patch for the time being until the build environment gets rewritten. I attempted to have EXCLUDED_ARCHS set from the xcconfig, but it just wouldn't take and I'd still get issues when it'd try cross compiling the arm64 build. I've left a comment about that unless there's another way of getting the following line to work:

EXCLUDED_ARCHS[sdk=macosx*][arch=x86_64] = arm64

This is all under the assumption that universal builds just won't make the cut for iSH.

@randomnessExistance
Copy link
Copy Markdown

2129

@62f
Copy link
Copy Markdown

62f commented Jun 12, 2023

2129 @randomnessExistance No, more like so:

/pull/2129

@Robertearl5770
Copy link
Copy Markdown

This 2 file changed was that on the ish shell ?

Copy link
Copy Markdown

@AimSD23nSiR AimSD23nSiR left a comment

Choose a reason for hiding this comment

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

Random

Copy link
Copy Markdown

@AimSD23nSiR AimSD23nSiR left a comment

Choose a reason for hiding this comment

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

A small patch for the time being until the build environment gets rewritten. I attempted to have EXCLUDED_ARCHS set from the xcconfig, but it just wouldn't take and I'd still get issues when it'd try cross compiling the arm64 build. I've left a comment about that unless there's another way of getting the following line to work:

EXCLUDED_ARCHS[sdk=macosx*][arch=x86_64] = arm64

This is all under the assumption that universal builds just won't make the cut for iSH.

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.

6 participants