Skip to content

Fix DDC reliability on latest Xcode versions, migrate to SPM, more fixes#453

Merged
JoniVR merged 11 commits intomasterfrom
refactor/Preferences
Jun 15, 2021
Merged

Fix DDC reliability on latest Xcode versions, migrate to SPM, more fixes#453
JoniVR merged 11 commits intomasterfrom
refactor/Preferences

Conversation

@JoniVR
Copy link
Copy Markdown
Member

@JoniVR JoniVR commented Jun 12, 2021

Description

This PR makes the following changes:

This PR is mostly needed so we can actually have a working build again (and hopefully merge some of the open PRs that have been blocked by this). It will probably introduce some conflicts in other PRs (storyboard hell) but still seems necessary in order to test them reliably.

Fixes #238
Closes #448

image

@JoniVR JoniVR added bug Issue is a bug (e.g. Crash, …) Type: Enhancement labels Jun 12, 2021
@JoniVR JoniVR requested a review from the0neyouseek June 12, 2021 16:21
@the0neyouseek
Copy link
Copy Markdown
Member

As always, thanks for the awesome work @JoniVR
I'll look into reviewing this on Monday

Copy link
Copy Markdown
Member

@the0neyouseek the0neyouseek left a comment

Choose a reason for hiding this comment

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

Everything looks good to me ! Great job as always 👌

// MonitorControl
//
// Created by Joni Van Roost on 22/11/2020.
// Copyright © 2020 Guillaume Broder. All rights reserved.
Copy link
Copy Markdown
Member

@the0neyouseek the0neyouseek Jun 15, 2021

Choose a reason for hiding this comment

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

We should remove my name from the "Organization" field in the xcodeproj to remove copyright mentions when generating new files. (And also remove them from current files in the future)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done 🙂 The only place I haven't updated is the bundle identifier since I thought that'd probably have an impact on other things.

@JoniVR JoniVR merged commit 1c611b0 into master Jun 15, 2021
@JoniVR JoniVR deleted the refactor/Preferences branch June 15, 2021 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Issue is a bug (e.g. Crash, …)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix dependencies DDC commands not being issued reliably when building with Xcode > 11.3.1

2 participants