Bug 44024 - [Unified Migration] Will break with XI 10 (monotouch.dll not shipped anymore)
Summary: [Unified Migration] Will break with XI 10 (monotouch.dll not shipped anymore)
Status: VERIFIED FIXED
Alias: None
Product: iOS
Classification: Xamarin
Component: Xamarin.iOS.dll ()
Version: XI 9.99 (iOS 10 previews)
Hardware: PC Mac OS
: Normal major
Target Milestone: Xcode8 (iOS10)
Assignee: Sebastien Pouliot
URL:
Depends on:
Blocks:
 
Reported: 2016-09-06 14:10 UTC by Vincent Dondain [MSFT]
Modified: 2016-09-09 11:36 UTC (History)
6 users (show)

Tags:
Is this bug a regression?: ---
Last known good build:

Notice (2018-05-24): bugzilla.xamarin.com is now in read-only mode.

Please join us on Visual Studio Developer Community and in the Xamarin and Mono organizations on GitHub to continue tracking issues. Bugzilla will remain available for reference in read-only mode. We will continue to work on open Bugzilla bugs, copy them to the new locations as needed for follow-up, and add the new items under Related Links.

Our sincere thanks to everyone who has contributed on this bug tracker over the years. Thanks also for your understanding as we make these adjustments and improvements for the future.


Please create a new report on Developer Community or GitHub with your current version information, steps to reproduce, and relevant error messages or log files if you are hitting an issue that looks similar to this resolved bug and you do not yet see a matching new report.

Related Links:
Status:
VERIFIED FIXED

Description Vincent Dondain [MSFT] 2016-09-06 14:10:40 UTC
It looks like the Migration tool (Classic to Unified) is dependent on monotouch.dll being referenced and built.

I'm getting this error in XS logs with XI 10: https://gist.github.com/VincentDondain/b2b0e0d7dcbaa16253cfda25931c9c84

Our msbuild error when people try to build a project that uses Classic with XI 10 suggests that people run the migration tool so the priority of this bug might be fairly high.
Comment 1 Sebastien Pouliot 2016-09-06 14:25:52 UTC
That's a circular issue, if we can't remove it to allow migration then it stays...

Can that code be changed to load something else ? e.g. if `monotouch.dll` is not present, then try loading `monotouch-migration.dll` ?

If so we could rename the assembly (instead of deleting it) until the next cycle (where it won't exists in any form). At that point the migration message should point people to support (and get people an older version of the tools).

Also is there other assemblies being loaded like this ? e.g. MonoTouch.Dialog-1.dll ?
Comment 2 Miguel de Icaza [MSFT] 2016-09-07 14:11:42 UTC
Team, because this breaks the functionality on the IDE, let us ship the monotouch.dll on this release, and then in the future, we can coordinate with the IDE team what to do about removing the migration tool or using a differently named assembly.

We could just ship the old binaries for now, we do not need to build them.
Comment 3 Sebastien Pouliot 2016-09-07 20:25:02 UTC
https://github.com/xamarin/xamarin-macios/pull/799 adds back the assembly so the migration should work again.
Comment 4 Sebastien Pouliot 2016-09-07 22:07:16 UTC
PR was merged and a 10.0.0.x build should come out of

https://wrench.internalx.com/Wrench/ViewLane.aspx?lane_id=3426&host_id=306&revision_id=802098

for validation.
Comment 5 Vincent Dondain [MSFT] 2016-09-08 13:14:46 UTC
I just tried that new Xamarin.iOS build (with latest XS) and I can confirm that we do get the expected build error when building a classic project. The message mentions how you're supposed to migrate and the migration works as expected.

All good.
Comment 6 Mohit Kheterpal 2016-09-09 11:36:18 UTC
As per comment 5, this issue is resolved for Vincent, hence closing this issue by marking it as Verified.

thanks