Paket.PowerShell support for Package Manager Console

[ctaggart]

This is for #874 and is not ready yet to be merged. It is in progress and ready for review. I could use some help solving the async / trace dilemma. PowerShell requires that the tracing be on the same thread and it appears async puts it on another.

[ctaggart]

This thread ID check appears it will work. Messages in async will not print, but also will not error out the program now.

[ctaggart]

I asked around at work about how to get all the messages on the same thread. I was told to look at Async.SwitchContext. There is some good related code already in Utils.fs. Hoping that I can capture SynchronizationContext.Current instead of the ManagedThreadId and use it to put all the messages on the right thread for PowerShell. I'll try it out tonight.

[ctaggart]

I spent hours on this, this evening, without any luck. I opened up a stackover question:
http://stackoverflow.com/questions/30859885/post-messages-on-async-thread-to-original-thread-in-f

[ctaggart]

I believe this is ready to go. Logging now works correctly in PowerShell and in Package Manager Console. It displays output as it goes.

I created type EventSink and function processWithLogging to allow the trace messages to be output on the main PowerShell thread, which is required by PowerShell (according to the error messages I get when I don't do so).

[ctaggart]

The feature is Paket.PowerShell support for Paket Manager Console and to add support I had to refactor Logging to use Trace observable.

[forki]

cool. thanks

[forki]

maybe it even fixes the logging issues in paket.vs - you never know ;-)

[forki]

Did this by any chance changed the exit code behavior? It seems we always get exit code 1 - even on success.

[ctaggart]

:/ I'll have a look.

[forki]

I mean it's probably something different, but I have no idea at the moment
On Jun 20, 2015 8:18 PM, "Cameron Taggart" [email protected] wrote:

:/ I'll have a look.

—
Reply to this email directly or view it on GitHub
#875 (comment).

[ctaggart]

Looking at the changes I made to Program.fs, I think I see this issue.

[ctaggart]

@forki Just to be clear, the exit code wasn't broken. What was broken was the exception not being logged.

[forki]

I tried to update Paket.VisualStudio and got:

What do I need to do?

[ctaggart]

Paket.Logging.event is public. event.Publish can be subscibed to.
On Jun 23, 2015 1:32 AM, "Steffen Forkmann" [email protected]
wrote:

I tried to update Paket.VisualStudio and got:

[image: image]
https://cloud.githubusercontent.com/assets/57396/8301978/1e504036-1993-11e5-88bf-60f4f1eb7fdc.png

What do I need to do?

—
Reply to this email directly or view it on GitHub
#875 (comment).

[forki]

I don't get it working. Could you please help me a bit? thx

[dungpa]

@forki Can you change event to use DelegateEvent so that you could use += notation in C#? See https://github.com/fsprojects/VisualFSharpPowerTools/blob/40bd0f678925a4e64c60d09014660e1f03c9c1e0/src/FSharpVSPowerTools.Logic/ShellEventListener.fs#L28

[forki]

fsprojects/Paket.VisualStudio@8bb932a does this look OK?