-
-
Notifications
You must be signed in to change notification settings - Fork 764
Conversation
… superceeded by the MS.Owin package.
@leastprivilege @brockallen MSTest is tightly coupled with Visual Studio and makes things akward from a single click build and in CI scenarios. The problem is, instead of being able to use the test runner via nuget solution package it requires Visual Studio to be installed (and it's a disaster!). This is sort-of when run on dev computers but wont work at all on build servers, i.e. teamcity.codebetter etc. Would it be possible to convert the tests to xunit? The xunit runner integrates will with the VS Testrunner so your experience will be practically similar... Please research / chew on it before rejection this :) (I can do this conversion and send as separate PR) |
I have no objections - next week I have more time. Maybe you need to give me some intro to all that via skype. |
Sure np. Am away this weekend. Hit me up on skype next week when you have a moment. |
Update version on build
Il-Repack
@leastprivilege Am getting some help with this from @mauricedb . It appears we're having some issues with ILRepack and internalizing the Autofac dependency (@mauricedb - please shed some light here). Do you have any strong opinions regarding Autofac or if we find an equally competent container that is internalizable, would you be open to that? |
Running ILRepack with Autofac.dll results in an exception: Autofac actually refers to System.Core 2.0.5.0 which is the result of it being a Portable Class Library. See: https://github.com/autofac/Autofac/wiki/Frequently-Asked-Questions |
I'd rather add Alex to this thread than moving away from autofac. |
i pinged him. |
@mauricedb Is there a changeset / branch one can checkout and easily reproduce the problem? |
@damian Use the ILRepack branch and remove the exclusion of Autofac.dll from the ILRepack task in default.ps1. I also have a real simple repro using a console app on disk. I can push that to GitHub if needed. |
I tested ILMerge and was able to internalize Autofac.dll into a test assembly. |
Thanks @alexmg! @mauricedb Switch to ILMerge then I guess. |
It looks like it. The exception is a Mono.Cecil.AssemblyResolutionException. |
Looks like a known problem with with Mono.Cecil. jbevain/cecil#152 No clue as to if / when it will be addressed. I suggest go with ILMerge if it is known to definitely work. |
Already on it. Seems like a pretty trivial task so far. |
IlMerge instead of ILRepack
…ure conversion for our 'private' types.
@leastprivilege Now fully embeddable and dependency free. Ready for a look. To build, open powershell prompt: You will probably need to tweak the nuspec details What it looks like in a sample project: Note No WebAPI references here! Typical packages.config of a consumer: <?xml version="1.0" encoding="utf-8"?>
<packages>
<package id="Microsoft.Owin" version="2.1.0" targetFramework="net45" />
<package id="Microsoft.Owin.Host.HttpListener" version="2.1.0" targetFramework="net45" />
<package id="Microsoft.Owin.Hosting" version="2.1.0" targetFramework="net45" />
<package id="Owin" version="1.0" targetFramework="net45" />
<package id="Thinktecture.IdentityServer" version="3.0.0" targetFramework="net45" />
</packages> |
Thanks to @mauricedb for helping out 😄 |
good job! thanks to you both! Will have a look later/tomorrow! |
Build script / nuget package
Hi guys, I would be interested to know why the ILMerge is internalising so many assemblies. Microsoft.Owin.* thanks |
@gregpakes it's to make idsvr completely self-contained |
Thanks @brockallen How does this work with embedded scenarios? I am embedding and I am concerned about potential clashes between the embedded and the actual versions of the assemblies. I haven't had any issues yet, i'm just thinking aloud. |
That's the point, ilmerging prevents any clashes.
|
❗ This is a work in progress, please do not pull / merged yet. Opening it now so you can see and comment on progress :) ❗Ready to go!
Fixes #60
Run testslets skip this for now, can introduce to the build script in a separate bit of work.