Skip to content
This repository was archived by the owner on Dec 18, 2017. It is now read-only.

Implement kpm install #201

Closed
suhasj opened this issue May 16, 2014 · 10 comments
Closed

Implement kpm install #201

suhasj opened this issue May 16, 2014 · 10 comments

Comments

@suhasj
Copy link
Contributor

suhasj commented May 16, 2014

The command 'kpm install' does not seem to install the specific package as intended. Tried this from the location of the app root which has the project.json but the package is not installed. If this does work, need more pointers for this

Commands tried

  • kpm install Microsoft.AspNet.Identity
  • kpm install Microsoft.AspNet.Identity -version 0.1-alpha-1030
  • kpm install Microsoft.AspNet.Identity -version 0.1-alpha-1030 -s <latest Coherence>
@davidfowl
Copy link
Member

kpm install exists?

@lodejard
Copy link
Contributor

Ah! It does not exist. kpm help install is an old code fragment that was used to test command line switch options, nothing more.

@suhasj
Copy link
Contributor Author

suhasj commented May 17, 2014

It is actually a good feature. We can install a single package instead of going through all the packages and the dependency tree for the project.

@davidfowl
Copy link
Member

@suhasj you still have to walk the dependency tree, that doesn't change. It's equivalent to adding it by hand and calling restore

@glennc glennc added this to the 1.0.0-alpha3 milestone Jun 16, 2014
@glennc glennc changed the title kpm install not seemed to be working Implement kpm install Jun 16, 2014
@glennc
Copy link
Member

glennc commented Jun 16, 2014

We will implement this at some point. Changed the title to reflect that.

@glennc
Copy link
Member

glennc commented Jul 15, 2014

To begin with we should implement kpm add <name> <version>

Both name and version would be mandatory, since I am fairly certain we don't allow empty versions in project.json anymore.

The command would simply add the given values to the project.json.

We could add an install command later that calls add and then restore. But add is the primitive that we need to begin with.

@Alxandr
Copy link
Contributor

Alxandr commented Aug 19, 2014

If you omit version, shouldn't it lookup the latest version, then add that? Like Install-Package does today.

@davidfowl
Copy link
Member

It does

@Alxandr
Copy link
Contributor

Alxandr commented Aug 19, 2014

Ah. Good. I was just commenting on the last comment that said "Both name and version would be mandatory".

@ChengTian
Copy link
Contributor

Hi @Alxandr , for kpm add, it's true that both name and version are mandatory. kpm add is a primitive used by kpm install, which figures out the latest version if version is ignored.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants