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

Change install location or add logic to support beta powershell versions #34

Open
bobfrankly opened this issue May 26, 2017 · 2 comments

Comments

@bobfrankly
Copy link
Contributor

Regarding Line 24 of:
https://github.com/Unitrends/unitrends-pstoolkit/blob/master/Unitrends/Install.ps1

I'm suggesting changing the install path to C:\Program Files\Powershell\Modules
OR making decision logic for the install location, based on the $env:psmodulepath variable.

The reasoning for this is that beta versions of powershell (the one's I've observed so far anyway) do not have the "C:\Windows\System32\WindowsPowerShell\v1.0\Modules\Unitrends" path within the $env:psmodulepath variable, so beta versions of powershell are incapable of using/testing the unitrends module as it stands.

This reasoning may also apply to the open-source standalone version of powershell in it's entirety, but I have not tested that far out.

@MartynKeigher
Copy link
Contributor

Hey @bobfrankly

Not been on here for quite some time... Job changes, relocation, new roles/responsibilities, etc... but, I'm interested in getting this going. Still a thing you need or did you get this taken care of already??

Let me know & if its still a thing you need... I'll work on it.

://mk

@bobfrankly
Copy link
Contributor Author

I don't need it at this time, but it was a detail that I encountered testing out PSCore. Seeing as PSCore is the direction that they're taking for future PS development, I made this issue as a heads up.

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

No branches or pull requests

2 participants