These are chat archives for chocolatey/ChocolateyGUI

8th
Jun 2017
Manfred Wallner
@mwallner
Jun 08 2017 11:12
hey @RichiCoder1 - the current /develop GUI fails to pull the current version of packages:
image.png
can you reproduce this / is this actually a bug?
Manfred Wallner
@mwallner
Jun 08 2017 13:46
  • not the current, but the latest available ...
Manfred Wallner
@mwallner
Jun 08 2017 14:24
the "latestVersion" got from chocolateyService is null ...
image.png
Manfred Wallner
@mwallner
Jun 08 2017 14:44
(couldn't debug into the service contract - maybe @RichiCoder1 or @gep13 could take a look? :-))
Richard Simpson
@RichiCoder1
Jun 08 2017 15:01
@mwallner If you set VS's Startup to start both Subprocess & GUI (In that order), you can debug into subprocess where all the logic is hapenning
That being said, it must have just broke. Was working literally yesterday
Manfred Wallner
@mwallner
Jun 08 2017 18:25
do you think you can fix it soon (like, today :D) - I'd have the chance to 'hand out' a beta to a couple of collegues tomorrow to play with/ QA the thing before we do a 0.15 release ;-)
Richard Simpson
@RichiCoder1
Jun 08 2017 18:35
Workin fine here :|
Manfred Wallner
@mwallner
Jun 08 2017 18:36
hm ..
oh: I think I broke the AppVeyor build - but it seems to be Nuget/Cake related ...
Richard Simpson
@RichiCoder1
Jun 08 2017 18:37
xD
Manfred Wallner
@mwallner
Jun 08 2017 18:37
image.png
Richard Simpson
@RichiCoder1
Jun 08 2017 18:38
Hmm. Looks like we're hitting out MyGet quota or something. @gep13 ?
Kim J. Nordmo
@AdmiringWorm
Jun 08 2017 18:41
@RichiCoder1 probably used up the 500MB quota, I suggest perhaps deleting older versions on myget (Possibly enable Package Retention cleanup)
Richard Simpson
@RichiCoder1
Jun 08 2017 18:42
I think that's what I'll do. I can't configure cleanup as I'm only a contributer, not an admin
Will consult w/ @gep13 but will probably just nuke all but latest prerelease and just start only keeping 10 or so pre. We should probably also look into applying for OSS license
Manfred Wallner
@mwallner
Jun 08 2017 18:44
uh, this might help @RichiCoder1 - it only fetches the latest version from the public feed (if I enable chocolatey.org/api/v2) I get following
image.png
Kim J. Nordmo
@AdmiringWorm
Jun 08 2017 18:47
@RichiCoder1 yeah, I think keeping the latest 10 would be a good choice there.
Richard Simpson
@RichiCoder1
Jun 08 2017 18:47
Hmmm. We do some custom logic as choco outdated doesn't provide a convenient list logic we can use. Still confused, though, as I could swear the logic we're using takes into account multiple feeds.
Let me check something
Manfred Wallner
@mwallner
Jun 08 2017 18:50
chocolatey/ChocolateyGUI#445 -> @RichiCoder1
Richard Simpson
@RichiCoder1
Jun 08 2017 18:57
@mwallner I think I may already have a fix
Random question for room: Do you like that GUI only shows latest for packages that have updates, or would you prefer to see the latest version of a packages regardless?
Manfred Wallner
@mwallner
Jun 08 2017 18:58
@RichiCoder1 great!
and: I don't get your question x_x
Richard Simpson
@RichiCoder1
Jun 08 2017 18:59
or
Manfred Wallner
@mwallner
Jun 08 2017 19:01
hm .. good point, I think the one where only those with a newer version are displayed is pretty intuitive
Richard Simpson
@RichiCoder1
Jun 08 2017 19:01
Agreed.
Manfred Wallner
@mwallner
Jun 08 2017 19:01
but it should work for any feeds :D (internal also ;-))
Richard Simpson
@RichiCoder1
Jun 08 2017 19:02
@mwallner You can pull down source right? Mind if a create a PR and then have you pull it down and test?
Manfred Wallner
@mwallner
Jun 08 2017 19:03
of course, let's fix this :-)
Kim J. Nordmo
@AdmiringWorm
Jun 08 2017 19:06
Regarding the show only if new version is available or not, I think it looks better when it only shows when there is an update available.
With that said though, trying to look at it from a different perspective.
Let's say the user doesn't know that it only shows for versions with an update available, they would possibly think that there is something wrong and chocolatey gui is unable to grab the latest version for those packages.
Manfred Wallner
@mwallner
Jun 08 2017 19:10
in this case, would you prefer newer versions to be highlighted, or 'same' version numbers to be grayed-out ..?
Kim J. Nordmo
@AdmiringWorm
Jun 08 2017 19:10
Personally I would think same version to be grayed-out
Manfred Wallner
@mwallner
Jun 08 2017 19:13
I think this is a nice feature request, at least worth submitting an issue (don't think it should be mixed with #445)
Richard Simpson
@RichiCoder1
Jun 08 2017 19:19
@mwallner chocolatey/ChocolateyGUI#446
Let me know how that works for you
Richard Simpson
@RichiCoder1
Jun 08 2017 19:31
Also went ahead and killed all pres that had < 5 downloads. Will kill the rest later.
Manfred Wallner
@mwallner
Jun 08 2017 19:40
works great @RichiCoder1 - will review changes and merge into develop - ok?
Richard Simpson
@RichiCoder1
Jun 08 2017 19:40
:+1:
Richard Simpson
@RichiCoder1
Jun 08 2017 20:02
Random: Just improved the way we handle scaling. If you have a high dpi setup, markdown should (should) render much cleaner. Also, package icons should be much pretty. Apparently I wasn't handling scaling properly, so they'd get rendered at 65px and then upscaled to something like 100px (poorly)
Now they render at the correct size. Especially noticable for packages w/ SVG icons.
Manfred Wallner
@mwallner
Jun 08 2017 20:03
@RichiCoder1 ++
sadly the bot doesn't listen to me :D
Kim J. Nordmo
@AdmiringWorm
Jun 08 2017 20:04
@mwallner skip the space
@RichiCoder1++
The chocolatiest bot this side of the Mississippi
@choco-bot
Jun 08 2017 20:04
richicoder1 leveled up! (Karma: 10)
Manfred Wallner
@mwallner
Jun 08 2017 20:05
-_- :astonished:
Richard Simpson
@RichiCoder1
Jun 08 2017 20:09
?????Choco is carrying over previously set config settings for new commands?????
Manfred Wallner
@mwallner
Jun 08 2017 20:12
talking of chocolatey.dll or choco.exe ?
Richard Simpson
@RichiCoder1
Jun 08 2017 20:12
dll
Manfred Wallner
@mwallner
Jun 08 2017 20:13
CC @ferventcoder ^^
Richard Simpson
@RichiCoder1
Jun 08 2017 20:15
Yup. I don't know what I did, but I'm 100% certain that config settings are carrying over across commands.
Hmm. Looks like GetConfiguration is a fault. Run & List reset the config automagically, but they don't get reset when I do get config directly.
Richard Simpson
@RichiCoder1
Jun 08 2017 20:25
Commited a temp hack fix for now
Manfred Wallner
@mwallner
Jun 08 2017 20:27
already seen - building ...
Richard Simpson
@RichiCoder1
Jun 08 2017 20:28
Looks like downloads was also a (weird edge case) side effect of that
Fixed and pushed
I think that's all the commands with side effects.
Rob Reynolds
@ferventcoder
Jun 08 2017 20:52
There is a message to that affect in the XML documentation for that function
;)
but yeah
Richard Simpson
@RichiCoder1
Jun 08 2017 20:53
Lord, you can tell how much I read. But I basically copy & pasted your original_configuration function and everything appears to be happy now
The chocolatiest bot this side of the Mississippi
@choco-bot
Jun 08 2017 20:53
Rob Reynolds
@ferventcoder
Jun 08 2017 20:54
pushing chocolatey.lib 0.10.7 now
Richard Simpson
@RichiCoder1
Jun 08 2017 20:56
Oh? What's new?
the high priority fix is in file/file64 not being aliases
broken by 0.10.6
Manfred Wallner
@mwallner
Jun 08 2017 20:58
chocolatey/choco#1328 -> fixed within 2 days ?!? great customer support!!
Richard Simpson
@RichiCoder1
Jun 08 2017 20:58
Hmm. At least 3 of those are relevant to Gui too. Will pull it in later today.
Rob Reynolds
@ferventcoder
Jun 08 2017 20:58
and the switch in the exe back to asInvoker
okay, can't push just yet
Richard Simpson
@RichiCoder1
Jun 08 2017 21:02
?
Rob Reynolds
@ferventcoder
Jun 08 2017 21:02
Response status code does not indicate success: 400 (Your account requires client version '4.1.0' or higher to be able to push packages. Please contact support@nuget.org to get more details.).
Seriously WTactualF?!
Richard Simpson
@RichiCoder1
Jun 08 2017 21:02
Do you break the feed?
Rob Reynolds
@ferventcoder
Jun 08 2017 21:02
that's on NuGet
guess of at least two days ago
Richard Simpson
@RichiCoder1
Jun 08 2017 21:03
Well, they did just flip the switch on some big changes, but I'm suprised they're required the new version??
Rob Reynolds
@ferventcoder
Jun 08 2017 21:03
I want to go set this back down, but I don't see where
Manfred Wallner
@mwallner
Jun 08 2017 21:14
hm .. https://github.com/chocolatey/ChocolateyGUI/commits/develop doesn't look that nice .. I should have squashed the commits before merging - right? / is there a process for this @RichiCoder1 ?
Rob Reynolds
@ferventcoder
Jun 08 2017 22:35
another WTF - NuGet/NuGetGallery#4070
whoa, you can edit package metadata again
wonder when they made that available