Linux Archive

Linux Archive (http://www.linux-archive.org/)
-   ArchLinux Pacman Development (http://www.linux-archive.org/archlinux-pacman-development/)
-   -   Fix thinko in configure.ac CFLAGS empty checking (http://www.linux-archive.org/archlinux-pacman-development/705388-fix-thinko-configure-ac-cflags-empty-checking.html)

Allan McRae 09-19-2012 05:56 AM

Fix thinko in configure.ac CFLAGS empty checking
 
On 19/09/12 12:34, Dan McGee wrote:
> 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;
>

Ack...

Turns out the combination of both the stuff ups in that commit fixed the
bug I was trying to fix! :P


All times are GMT. The time now is 01:05 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.