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

@0x41c 0x41c commented Jun 5, 2023

Copy link
Copy Markdown
Contributor

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

62f commented Jun 12, 2023

Copy link
Copy Markdown

2129 @randomnessExistance No, more like so:

/pull/2129

@Robertearl5770

Copy link
Copy Markdown

This 2 file changed was that on the ish shell ?

@AimSD23nSiR AimSD23nSiR left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Random

@AimSD23nSiR AimSD23nSiR left a comment

Copy link
Copy Markdown

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