-
Notifications
You must be signed in to change notification settings - Fork 525
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
Paket.PowerShell #839
Paket.PowerShell #839
Conversation
This looks interesting. I think we need detailed docs on how to install / use this. |
mhm. that's interesting, but at the moment most people install paket via the bootstrapper (without ever using nuget at all). this would only be something for you? I wonder if this should be something in https://github.com/fsprojects/Paket.VisualStudio |
I think this would be used by a lot of developers who just want to stay in Visual Studio. The PowerShell integration also doesn't require Visual Studio and makes it easier to use from |
I don't mind to add the small file. But I don't really think people will use it like that. Also it will not really work from inside VS since we still have the file locking issues from VS. We worked around that in the paket.visualstudio project by unloading and reloading projects. |
attempt to load for paket.exe as assembly
The translation from PowerShell args to UnionArgParser args looks pretty straight forward. I implemented a couple commands with my last commit. My plan for deployment isn't working quite yet: I'm not sure if the I'm hoping there is a clean solution. |
When I This looks like an FSharp.Core redirect issue. :/ Like this one: |
I think I'll trust you on this (since I'm not a heavy PS user). Just let me know when you think this is ready for prime time. |
It is functional now. Several enhancements can be made, but the basics are in place. I ended up not modifying the |
@@ -30,6 +30,8 @@ | |||
<StartArguments>update</StartArguments> | |||
<StartAction>Project</StartAction> | |||
<StartProgram>paket.exe</StartProgram> | |||
<StartWorkingDirectory>C:\projects\SourceLink</StartWorkingDirectory> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@latkin @enricosada I hope we can solve this issue with private debug info in F# vNext...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i really hope so, see dotnet/fsharp#496 , the PTVS has it.
We can import PTVS code or use a new common project system
Pls @forki add yours missing features and bugs to issue, i'll check if PTVS supports it already
@@ -0,0 +1,2 @@ | |||
# copies System.Management.Automation.dll from the GAC to this folder | |||
cp ([PSObject].Assembly.Location) . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Powershell.exe should be invoked with exra args "-executionpolicy bypass" so that it works even for those nonbelievers like @forki who don't use powershell and still have default Restricted policy ;-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
actually I have "remote-signed", but I wonder if we can get rid of that step.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@latkin doesn't seem to work?!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems to work if I change the order of the params as described in https://blog.netspi.com/15-ways-to-bypass-the-powershell-execution-policy/
🚮
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ya, same as FSI - flags before the script arg are flags for the script engine, flags after the script arg are flags for the script itself.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, that makes totally sense as explanation.
this is released in 1.13 - but I think we need some docs |
Good morning. Thank you @latkin! |
This is just a design proposal right now with stuff stubbed out. There are two main purposes:
have Visual Studio automatically do Paket-Restore when the solution is opened so that we finally have a workable solution for paket.targets per package #417This can be make possible by having a single NuGet package,
Paket.PowerShell
, installed inpackages.config
.I stubbed out the commands and it looks quite doable:
To do development of this, you just need to build the project and import:
That warning is suppressible. I did so in
init.ps1
. I found it much easier to start everything withPaket-
.