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

bower list doesn't list full dependency tree #622

Closed
passy opened this issue Jul 15, 2013 · 1 comment · Fixed by #709
Closed

bower list doesn't list full dependency tree #622

passy opened this issue Jul 15, 2013 · 1 comment · Fixed by #709

Comments

@passy
Copy link
Contributor

passy commented Jul 15, 2013

If you run bower list in a directory with a component that has their own dependencies, the output depends on whether you have the main dependency saved in your bower.json or not. I talked to @satazor about this and we agreed that this behavior is a little counter-intuitive.

Example

If you're in an empty directory and run bower install imagesloaded && bower list, you get the complete dependency tree:

├── eventEmitter#4.2.0 extraneous
├── eventie#1.0.3 extraneous
└─┬ imagesloaded#3.0.1 extraneous
  ├── eventEmitter#4.2.0
  └── eventie#1.0.3

If you, however, save the dependency with bower install --save imagesloaded && bower list, you only see the top-level dependency:

└── imagesloaded#3.0.1

If you for instance rely on the this to include the dependencies in your HTML, you need the full output. It should be mentioned, that bower list -p does list all dependencies in both cases.

@satazor
Copy link
Member

satazor commented Jul 16, 2013

@passy With bower.1.0.0-alpha.7:

bower install --save imagesloaded && bower list:

$ bower list
bower check-new     Checking for new versions of the project dependencies..
foo /Users/satazor/Work
└─┬ imagesloaded#3.0.1
  ├── eventEmitter#4.2.0
  └── eventie#1.0.3

Though, when a bower.json file is not presented there's no feasible way to make them nested unless we add an internal prop to the package meta(.bower.json) file saying that imagesloaded was installed directly and not as a dependency of another.
Will look at this later tonight.

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

Successfully merging a pull request may close this issue.

2 participants