Update Makefile - refactor swift-build option by ahmetgeymen · Pull Request #214 · XcodesOrg/xcodes

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Merged

MattKiazyk merged 1 commit intoXcodesOrg:mainfrom

ahmetgeymen:bugfix/swift-build-warning

Sep 17, 2022

Merged

Update Makefile - refactor swift-build option#214

MattKiazyk merged 1 commit intoXcodesOrg:mainfrom

ahmetgeymen:bugfix/swift-build-warning

Conversation

@ahmetgeymen

Copy link

Contributor

trying to fix warning message comes from swift build during installation with brew.

"warning: '--build-path' option is deprecated; use '--scratch-path' instead"

reason: swiftlang/swift-package-manager#4195

fixed: "warning: '--build-path' option is deprecated; use '--scratch-path' instead"

@ahmetgeymen ahmetgeymen requested a review from a team as a code owner

September 14, 2022 22:49

@MattKiazyk MattKiazyk added the chore

Repo upkeep

label

Sep 17, 2022

@MattKiazyk MattKiazyk merged commit f943fcb into XcodesOrg:main

Sep 17, 2022

@MattKiazyk

Copy link

Contributor

Thanks @ahmetgeymen

ahmetgeymen reacted with thumbs up emoji

@ahmetgeymen ahmetgeymen deleted the bugfix/swift-build-warning branch

September 17, 2022 08:49

@MattKiazyk

Copy link

Contributor

@ahmetgeymen letting you know that I've reverted this change in main for now, because we're still building with Swift 5.6 at the moment and that option doesn't exist yet

@ahmetgeymen

Copy link

Contributor Author

Ok then, not problem.

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

Reviewers

@MattKiazyk MattKiazyk MattKiazyk approved these changes

Assignees

No one assigned

Labels

chore

Repo upkeep

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ahmetgeymen @MattKiazyk