Skip to content
New issue

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

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

Already on GitHub? # to your account

iOSBuild relies on xbuild instead of mdtool #945

Merged
merged 1 commit into from
Sep 11, 2015
Merged

Conversation

molinch
Copy link
Contributor

@molinch molinch commented Sep 10, 2015

iOSBuild relies on xbuild instead of mdtool (Xamarin pushes to use xbuild instead of mdtool).

This implies some changes for iOSBuild parameters:

  • Configuration only contains the configuration now (Debug, Release, Adhoc, AppStore) and no longer the platform. This is an additional property called Platform (iPhone vs iPhoneSimulator)
  • BuidlIpa true/false to indicate wether you want an IPA file to be produced

@molinch
Copy link
Contributor Author

molinch commented Sep 10, 2015

@gshackles As said on Twitter this is the PR.

@thomasvdb
Copy link
Contributor

Great!
If the pull request is merged, the issue #940 can be closed.

@forki
Copy link
Member

forki commented Sep 11, 2015

thx a lot

forki added a commit that referenced this pull request Sep 11, 2015
iOSBuild relies on xbuild instead of mdtool
@forki forki merged commit 8a3e643 into fsprojects:master Sep 11, 2015
@thomasvdb
Copy link
Contributor

Having a strange issue with the location of the IPA file.
If I specify the OutputPath to, for example, C:\Projects\iOS\ then all the sources are placed inside that folder during the build but the IPA file itself is placed one directory higher (C:\Projects in the example).

Any idea what the issue could be?

@molinch
Copy link
Contributor Author

molinch commented Sep 15, 2015

Absolutely no idea, we don't use this parameter on our side. Does it behave correctly with android builds?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants