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

Add check for background service #382

Closed
wants to merge 11 commits into from

Conversation

RichiCoder1
Copy link
Member

@RichiCoder1 RichiCoder1 commented Jan 31, 2017

Also fix a few typos and back resources.

Closes #371

Also fix a few typos and back resources.
@RichiCoder1
Copy link
Member Author

/cc @ferventcoder

@RichiCoder1
Copy link
Member Author

Had a change to poke at this @ferventcoder?

@ferventcoder
Copy link
Member

I did, but there is an issue I am going to dive deeper into this week to see whether it stems from this or from the API.

@RichiCoder1 RichiCoder1 added this to the 0.15.0 milestone Feb 17, 2017
@RichiCoder1 RichiCoder1 added the Feature Issues that introduce new functionality to the project, instead of updating existing functionality label Feb 17, 2017
* develop: (43 commits)
  Del "a" typo
  Correct name Christiaan
  Add missing Resources.sv.Designer.cs
  (GH-396) Localizate to Swedish
  Remove unused resource strings
  Make it possible to localizate On and Off in Settings view
  Set indent size for XAML.
  Fix build errors.
  (GH-372) Dynamically select open port for subprocess
  Reverted changes for GoodbyeDetails
  Fixed CodeFactor issues
  (GH-405) Changed Pin Button to correct resource string
  (GH-405) Extracted C# user displayed text
  (GH-405) Added missing string bindings
  (GH-405) Added missing strings
  Fix typo
  (GH-393) Corrected hard-coded path
  (GH-394) Corrected path to assets folder
  (maint) Corrected link to documentation site
  (doc) Minor copy editing
  ...

# Conflicts:
#	Source/ChocolateyGui/Bootstrapper.cs
#	Source/ChocolateyGui/Properties/Strings1.Designer.cs
#	Source/ChocolateyGui/Resources/Controls.xaml
#	Source/ChocolateyGui/Services/ChocolateyRemotePackageService.cs
#	Source/ChocolateyGui/ViewModels/SourcesViewModel.cs
#	Source/ChocolateyGui/Views/PackageView.xaml
- Added necessary UI elements
- Updated model classes with new properties
- Need to wait on updated Chocolatey Lib in order to fully test
@RichiCoder1
Copy link
Member Author

bump

@ferventcoder
Copy link
Member

Gary and I looked deeper into this - what is happening is the licensed.dll seems to run into an error that is super hard to determine - may need to find a way to attach a debugger. What I think it is - because the licensed edition takes a dependency on choco.exe, it's having issues when it is the DLL instead.

@RichiCoder1
Copy link
Member Author

RichiCoder1 commented Mar 12, 2017

Odd. Let me know how I can assist. I believe the chocolatey.lib includes symbols right? Even if not, you should be able to switch Gui's reference to a locally build version, turn off exact source match in debug settings, and then just attached a debugger with chocolatey to Gui. The ilmerge might futz with things though, and I'm not sure if and how to make Gui work directly with the source dlls....

Edit: Looks like ILMerge does the right thing, so you should still be able to debug against the merged dll.

@RichiCoder1
Copy link
Member Author

bump

@ferventcoder
Copy link
Member

There have been some fixes into chocolatey.lib and the extension that may fix this. Getting ready to try this again.

@RichiCoder1
Copy link
Member Author

Excellent! I'd merge mainline develop into this as it just updated all the the packages (including lib)

@ferventcoder
Copy link
Member

Can I push to this PR?

@RichiCoder1
Copy link
Member Author

Absolutely

@ferventcoder
Copy link
Member

ferventcoder commented May 18, 2017

Should I wait until we get #438 through? I prefer to reduce merge conflicts and so much has changed.

@RichiCoder1
Copy link
Member Author

#438 is merged and cleaned up. You should be good to go now.

@RichiCoder1
Copy link
Member Author

Brought this branch back up to Develop. Builds and runs just fine, though I'm not sure how much of the BP functionality is still preserved.

@ferventcoder
Copy link
Member

BP?

@RichiCoder1
Copy link
Member Author

Background service stuff. BS doesn't have the same ring haha.

@ferventcoder
Copy link
Member

Ah true!

@ferventcoder
Copy link
Member

Superseded by #451

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Issues that introduce new functionality to the project, instead of updating existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add check for useBackgroundService
3 participants