Skip to content

Conversation

@rozd
Copy link
Contributor

@rozd rozd commented Sep 18, 2019

Initial support for new types of Apple certificates introduced in Xcode 11

Adds support for new types of Apple certificates introduced in Xcode 11
@Jeehut
Copy link

Jeehut commented Oct 2, 2019

@rozd Is there a way to install your fork with the fix without checking out the repo and installing all the required tools? Could you for example upload a build for macOS maybe? :)

macOSInstallerCertificateNames = []string{
"3rd Party Mac Developer Installer", // type: "Mac Installer Distribution"
"Developer ID Installer", // type: "Developer ID Installer"
"Apple Development", // type: "iOS Development for Xcode 11", id: "83Q87W3TGH"
Copy link
Contributor

Choose a reason for hiding this comment

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

it seems Apple Development and Apple Distribution should be moved to the macOSCertificateNames, they are macOS codesigning certificates.

you can validate it, just create a new macOS project using Xcode 11, with default options. The project's Code Signing Identity will be set to: Apple Development. Now if you execute codeswigndoc against this project, it will fail: failed to analyze archive, error: installed certificate not found with common name or sha1 hash: Apple Development: XYZ

Copy link
Contributor

@godrei godrei left a comment

Choose a reason for hiding this comment

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

Hi @rozd thanks for the PR, I found 1 issue, please fix it.

@wayne530
Copy link

wayne530 commented Oct 3, 2019

@godrei There are many people waiting for this PR. You could easily have fixed and merged it by now. It is literally a two line change. Please stop dragging your feet...

@godrei
Copy link
Contributor

godrei commented Oct 4, 2019

Hi, @wayne530 I just wanted to let the original author finish the PR. My original plan was to do what you recommended, after giving some time for the author to fix it.

Other than that we do not merge PRs with open change requests, to keep the master branch as stable as possible.

@godrei godrei merged commit 5b0b8e8 into bitrise-io:master Oct 4, 2019
@rozd
Copy link
Contributor Author

rozd commented Oct 7, 2019

Sorry guys I missed the comments on this thread :/ Thank you @godrei to gave me a chance to fix an issue in this PR 😃

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.

4 participants