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.

Sign up for GitHub

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

Jump to bottom

Include software install path in choco info output #1901

Open
Foadsf opened this issue Aug 20, 2019 · 5 comments · Fixed by #3317
Open

Include software install path in choco info output #1901

Foadsf opened this issue Aug 20, 2019 · 5 comments · Fixed by #3317
Assignees
@TheCakeIsNaOH
Labels
2 - Working Feature Issues that introduce new functionality to the project, instead of updating existing functionality Requires Upstream Change Requires changes to a different location once issue is fixed or implemented
Milestone
3.x

Comments

@Foadsf
Copy link

Foadsf commented Aug 20, 2019 •
edited
Loading

Following this tweet, it would be great if the installation folder could be inquired somehow. For example, the command choco info packageName could also report the path.
@ferventcoder ferventcoder changed the title get instalation path with choco info comand info - get installation path with choco info comand Aug 20, 2019
@bcurran3
Copy link

bcurran3 commented Aug 20, 2019 •
edited
Loading

...or maybe a new command such as "diags" that could display choco version info, install location, Chocolatey cache location, nuget cache location, configured features (not a list of all of them), system state, validation checks, a quick check and report that the CRCs are OK on all Chocolatey files, a quick ping to the configured sources, etc.

@gep13 gep13 changed the title info - get installation path with choco info comand Include installation path in the output from the choco info command Aug 21, 2019
@Foadsf
Copy link
Author

Foadsf commented Aug 21, 2019

@bcurran3 anything giving the information required is appreciated. But I think the info command is more intuitive. If I'm not mistaken HomeBrew does the same.

@bcurran3
Copy link

bcurran3 commented Aug 21, 2019 •
edited
Loading

I think I misunderstood. I was thinking more upstream and about Chocolatey versus just a single package.

I guess my suggestion is a completely different one. I'll create a new issue. Done: #1903

@Foadsf Foadsf mentioned this issue Oct 3, 2019
Open
@fuhrmanator
Copy link

My use-case (problem) was I installed irfanview on a brand new Win 10 machine, but it's not in Program Files, I can't find it in the start menu (search results send me to the Web), etc. I know if you're clever you realize all things get installed in the secret choco place, but I can't remember what that is and expected it to come out of an info command.

@Foadsf Foadsf mentioned this issue Apr 7, 2020
Closed
This was referenced Sep 7, 2023
Closed
Open
TheCakeIsNaOH added a commit to TheCakeIsNaOH/choco that referenced this issue Sep 7, 2023
@TheCakeIsNaOH
(chocolatey#1901) Display software installation location
2a7f8e9 
Displays the saved software installation location for info and list
commands. If the package has no software install location, it displays
nothing, and if the package was installed/upgraded before the saving
feature was added, again, it does not show that line.
@TheCakeIsNaOH TheCakeIsNaOH mentioned this issue Sep 7, 2023
Merged
10 tasks
@gep13 gep13 added this to the 2.3.0 milestone Dec 14, 2023
@TheCakeIsNaOH TheCakeIsNaOH self-assigned this Jan 9, 2024
TheCakeIsNaOH added a commit to TheCakeIsNaOH/choco that referenced this issue Jan 10, 2024
@TheCakeIsNaOH
(chocolatey#1901) Display software installation location
bf30ac7 
Displays the saved software installation location for info and list
commands. If the package has no software install location, it displays
nothing, and if the package was installed/upgraded before the saving
feature was added, again, it does not show that line.
gep13 pushed a commit to TheCakeIsNaOH/choco that referenced this issue Apr 26, 2024
@TheCakeIsNaOH @gep13
(chocolatey#1901) Display package deploymen location
2087bcc 
Displays the saved package deployment location for info and list
commands. If the package has no deployemnt location, it displays
nothing, and if the package was installed/upgraded before the saving
feature was added, again, it does not show that line.
gep13 added a commit that referenced this issue Apr 26, 2024
@gep13
Merge pull request #3317 from TheCakeIsNaOH/gh1901-list-location
ef05daf 
(#1901) Display software installation location
@gep13 gep13 added 4 - Done Feature Issues that introduce new functionality to the project, instead of updating existing functionality labels Apr 26, 2024
@pauby pauby changed the title Include installation path in the output from the choco info command Include install path in choco info output May 28, 2024
@pauby pauby changed the title Include install path in choco info output Include software install path in choco info output May 28, 2024
@gep13 gep13 modified the milestones: 2.3.0, 2.4.0 May 31, 2024
@gep13 gep13 reopened this May 31, 2024
@gep13
Copy link
Member

gep13 commented May 31, 2024

Unfortunately, I am going to have to re-open this issue, as it will not be possible to bring this change in at present.

This change, while small and which builds on this issue, can't be implemented fully in Chocolatey Licensed Extension without changing the dependency on the Chocolatey package. This is not something we are planning in this release, so this change will be reverted and will be re-added in the next release.

Apologies on this. Please let me know if you have any questions.

@gep13 gep13 added the Requires Upstream Change Requires changes to a different location once issue is fixed or implemented label May 31, 2024
gep13 added a commit that referenced this issue May 31, 2024
@gep13
(#1901) Revert commit to display location
c74a19c 
Based on some testing that was completed, we can't bring in this change
currently, since we can't provide the same functionality in CLE without
bumping the version number of the CLI dependency in the nuspec file.
This is not something that we are ready to do, so taking out this
change, and the issue will be moved to the next milestone.
gep13 added a commit that referenced this issue Jun 5, 2024
@gep13
Merge branch 'release/2.3.0'
5dc8330 
* release/2.3.0: (153 commits)
  (#23) Move call to SetRemotePackageNamesIfAllSpecified
  (#1901) Revert commit to display location
  (maint) Update Authenticode Signature
  (tests) Some minor tweaks for test consistency
  (#310) Set-EnvironmentVariable: delete values properly
  (tests) Remove v2 import tests
  (#310) Fix test issues
  (#310) Fixup v2 compatibility
  (tests) Working on tests during pairing session
  (#2050) Add Pester test for --ignore-pinned option
  (#3381) Add Pester tests for rule command
  (#1144) Add Pester test to cover hash validation
  (#72) Add Pester tests to cover new functionality
  (test) Update assertions for push commands
  (#1310) Add Pester test to validate output
  (#23) Add tests to handle install all
  (#2200) Add test for new enhanced exit code
  (#1764) Add test for new enhanced exit code
  (#1760) Add test for new enhanced exit code
  (#1759) Add test for new enhanced exit code
  ...
gep13 added a commit that referenced this issue Jun 5, 2024
@gep13
Merge branch 'master' into develop
484999b 
* master: (37 commits)
  (#310) Fix path to chocolatey.lib files
  (maint) Corrected casing of file
  (#23) Move call to SetRemotePackageNamesIfAllSpecified
  (#1901) Revert commit to display location
  (maint) Update Authenticode Signature
  (tests) Some minor tweaks for test consistency
  (#310) Set-EnvironmentVariable: delete values properly
  (tests) Remove v2 import tests
  (#310) Fix test issues
  (#310) Fixup v2 compatibility
  (tests) Working on tests during pairing session
  (#2050) Add Pester test for --ignore-pinned option
  (#3381) Add Pester tests for rule command
  (#1144) Add Pester test to cover hash validation
  (#72) Add Pester tests to cover new functionality
  (test) Update assertions for push commands
  (#1310) Add Pester test to validate output
  (#23) Add tests to handle install all
  (#2200) Add test for new enhanced exit code
  (#1764) Add test for new enhanced exit code
  ...
gep13 added a commit to gep13/choco that referenced this issue Jun 21, 2024
@gep13
Revert "(chocolatey#1901) Revert commit to display location"
f6a59bb 
This reverts commit c74a19c.
josh-cooley added a commit to josh-cooley/choco that referenced this issue Jul 8, 2024
@josh-cooley
Merge branch 'develop' of https://github.com/josh-cooley/choco into d…
5e1e5cb 
…evelop

* 'develop' of https://github.com/josh-cooley/choco: (145 commits)
  (maint) Add CODEOWNERS files
  (build) Add templated notifications to all builds
  (build) Add verification of source scripts
  (doc) Further tweak docgen for Chocolatey.PowerShell
  (doc) Tweak docgen for Chocolatey.PowerShell
  (chocolatey#3446) Update GenerateDocs.ps1
  (chocolatey#3307) Do not create Tools directory from MSI
  (maint) Do not create Chocolatey directory in MSI
  (chocolatey#310) Fix path to chocolatey.lib files
  (maint) Corrected casing of file
  (chocolatey#23) Move call to SetRemotePackageNamesIfAllSpecified
  (chocolatey#1901) Revert commit to display location
  (maint) Update Authenticode Signature
  (tests) Some minor tweaks for test consistency
  (chocolatey#310) Set-EnvironmentVariable: delete values properly
  (tests) Remove v2 import tests
  (chocolatey#310) Fix test issues
  (chocolatey#310) Fixup v2 compatibility
  (tests) Working on tests during pairing session
  (chocolatey#2050) Add Pester test for --ignore-pinned option
  ...
gep13 added a commit to gep13/choco that referenced this issue Jul 14, 2024
@gep13
Revert "(chocolatey#1901) Revert commit to display location"
1d8638d 
This reverts commit c74a19c.
@pauby pauby modified the milestones: 2.4.0, 3.x Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TheCakeIsNaOH TheCakeIsNaOH

Labels
2 - Working Feature Issues that introduce new functionality to the project, instead of updating existing functionality Requires Upstream Change Requires changes to a different location once issue is fixed or implemented
Projects
None yet
Milestone
3.x
Development

Successfully merging a pull request may close this issue.

(#1901) Display software installation location TheCakeIsNaOH/choco
6 participants
@gep13 @fuhrmanator @pauby @Foadsf @bcurran3 @TheCakeIsNaOH