il-repack icon indicating copy to clipboard operation
il-repack copied to clipboard

Mono.Cecil 0.10 support

Open manu-st opened this issue 8 years ago • 11 comments

Hi,

Just wanted to know what was the status of using Mono.Cecil 0.10 in il-repack. I see there is a 0.10 branch, but it is not clear if it is up-to-date.

Thanks!

manu-st avatar Jan 31 '17 22:01 manu-st

The branch should be usable, save for debug (PDB/MDB) support, and more recent fixes (if any). It's stiff missing that part to be merged into master though.

gluck avatar Feb 13 '17 15:02 gluck

Didn't look into it yet, but what's missing for PDB/MDB exactly? Isn't it automatically supported by Cecil?

xen2 avatar Mar 10 '17 00:03 xen2

Update: found what needs to be done for PDB/MDB, investigating now.

xen2 avatar Mar 10 '17 07:03 xen2

FYI, I have everything working in https://github.com/xen2/il-repack/tree/0.10

It currently needs https://github.com/xen2/cecil/tree/ilrepack (little bit temporary, until things get cleaned up and merged back in master)

Specifically, https://github.com/jbevain/cecil/pull/351 and https://github.com/jbevain/cecil/pull/353 will allow full Native PDB support alongside Portable PDB.

xen2 avatar Mar 16 '17 02:03 xen2

Side note: I had weird problem with Baml (it was removing all extra assembly info, including from non merged types. Fixed in this commit: https://github.com/xen2/il-repack/commit/837337023f427368459a78a75b027c7e52fa4a1c

xen2 avatar Mar 16 '17 02:03 xen2

@xen2 your branch looks good, how are the symbols generated ? is it still native PDB or Portable ?

gluck avatar Mar 30 '17 13:03 gluck

It supports both of them. New cecil added portable PDB and recently reworked and integrated my native PDB branch.

xen2 avatar Mar 30 '17 16:03 xen2

@gluck any chance of this being pulled in any time soon?

SimonCropp avatar May 26 '17 06:05 SimonCropp

note that there were some upstream cecil pdb fixes recently image

SimonCropp avatar May 26 '17 06:05 SimonCropp

Any chance we could integrate cecil 0.10 on master? Looks like 0.10 was released as stable and 0.10.1 was already released

Thanks

tephe avatar Jan 26 '19 15:01 tephe

Any update on this? All bits for portable PDB support are added to Mono.Cecil.

MaxXor avatar May 20 '20 14:05 MaxXor

Just bumping this thread here as portable pdbs are now the defaults.

mattleibow avatar Mar 21 '23 08:03 mattleibow

Our master branch is now using the very latest Cecil.

KirillOsenkov avatar Jan 01 '24 23:01 KirillOsenkov

Thank you.

MaxXor avatar Jan 02 '24 16:01 MaxXor