These are chat archives for chocolatey/ChocolateyGUI

10th
May 2017
Rob Reynolds
@ferventcoder
May 10 2017 22:00
I think I found the issue
now I just need to see if there is a reason
Rob Reynolds
@ferventcoder
May 10 2017 22:12
Here's the commit - chocolatey/choco@61cd084
Rob Reynolds
@ferventcoder
May 10 2017 22:24
looks like it is able to be adjusted but may cause some issues if folks are reusing the functionality with multiple commands
Rob Reynolds
@ferventcoder
May 10 2017 22:46
Okay @RichiCoder1 this is where I need your help
Richard Simpson
@RichiCoder1
May 10 2017 22:47
Sup?
Rob Reynolds
@ferventcoder
May 10 2017 22:47
When I made that adjustment to config to pull from the container instead of creating a new one every time, it returns only the list of installed software for any source
in the GUI
where do you query?
Richard Simpson
@RichiCoder1
May 10 2017 22:48
What do you mean, where do I query? Like where do I do .Run and list and such?
Rob Reynolds
@ferventcoder
May 10 2017 22:48
I'm looking at ChocolateyService
I should just look at the bind for loading a source
and see where it goes from there
Rob Reynolds
@ferventcoder
May 10 2017 22:49
That's for install packages right?
Okay, badly named function sir
Richard Simpson
@RichiCoder1
May 10 2017 22:49
Basically UI -> ChocolateyGui.Services.ChocolateyService <- Wamp (RPC) -> ChocolateyGui.Subprocess.ChocolateyService
Rob Reynolds
@ferventcoder
May 10 2017 22:50
I need where you list out packages on a source, not the local ones
I believe. That name is historical
Rob Reynolds
@ferventcoder
May 10 2017 22:52
I'm not seeing a reason that would still have LocalOnly set to true
Richard Simpson
@RichiCoder1
May 10 2017 22:52
?? What do you mean??
Rob Reynolds
@ferventcoder
May 10 2017 22:52
read my first message
Richard Simpson
@RichiCoder1
May 10 2017 22:54
I'm not doing anything fancy there either. What you see is what you're getting.
Rob Reynolds
@ferventcoder
May 10 2017 22:55
ChocolateyConfiguration was recreated everytime you called run or list
now it won't be
be scared
Richard Simpson
@RichiCoder1
May 10 2017 22:56
uuhhhhhh
Rob Reynolds
@ferventcoder
May 10 2017 22:56
I am going to see what I can fix here.
Richard Simpson
@RichiCoder1
May 10 2017 22:56
Yah, that's going to break ChocoProcess. The first commands we run are A) Get Sources and B) get locally installed packages.
If config is carrying over, that's gonna break everything.
Well, actually, that's somewhat of a lie. But it will cause some really weird behaviors
Rob Reynolds
@ferventcoder
May 10 2017 22:57
Now you understand what I'm saying. If you keep the same Lets.GetChocolatey and reuse it after calling Run or List, it is a bug
Without resetting the config. But beware, there is still some issue here as it doesn't appear this fixed the problem yet
Richard Simpson
@RichiCoder1
May 10 2017 22:58
We do a fresh Lets.GetChocolatey() for every call, so it's persisting the config between those?
Rob Reynolds
@ferventcoder
May 10 2017 22:58
No, it should not be
that's what I'm investigating
Richard Simpson
@RichiCoder1
May 10 2017 22:58
Cool, I think we're on the same page now haha
Good luck! Let me know if you have any further questions.
I'm also available on Skype/Other if necessary
Rob Reynolds
@ferventcoder
May 10 2017 23:00
So low down on Licensed code - it changes from ChocolateyConfiguration to LicensedChocolateyConfiguration (a subclass of ChocolateyConfiguration), then it sets that on the container. For Licensed code to run, it needs a valid license and the configuration to be an instance of LicensedChocolateyConfiguration
So I switched it to use the container to get the Configuration, which should pull back the right one. I was thinking there may be some side effects to this, so I'm trying to go through and fix those items.