From: Chris Stylianou Date: Fri, 21 Jun 2013 14:02:39 +0000 (+0100) Subject: Fix incorrect uname usage. X-Git-Tag: accepted/tizen/20131212.181521~83^2~13 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=dd64078481573ff1f984bf261ba062d5ff532d40;p=platform%2Fupstream%2Fqttools.git Fix incorrect uname usage. The POSIX function uname() can return any non-negative value for success. - Example: Solaris 10 returns 1 on success. See: http://pubs.opengroup.org/onlinepubs/009695399/functions/uname.html Change-Id: Ic3d51f94a31d8b562b28a2d09df3d31b04c77dc8 Reviewed-by: Oswald Buddenhagen Reviewed-by: Daniel Teske (cherry picked from qtbase/e97c37a37ac27f9ab901e974f0a882e81ffd571d) --- diff --git a/src/linguist/shared/qmakeevaluator.cpp b/src/linguist/shared/qmakeevaluator.cpp index 6692917..4024e12 100644 --- a/src/linguist/shared/qmakeevaluator.cpp +++ b/src/linguist/shared/qmakeevaluator.cpp @@ -1043,7 +1043,7 @@ void QMakeEvaluator::loadDefaults() # endif #elif defined(Q_OS_UNIX) struct utsname name; - if (!uname(&name)) { + if (uname(&name) != -1) { vars[ProKey("QMAKE_HOST.os")] << ProString(name.sysname); vars[ProKey("QMAKE_HOST.name")] << ProString(QString::fromLocal8Bit(name.nodename)); vars[ProKey("QMAKE_HOST.version")] << ProString(name.release);