Bug 10536 - Recent change broke debugging on device for assemblies with embedded resources
Summary: Recent change broke debugging on device for assemblies with embedded resources
Status: RESOLVED FIXED
Alias: None
Product: iOS
Classification: Xamarin
Component: Tools ()
Version: 6.3.x
Hardware: PC Mac OS
: --- normal
Target Milestone: Untriaged
Assignee: Sebastien Pouliot
URL:
Depends on:
Blocks:
 
Reported: 2013-02-21 10:45 UTC by Marek Safar
Modified: 2013-02-21 21:09 UTC (History)
1 user (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:
RESOLVED FIXED

Description Marek Safar 2013-02-21 10:45:59 UTC
This is due to #9974 when assembly has embedded resources is being rewritten but mdb file is not updated which leads to tokens mismatch between symbol infos and actual method tokens
Comment 1 Sebastien Pouliot 2013-02-21 14:26:51 UTC
I don't think the issue shows with the old cecil, only the newer one (were token changes more easily). 

Do you have a test case handy ? the one for #9974 (e.g. linksdk) does not behave differently with or without my fix.
Comment 2 Marek Safar 2013-02-21 15:20:49 UTC
yes, you need large assembly. I was testing this with mscorlib tests which is at tests/bcl-test/mscorlib/mscorlib.csproj (you need to make it before to generate the csproj)
Comment 3 Sebastien Pouliot 2013-02-21 20:55:13 UTC
master: bb6687414a8cfad66ea514ac2dad8e5d7f2cef54

will merge (and adapt wrt new cecil) to master-3.0 tomorrow morning
Comment 4 Sebastien Pouliot 2013-02-21 21:09:28 UTC
master-3.0: f0fae929630c094af63cdbb7a88825e61e1e0d3b

small (and fast) merge :)