147480 - Lack of newlines when asking for password in checksetup.pl.

Status: RESOLVED FIXED

(Bugzilla :: Installation & Upgrading, defect)

Description

[Matthew Tuck [:CodeMachine]]

Enter a password for the administrator account: The password must be 3-16
characters in length.Enter a password for the administrator account:

Comment 1

[Vlad Dascalu]

Attachment: Trivial one-liner patch to make checksetup.pl function correctly.

I've tested this on the current CVS: it fixes the problem and works like
expected. You can also see from the diff that it reproduces the pattern of
"\n"s which is found out just 3 lines above.

Comment 2

[Vlad Dascalu]

Comment on attachment 127887 [details] [diff] [review]
Trivial one-liner patch to make checksetup.pl function correctly.

I'd like a review now on this because it's a one-line thing and it's kinda ugly
to see that non-sense on the screen when installing Bugzilla.

Requesting it from Paul but it's welcomed from anybody.

Comment 3

[Christian Reis]

Comment on attachment 127887 [details] [diff] [review]
Trivial one-liner patch to make checksetup.pl function correctly.

Looks great. Thanks for the patch.

Comment 4

[Vlad Dascalu]

Thanks, requesting approval.

Comment 5

[Vlad Dascalu]

<-- me

(if someone with CVS access has some free time feel free to commit this)

Comment 6

[Vlad Dascalu]

<-- me

Comment 7

[J. Paul Reed [:preed]]

Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision: 1.237; previous revision: 1.236
done

Comment 8

[Vlad Dascalu]

Thanks, jp. Marking as fixed.