Skip to content
This repository has been archived by the owner on Jan 24, 2021. It is now read-only.

Fix ignored assemblies. Loads them as late as possible. #638

Closed
wants to merge 6 commits into from

Conversation

albertjan
Copy link
Contributor

Loads the assemblies and scans through them when it is asked for it the first time. Updates the ignored assemblies when they're changed.

@albertjan
Copy link
Contributor Author

@grumpydev isn't the adats used to scan for the bootstrapper? Shouldn't it ignore the ignored assemblies when doing that?

@grumpydev
Copy link
Member

It is, but the ignored assemblies are set in the bootstrapper, and it's only done once on app startup, so I'm not too concerned about that.. as far as I know we've never had an issue where the bootstrapper locator crashed.

@albertjan
Copy link
Contributor Author

There has been see issue #629

@grumpydev
Copy link
Member

That says there's issues with the scanner, but not that the bootstrapper locator crashes.. I've seen scanner issues before, but never had them in the locator.

@grumpydev
Copy link
Member

Just added a nancy app to an existing mvc site (as mentioned in #629) and it works fine.

@albertjan
Copy link
Contributor Author

Allright but the adats does not ignore the ignored assemblies when looking for a bootstrapper.

@grumpydev
Copy link
Member

I didn't say it did, I said I didn't think that was a problem :-)

I'm actually thinking I might set a default in the static ctor of ADATS that ignores any assembly that doesn't have a nancy reference.. that should speed up the initial scan for the bootstrapper somewhat, and fix your concerns about it blowing up?

@albertjan
Copy link
Contributor Author

That would be a smart fix indeed! :) I didn't really have that many concerns about it, but the remaining concerns would definatelly be fixed :P. I didn't know you could see which references an assembly has.

@thecodejunkie
Copy link
Member

Quite handy..we added a smilar convention to another place in Nancy.
Basically from an Assembly you can call GetReferencedAssemblies to get info
on them

On Thu, Jan 17, 2013 at 11:14 AM, Albert-Jan Nijburg <
notifications@github.com> wrote:

That would be a smart fix indeed! :) I didn't really have that many
concerns about it, but the remaining concerns would definatelly be fixed
:P. I didn't know you could see which references an assembly has.


Reply to this email directly or view it on GitHubhttps://github.com//pull/638#issuecomment-12361706.

@grumpydev
Copy link
Member

See #916

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

Successfully merging this pull request may close these issues.

3 participants