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

Choco install with packages.config should print out the packages to install #878

Closed
andmos opened this Issue Jul 27, 2016 · 2 comments

Comments

Projects
None yet
3 participants
@andmos
Contributor

andmos commented Jul 27, 2016

What You Are Seeing?

When running

choco install packages.config

The output is

Installing the following packages:
C:\Users\anm\Desktop\Packages.config

What is Expected?

I expect the output to list the packages in the packages.config file:

Installing the following packages:
package-a
package-b
package-c

How Did You Get This To Happen? (Steps to Reproduce)

Create a packages.config file with some id's and versions and run
choco install packages.config

Output Log

$ choco install Packages.config -s $(pwd) --verbose --debug
Chocolatey is running on Windows v 10.0.14390.0
Attempting to delete file "C:/ProgramData/chocolatey/choco.exe.old".
Attempting to delete file "C:\ProgramData\chocolatey\choco.exe.old".
Command line: "C:\ProgramData\chocolatey\choco.exe" install Packages.config -s C:\Users\anm\Desktop --verbose --debug
Received arguments: install Packages.config -s C:\Users\anm\Desktop--verbose --debug
RemovePendingPackagesTask is now ready and waiting for PreRunMessage.
Sending message 'PreRunMessage' out if there are subscribers...
[Pending] Removing all pending packages that should not be considered installed...
The source 'C:\Users\anm\Desktop' evaluated to a 'normal' source type

NOTE: Hiding sensitive configuration data! Please double and triple 
 check to be sure no sensitive data is shown, especially if copying 
 output to a gist for review.
Configuration: CommandName='install'|
CacheLocation='C:\Users\anm\AppData\Local\Temp\chocolatey'|
ContainsLegacyPackageInstalls='True'|
CommandExecutionTimeoutSeconds='2700'|WebRequestTimeoutSeconds='30'|
Sources='C:\Users\anm\Desktop'|SourceType='normal'|
Debug='True'|Verbose='True'|Force='False'|Noop='False'|
HelpRequested='False'|RegularOutput='True'|QuietOutput='False'|
PromptForConfirmation='True'|AcceptLicense='False'|
AllowUnofficialBuild='False'|Input='Packages.config'|
AllVersions='False'|SkipPackageInstallProvider='False'|
PackageNames='Packages.config'|Prerelease='False'|ForceX86='False'|
OverrideArguments='False'|NotSilent='False'|IgnoreDependencies='False'|
AllowMultipleVersions='False'|AllowDowngrade='False'|
ForceDependencies='False'|Information.PlatformType='Windows'|
Information.PlatformVersion='10.0.14390.0'|
Information.PlatformName='Windows'|
Information.ChocolateyVersion='0.9.10.3'|
Information.ChocolateyProductVersion='0.9.10.3'|
Information.FullName='choco, Version=0.9.10.3, Culture=neutral, PublicKeyToken=79d02ea9cad655eb'|

Information.Is64Bit='True'|Information.IsInteractive='True'|
Information.IsUserAdministrator='True'|
Information.IsProcessElevated='True'|
Information.IsLicensedVersion='False'|Features.AutoUninstaller='True'|
Features.CheckSumFiles='True'|Features.FailOnAutoUninstaller='False'|
Features.FailOnStandardError='False'|Features.UsePowerShellHost='True'|
Features.LogEnvironmentValues='False'|Features.VirusCheck='False'|
Features.FailOnInvalidOrMissingLicense='False'|
Features.IgnoreInvalidOptionsSwitches='True'|
Features.UsePackageExitCodes='True'|
Features.UseFipsCompliantChecksums='False'|
ListCommand.LocalOnly='False'|
ListCommand.IncludeRegistryPrograms='False'|ListCommand.PageSize='25'|
ListCommand.Exact='False'|ListCommand.ByIdOnly='False'|
ListCommand.IdStartsWith='False'|ListCommand.OrderByPopularity='False'|
ListCommand.ApprovedOnly='False'|
ListCommand.DownloadCacheAvailable='False'|
ListCommand.NotBroken='False'|UpgradeCommand.FailOnUnfound='False'|
UpgradeCommand.FailOnNotInstalled='False'|
UpgradeCommand.NotifyOnlyAvailableUpgrades='False'|
NewCommand.AutomaticPackage='False'|
NewCommand.UseOriginalTemplate='False'|SourceCommand.Command='unknown'|
SourceCommand.Priority='0'|FeatureCommand.Command='unknown'|
ConfigCommand.Command='unknown'|PinCommand.Command='unknown'|
_ Chocolatey:ChocolateyInstallCommand - Normal Run Mode _
Installing the following packages:
Packages.config
By installing you accept licenses for the packages.
Package-A already installed.
 Use --force to reinstall, specify a version to install, or try upgrade.

Chocolatey installed 0/1 packages. 0 packages failed.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

Warnings:
 - Package-A - Package-A v0.0.1-RC0019 already installed.
 Use --force to reinstall, specify a version to install, or try upgrade.
Sending message 'PostRunMessage' out if there are subscribers...
Exiting with 0

@ferventcoder ferventcoder changed the title from Choco install with packages.config does not print out the packages to install to Choco install with packages.config should print out the packages to install Jul 27, 2016

@ferventcoder ferventcoder added this to the 0.9.10.x milestone Jul 27, 2016

@ferventcoder

This comment has been minimized.

Show comment
Hide comment
@ferventcoder

ferventcoder Jul 27, 2016

Member

This definitely looks like something that could be enhanced to do. I am on the fence about whether this is a bug or an enhancement, I could go either way.

Member

ferventcoder commented Jul 27, 2016

This definitely looks like something that could be enhanced to do. I am on the fence about whether this is a bug or an enhancement, I could go either way.

@ferventcoder ferventcoder added the Bug label Jul 27, 2016

@bpjohannessen

This comment has been minimized.

Show comment
Hide comment
@bpjohannessen

bpjohannessen Jul 27, 2016

I think I agree with you. Bug it is. I would love to see this fixed, as I had to write my own parser.

bpjohannessen commented Jul 27, 2016

I think I agree with you. Bug it is. I would love to see this fixed, as I had to write my own parser.

andmos pushed a commit to andmos/choco that referenced this issue Jul 28, 2016

andmos pushed a commit to andmos/choco that referenced this issue Jul 28, 2016

andmos added a commit to andmos/choco that referenced this issue Jul 28, 2016

(GH-878) When running choco install with a .config file this prints out
all packages set for installation

Previously when running choco install with a .config file as parameter the
output would say ''Installing the following packages:'' followed by the
path to the .config file. This change prints out that a .config file is
being used, as well as listing the packages set for installation.

andmos added a commit to andmos/choco that referenced this issue Jul 28, 2016

(GH-878) Installation with .config file prints packages in file
Previously when running choco install with a .config file as parameter the
output would say ''Installing the following packages:'' followed by the
path to the .config file. This change prints out that a .config file is
being used, as well as listing the packages set for installation.

andmos added a commit to andmos/choco that referenced this issue Jul 28, 2016

(GH-878) Installation with .config file prints packages in file
Previously when running choco install with a .config file as parameter the
output would say ''Installing the following packages:'' followed by the
path to the .config file. This change prints out that a .config file is
being used, as well as listing the packages set for installation.

andmos added a commit to andmos/choco that referenced this issue Jul 28, 2016

(GH-878) Installation with .config file prints packages in file
Previously when running choco install with a .config file as parameter the
output would say ''Installing the following packages:'' followed by the
path to the .config file. This change prints out that a .config file is
being used, as well as listing the packages set for installation.

andmos added a commit to andmos/choco that referenced this issue Jul 28, 2016

(GH-878) Installation with .config file prints packages in file
Previously when running choco install with a .config file as parameter the
output would say ''Installing the following packages:'' followed by the
path to the .config file. This change prints out that a .config file is
being used, as well as listing the packages set for installation.

@ferventcoder ferventcoder modified the milestones: 0.9.10.4, 0.9.10.x Aug 2, 2016

@ferventcoder ferventcoder added 1 - Ready and removed 0 - Backlog labels Aug 2, 2016

ferventcoder added a commit that referenced this issue Aug 3, 2016

(GH-878) Installation with .config file prints packages in file
Previously when running choco install with a .config file as parameter the
output would say ''Installing the following packages:'' followed by the
path to the .config file. This change prints out that a .config file is
being used, as well as listing the packages set for installation.

ferventcoder added a commit that referenced this issue Aug 3, 2016

Merge branch 'pr880' into stable
* pr880:
  (GH-878) Installation with .config file prints packages in file

ferventcoder added a commit that referenced this issue Aug 3, 2016

Merge branch 'stable'
* stable:
  (GH-878) Installation with .config file prints packages in file

@ferventcoder ferventcoder added 3 - Done and removed 1 - Ready labels Aug 3, 2016

@ferventcoder ferventcoder self-assigned this Aug 3, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment