Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Ready] Added new code signing guide #5178

Merged
merged 25 commits into from
Jun 30, 2016
Merged

[Ready] Added new code signing guide #5178

merged 25 commits into from
Jun 30, 2016

Conversation

KrauseFx
Copy link
Member

@KrauseFx KrauseFx commented Jun 21, 2016

@KrauseFx KrauseFx self-assigned this Jun 21, 2016
@fastlane-danger-bot
Copy link

fastlane-danger-bot commented Jun 21, 2016

2 Warnings
⚠️ Pull Request is Work in Progress
⚠️ Before submitting a Pull Request, please create an issue on GitHub to discuss the change. Please add a link to the issue in the PR body.
Big PR

Generated by 🚫 danger


Apple improved code signing a lot with the release of Xcode 8, the following has changed:

- No more `Fix Issue` button, instead all code signing processes run in the background and show the log right in Xcode
Copy link
Contributor

Choose a reason for hiding this comment

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

😢

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling d00580b on codesigning-guide into * on master*.

@@ -0,0 +1,38 @@
# Commong code signing issues
Copy link
Contributor

Choose a reason for hiding this comment

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

*Common


Using the `$(...)` notation, Xcode will use the provisioning profile matching the UUID, and fallback to `Automatic` if it's not provided.

If you use [fastlane](https://fastlane.tools) you don't have to do any extra work, since the environment variable for the latest profile are automatically set by [match](https://fastlane.tools/match)
Copy link
Contributor

Choose a reason for hiding this comment

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

"are automatically set" -> "is automatically set"

@mfurtak
Copy link
Contributor

mfurtak commented Jun 28, 2016

Looking great! Comments are mostly about cleaning up spelling & grammar 👍

- [cert](https://fastlane.tools/cert) will make sure you have a valid certificate and its private key installed on the local machine
- [sigh](https://fastlane.tools/sigh) will make sure you have a valid provisioning profile installed locally, that matches the installed certificate

You basically add the following lines to your `Fastfile`
Copy link
Contributor

Choose a reason for hiding this comment

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

remove "You basically"

@KrauseFx KrauseFx changed the title [WIP] Added new code signing guide [Ready] Added new code signing guide Jun 29, 2016
Add the following lines to your `Fastfile`

```ruby
lane :beta do
Copy link
Contributor

Choose a reason for hiding this comment

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

Wondering if we should use a different lane name here? Maybe build?

@mfurtak
Copy link
Contributor

mfurtak commented Jun 29, 2016

👍

@ohayon
Copy link
Contributor

ohayon commented Jun 29, 2016

This looks 💯! I left one or two comments, but otherwise 🚀 !

- [GettingStarted.md](GettingStarted.md#readme): Find the best code signing approach for your setup
- [XcodeProject.md](XcodeProject.md#readme): How to set up your Xcode project
- [Troubleshooting.md](Troubleshooting.md#readme): Helps you debug signing issues
- [CommonIssues.md](CommonIssues.md#readme): A list of some of the msot common signing issues and how to solve them
Copy link
Contributor

Choose a reason for hiding this comment

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

most is misspelled as msot here

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks

@KrauseFx KrauseFx merged commit cc93a2b into master Jun 30, 2016
@KrauseFx KrauseFx deleted the codesigning-guide branch June 30, 2016 10:05
KrauseFx added a commit that referenced this pull request Jul 16, 2016
* Added new code [signing guide](https://github.com/fastlane/fastlane/tree/master/fastlane/docs/Codesigning) (#5178)
* Support multiple md5 hashes of the PackageApplication script to support multiple Xcode versions (#5391)
* Support relative path like "~/file" (#5350)
* Add warnings when using Xcode's 7 build options with legacy build api (#5229)
* Deprecated provisioning_profile_path (#5106)
* Proper escaping of codesigning identity (#5148)
* Escape code signing identity
* Add support for `rbenv` in `wrap_xcodebuild/xcbuild-safe.sh` (#4625)
* Print out legacy information every time you run gym (#4890)
* Fix gym with path with spaces (#4721)
@KrauseFx KrauseFx mentioned this pull request Jul 16, 2016
KrauseFx added a commit that referenced this pull request Jul 16, 2016
* Added new code [signing guide](https://github.com/fastlane/fastlane/tree/master/fastlane/docs/Codesigning) (#5178)
* Support multiple md5 hashes of the PackageApplication script to support multiple Xcode versions (#5391)
* Support relative path like "~/file" (#5350)
* Add warnings when using Xcode's 7 build options with legacy build api (#5229)
* Deprecated provisioning_profile_path (#5106)
* Proper escaping of codesigning identity (#5148)
* Escape code signing identity
* Add support for `rbenv` in `wrap_xcodebuild/xcbuild-safe.sh` (#4625)
* Print out legacy information every time you run gym (#4890)
* Fix gym with path with spaces (#4721)
@mfurtak mfurtak mentioned this pull request Aug 3, 2016
@fastlane fastlane locked and limited conversation to collaborators Feb 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants