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.

Sign up for GitHub

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

Jump to bottom

Update README.ios.md #3186

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Update README.ios.md #3186

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
6395d7f
Update README.ios.md
petersmythe May 26, 2024
bc02f80
Update README.md
petersmythe May 26, 2024
ff0b3af
Update README.md
petersmythe May 26, 2024
df15276
Update README.ios.md
petersmythe May 27, 2024
9a6af9d
Update README.md
petersmythe May 31, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.ios.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

## Prerequisites (Mac OS X)

* Xcode 14 or higher
* Xcode 14 or higher (and install platform iOS 12.0 SDK or higher)
* macOS 12 or higher

(optional) To automatically install the companion app on a connected, authorized iOS device, install ideviceinstaller via Homebrew:
Expand Down
11 changes: 9 additions & 2 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,10 +76,17 @@ building on versions below the ones stated. To install on a device,
you **must** have a valid Apple Developer account license and have
added the relevant mobile provisioning profiles from the Developer
portal to your Xcode organizer (see Apple's website on instructions on
how to do this).
how to do this). The profile should have these **exact** details, and
should include a development certificate (a distribution certificate is
not necessary):

```conf
Name: MIT App Inventor Development
App ID: edu.mit.appinventor.*
Comment on lines +84 to +85
Copy link
Member

Choose a reason for hiding this comment

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

We do not want people using MIT in the identifiers for their testing. They should be using something a namespace they might reasonably control. Worst case, they could use a reverse DNS style entry based on their email address or their GitHub username.

Copy link
Author

Choose a reason for hiding this comment

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

This does not work, in my (very limited) experience. The file AICompanionApp.xcodeproj/project.pbxproj has PRODUCT_BUNDLE_IDENTIFIER = edu.mit.appinventor.aicompanion3 and this is the App ID that must be included in the provisioning profile, otherwise Xcode reports:

image

and the ant ios build fails with:

image

Alternatively, we could include instructions to update AICompanionApp.xcodeproj/project.pbxproj in multiple places to a namespace that they control. What do you think @ewpatton ?

```

To build the MIT App Inventor companion, you will need to create a
file called AICompanionApp.xcconfig in the components-ios directory
file called AICompanionApp.xcconfig in the `appinventor` directory
that sets your development team. The easiest way to do this is to copy
the AICompanionApp.xcconfig.sample file and edit it. Alternatively,
create a file with the following line:
Expand Down