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

Regression: -y suppresses prompts from package manager #16

Closed
Lasall opened this issue Jun 11, 2012 · 4 comments · Fixed by #19
Closed

Regression: -y suppresses prompts from package manager #16

Lasall opened this issue Jun 11, 2012 · 4 comments · Fixed by #19

Comments

@Lasall
Copy link
Collaborator

Lasall commented Jun 11, 2012

The new behavior is ${_APTMGR} -y to suppress prompts from package manager. This is a regression because you don't see anymore which packages have been kept back, which will be upgraded and which will be removed. With a complex setup a non-interactive behavior can be dangerous.

I know why this switch was introduced but imho two prompts are better then a single less meaningful one. If you save decision of first promt (see #15) -y can easily be removed.

@ilikenwf
Copy link
Owner

I recklessly merged a few requests a while back and need to clean things up anyway - duly noted.

@lilianmoraru
Copy link

@ilikenwf
Copy link
Owner

Nice. I'll try and get some bugs worked out of it by this weekend.

On Wed, Jun 13, 2012 at 9:47 AM, Lilian Anatolie Moraru
reply@reply.github.com
wrote:

Read comments here: http://iloveubuntu.net/apt-fast-16-released-official-ppa-available


Reply to this email directly or view it on GitHub:
#16 (comment)

@ilikenwf
Copy link
Owner

Oh, and your commentators are having issues.

Lasall added a commit to Lasall/apt-fast that referenced this issue Jun 13, 2012
 * Add update option. Closes: ilikenwf#17
 * Remove -y as default ${_APTMGR} command. Closes: ilikenwf#16
 * Quote variables.
 * Default behavior of "If you want to download and install the files on your
   System press Y else n to abort. [Y/n]" changed to "Yes, please download".
 * Minor changes (hardened find command, etc.).
 * Fix indentation and code style (e.g. ; removed).
@Lasall Lasall closed this as completed Jun 14, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants