Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

NHibernate.dll in Custom Actions #86

Closed
NikGovorov opened this issue Jun 9, 2017 · 3 comments
Closed

NHibernate.dll in Custom Actions #86

NikGovorov opened this issue Jun 9, 2017 · 3 comments

Comments

@NikGovorov
Copy link

Hi Oleg,

I have an issue packing NHibernate.dll to *.CA.dll, it's packed just as NHibernate(without dll extension). I assume we loose the file extension on the following line:

return IO.Path.Combine(dir, asm.ManifestModule.ScopeName);
. Probably it happens because NHibernate is ilmerged. The only workarounds I see now are run-time ones: custom assembly resolver, assemblyBinding/dependentAssembly/codeBase section in CustomAction.config or even just rename 'NHibernate' file to 'NHibernate .dll' before executing a custom action.

Thoughts?

@oleg-shilo oleg-shilo added the bug label Jun 9, 2017
@oleg-shilo
Copy link
Owner

It's a defect. Will look at it.
As a temp work around indeed AppDomain assembly resolver seems to be the most practical approach.

oleg-shilo added a commit that referenced this issue Jun 9, 2017
@oleg-shilo
Copy link
Owner

Fixed (commit a248dd6). The fix will be available with the very next release. There are some pull requests to be released so I will be making a release early next week.

Thank you for reporting the defect.

@NikGovorov
Copy link
Author

Great job, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants