[Fedora-directory-commits] adminserver/admserv/newinst/src ASDialogs.pm.in, 1.4, 1.5

[Richard Allen Megginson (rmeggins)]

Author: rmeggins

Update of /cvs/dirsec/adminserver/admserv/newinst/src
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv15803/adminserver/admserv/newinst/src

Modified Files:
	ASDialogs.pm.in 
Log Message:
Resolves: bug 248145
Bug Description: Replace ds_newinst binary with perl script
Reviewed by: nhosoi (Thanks!)
Fix Description: 1) Inf needs to be able to read the .inf file from stdin.  This is what ds_newinst.pl does currently.
2) getlogin seems not to work in all cases, so add a more robust replacement.
Platforms tested: RHEL4, FC6
Flag Day: no
Doc impact: no



Index: ASDialogs.pm.in
===================================================================
RCS file: /cvs/dirsec/adminserver/admserv/newinst/src/ASDialogs.pm.in,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -r1.4 -r1.5
--- ASDialogs.pm.in	20 Jun 2007 16:35:02 -0000	1.4
+++ ASDialogs.pm.in	13 Jul 2007 19:53:17 -0000	1.5
@@ -138,7 +138,7 @@
         }
     }
     if ($> != 0) { # if not root, the user must be our uid
-        my $username = getlogin;
+        my $username = getLogin;
         if ($strans ne $username) {
             $self->{manager}->alert("dialog_assysuser_must_be_same", $username);
             return $DialogManager::SAME;
@@ -170,7 +170,7 @@
             if ($> == 0) { # if root, use the default user
                 $user = "@httpduser@";
             } else { # if not root, use the user's uid
-                $user = getlogin;
+                $user = getLogin;
             }
         }
         return $user;