FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > ArchLinux > ArchLinux Pacman Development

 
 
LinkBack Thread Tools
 
Old 09-19-2012, 02:34 AM
Dan McGee
 
Default Fix thinko in configure.ac CFLAGS empty checking

Since commit d2669b47, CFLAGS specified on the command line haven't been
respected at all, resulting in no optimization being applied to builds.
This exposed one warning flag issue in some new code, which is also
fixed here.

Signed-off-by: Dan McGee <dan@archlinux.org>
---
configure.ac | 2 +-
src/pacman/query.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/configure.ac b/configure.ac
index 7fe696a..2d7580e 100644
--- a/configure.ac
+++ b/configure.ac
@@ -67,7 +67,7 @@ LIB_VERSION=`expr lib_current - lib_age`.lib_age.lib_revision
LIB_VERSION_INFO="lib_current:lib_revision:lib_age "

# Respect empty CFLAGS during compiler tests
-if test "x$CFLAGS" != "x"; then
+if test "x$CFLAGS" = "x"; then
CFLAGS=""
fi

diff --git a/src/pacman/query.c b/src/pacman/query.c
index fc2c90c..398adac 100644
--- a/src/pacman/query.c
+++ b/src/pacman/query.c
@@ -134,7 +134,7 @@ static int query_fileowner(alpm_list_t *targets)
struct stat buf;
alpm_list_t *i;
size_t len;
- int found = 0;
+ unsigned int found = 0;

if((filename = strdup(t->data)) == NULL) {
goto targcleanup;
--
1.7.12
 

Thread Tools




All times are GMT. The time now is 06:36 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org