Skip to content
This repository has been archived by the owner on Aug 9, 2019. It is now read-only.

When given a [Y/n] prompt, and input other than 'n' means 'Y'. #285

Open
wallywyyoung opened this issue Aug 9, 2016 · 6 comments
Open

Comments

@wallywyyoung
Copy link

Using -Syu etc. any input that isn't 'n' continues the build/install process- I've seen it on '7' and 'h' so far.

@larchunix
Copy link
Collaborator

IIRC it should have been fixed by #230. Could you try using git version of yaourt ?

@f2404
Copy link
Collaborator

f2404 commented Aug 9, 2016

I'm sorry but isn't it supposed to be working this way? I thought capital 'Y' meant default, i.e. anything other than 'n' would be 'yes'.

UP: However, this also works inconsistently: when I pressed space to the first prompt, it was handled as 'yes'; but the second one handled space as 'no' - build stopped.

$ LANG=C yaourt -S yaourt-git 

==> Downloading yaourt-git PKGBUILD from AUR...
x .SRCINFO
x PKGBUILD

yaourt-git 1.8.1.r11.g153c1f3-1  (2016-03-20 16:37)
( Unsupported package: Potentially dangerous ! )
==> Edit PKGBUILD ? [Y/n] ("A" to abort)
==> ------------------------------------
==>  

==> yaourt-git dependencies:
 - package-query-git (already installed)
 - pacman>=5.0 (already installed)
 - git (already installed) [makedepend]


==> Continue building yaourt-git ? [Y/n]
==> ------------------------------------
==>  

@larchunix
Copy link
Collaborator

I'm sorry but isn't it supposed to be working this way? I thought capital 'Y' meant default, i.e. anything other than 'n' would be 'yes'.

It was the original behaviour but we agreed to align to pacman behaviour in #218.

UP: However, this also works inconsistently: when I pressed space to the first prompt, it was handled as 'yes'; but the second one handled space as 'no' - build stopped.

Good catch, I probably missed this one in #230.

@f2404
Copy link
Collaborator

f2404 commented Aug 10, 2016

It was the original behaviour but we agreed to align to pacman behaviour in #218.

Yes, for Y/n options the behavior seems to be aligned. There seems a problem with e.g. Y/n/A options.

@larchunix
Copy link
Collaborator

On Wed, Aug 10, 2016 at 01:40:45AM -0700, Igor wrote:

 It was the original behaviour but we agreed to align to pacman behaviour
 in #218.

pacman behavior is different: it ignores (just prints) all symbols but 'Y'
and 'n':

$ sudo pacman -S pacman
warning: pacman-5.0.1-4 is up to date -- reinstalling
resolving dependencies...
looking for conflicting packages...

Packages (1) pacman-5.0.1-4

Total Installed Size: 4.42 MiB
Net Upgrade Size: 0.00 MiB

:: Proceed with installation? [Y/n] lkkh kjg

IIRC, yaourt does the same by setting NOENTER=0 in yaourtrc.

@M1cha
Copy link

M1cha commented Jan 22, 2017

I have the same problem. how it should work imo:
enter: use default
Y/y: yes
N/n: no

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

No branches or pull requests

4 participants