fix quiet mode of debian installer

[mathias-luedtke]

No description provided.

[codecov-io]

Codecov Report

Merging #612 into master will increase coverage by 0.01%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master     #612      +/-   ##
==========================================
+ Coverage   75.27%   75.29%   +0.01%     
==========================================
  Files          30       30              
  Lines        2884     2882       -2     
==========================================
- Hits         2171     2170       -1     
+ Misses        713      712       -1

Impacted Files	Coverage Δ
src/rosdep2/platforms/debian.py	85.71% <75%> (+0.52%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cccca5d...d9ddcc4. Read the comment docs.

[tfoote]

Good catch, I updated it for a slightly more compact implementation.

[mathias-luedtke]

The same structure is used in redhat.py, but without the typo.
The test failure is unrelated.

[tfoote]

lgtm. Thanks I retriggered the failed test that failed in the setup phase.

We could clean up the redhat logic too but that could easily be a followup too.

[wjwwood]

@tfoote should I merge this then (you can feel free to merge it as well)? I don't have time to do a release right now. Though a rosdep release is something I should do soon, maybe just after the ROS 2 release.

[tfoote]

@wjwwood I don't think it's too urgent for getting a release out. The failure mode for this is that it won't pass the quiet flags through if you use -y. All the other changes are mostly docs, and other small cleanups.

[mathias-luedtke]

The failure mode for this is that it won't pass the quiet flags through if you use -y

The quiet flag was not passed at all.
This bug was present for 4 years, so a release is not urgent.

[708yamaguchi]

@tfoote @wjwwood
Could you please release 0.12.3 version of rosdep?
I have a problem that our travis fails because the travis log exceeds its maximum log length. (for example jsk-ros-pkg/jsk_robot#963 and jsk-ros-pkg/jsk_robot#969)
I think this issue will be solved if this commit is merged.

[mathias-luedtke]

The last release is more than 7 months old..

[nuclearsandwich]

This pull request has been released in rosdep 0.13.0. Thanks for contributing!