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.

Sign up for GitHub

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

Jump to bottom

Load built-in Chocolatey functions, then load extensions #1200

Closed
ferventcoder opened this issue Mar 13, 2017 · 2 comments
Closed

Load built-in Chocolatey functions, then load extensions #1200

ferventcoder opened this issue Mar 13, 2017 · 2 comments
Assignees
@ferventcoder
Labels
5 - Released Bug Priority_HIGH
Milestone
0.10.4

Comments

@ferventcoder
Copy link
Member

There appears to be a bug in loading that doesn't make it very deterministic in not loading a function if it is already loaded by newer versions of Chocolatey.
@ferventcoder ferventcoder added this to the 0.10.4 milestone Mar 13, 2017
@ferventcoder ferventcoder self-assigned this Mar 13, 2017
@ferventcoder ferventcoder changed the title Load Chocolatey functions, then load extensions Load built-in Chocolatey functions, then load extensions Mar 13, 2017
@ferventcoder ferventcoder mentioned this issue Mar 13, 2017
Closed
@ferventcoder
Copy link
Member Author

It appears that this has been an issue since 0.9.9.5 with #230 and specifically 039f70a

@ferventcoder
Copy link
Member Author

This is fixed for 0.10.4.

ferventcoder added a commit that referenced this issue Mar 22, 2017
@ferventcoder
(GH-1200) Export built-in functions early
541daeb 
Export built-in functions prior to loading extensions so that
extension-specific loading behavior can be used based on built-in
functions. This allows those overrides to be much more deterministic.

This behavior of not loading built-in functions first was changed in
039f70a with GH-230. That was first released in 0.9.9.5.
ferventcoder added a commit that referenced this issue Mar 22, 2017
@ferventcoder
Merge branch 'stable'
19ecfac 
* stable:
  (GH-1203) Ensure PowerShell uses CRLF
  (GH-1200) Export built-in functions early
  (maint) Update NuGet.Core
  (maint) update scenarios
  (GH-1054) Document requested execution levels
  (GH-1054) Externalize the app.manifest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ferventcoder ferventcoder

Labels
5 - Released Bug Priority_HIGH
Projects
None yet
Milestone
0.10.4
Development

No branches or pull requests
2 participants
@ferventcoder @gep13