Просмотр исходного кода

Ticket #48224 - redux 2 - logconv.pl should handle *.tar.xz, *.txz, *.xz log files

https://fedorahosted.org/389/ticket/48224
Reviewed by: nhosoi (Thanks!)
Branch: master
Fix Description: Use $? instead of $! to get pipe errors.
Platforms tested: Fedora 21, RHEL 7.2 candidate
Flag Day: no
Doc impact: no
Rich Megginson 10 лет назад
Родитель
Сommit
29043c5716
1 измененных файлов с 3 добавлено и 3 удалено
  1. 3 3
      ldap/admin/src/logconv.pl

+ 3 - 3
ldap/admin/src/logconv.pl

@@ -437,9 +437,9 @@ sub doUncompress {
 		# so use the xz command directly
 		# NOTE: This doesn't work if the argument is a file handle e.g. from
 		# Archive::Tar
-		$! = 0; # clear
-		if (!open($TARFH, "xz -dc $_ |") or $!) {
-			openFailed($!, $_);
+		$? = 0; # clear
+		if (!open($TARFH, "xz -dc $_ |") or $?) {
+			openFailed($?, $_);
 			return;
 		}
 	} else {