Bug 896415

Summary: accept small letters for answers on F17
Product: [Fedora] Fedora Reporter: Gabor Garami <hron>
Component: rpmconfAssignee: Miroslav Suchý <msuchy>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 17CC: msuchy, xjakub
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-07-02 15:45:27 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Gabor Garami 2013-01-17 09:23:20 UTC 
Description of problem:

It would be nice if rpmconf could accept small letter answers for questions, e.g. y n i d, not just Y N I D. 


How reproducible:

For example by running fedora-upgrade, rpmconf is fired up to solve .rpmnew files.

Steps to Reproduce:
1.
2.
3.
  
Actual results:

Currently if you answer small letters, rpmconf repeats the question

Expected results:

Accept small letters as answer

Additional info:
Comment 1 Miroslav Suchý 2013-01-17 09:34:13 UTC 
rpmconf accepts both lowercase and uppercase letters. And it works for me.

In fact all input is converted to uppercase by this code

echo $1 | tr "[:lower:]" "[:upper:]"

Can you please try run on your system this command:

echo y | tr "[:lower:]" "[:upper:]"

It should return "Y", if not we need to find why.
Comment 2 Gabor Garami 2013-01-17 10:07:49 UTC 
It converts to Y.

However, when I upgraded yesterday night from F17 to F18, rpmconf asked the question again instead of accepting i/y. Maybe F18 version is fixed?
Comment 3 Miroslav Suchý 2013-01-17 10:36:49 UTC 
They both have the same version.

How it behave now on F18 for you? Can you enter small letters?
Comment 4 Gabor Garami 2013-01-17 16:30:30 UTC 
Yes, it accepts.

I currently do not have F17 install so I can't try it out in that...
Comment 5 Miroslav Suchý 2013-01-17 16:41:17 UTC 
I will try it on F17 in few days.
Comment 6 Milos Jakubicek 2013-01-17 17:07:57 UTC 
I just tried on F17, the above command returns an "Y" properly. The only reason why it might go wrong I can imagine would be some strange locale settings. Could you try running:

LANG=C rpmconf

whether it then works for you?
Comment 7 Miroslav Suchý 2013-07-02 15:45:27 UTC 
It works for me. If you can provide some reproducer, please reopen.