linux-ext4 - Re: [PATCH e2fsprogs] Apply LDFLAGS when building tests in lib/e2p

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CADU+-uBOXVHUUX4f3xjJmzdDpLwUGNao2R9pKSunMsUKoaqfxw@mail.gmail.com>
Date:	Thu, 26 May 2016 10:47:54 -0700
From:	Filipe Brandenburger <filbranden@...gle.com>
To:	"Theodore Ts'o" <tytso@....edu>
Cc:	linux-ext4@...r.kernel.org, Eric Christopher <echristo@...gle.com>,
	Michael Forney <forney@...gle.com>,
	Filipe Brandenburger <filbranden@...gle.com>
Subject: Re: [PATCH e2fsprogs] Apply LDFLAGS when building tests in lib/e2p

Hi Ted,

Can you please include this patch in upstream e2fsprogs git repo?

This is a complement for 53904ae543591a which added LDFLAGS to the
test cases but seemed to miss these two...

Thank you!
Filipe


On Fri, May 6, 2016 at 4:40 PM, Filipe Brandenburger
<filbranden@...gle.com> wrote:
> This was previously addressed by Michael Forney on commit 53904ae543591a
> but it seems the test cases from lib/e2p (tst_ostype and tst_feature)
> were missed.
>
> Fix them by adding the same $(ALL_LDFLAGS) to those two targets.
>
> Tested by building it on a system that depends on LDFLAGS to produce
> working binaries and confirming that `make check` succeeds.
>
> Reported-by: Eric Christopher <echristo@...gle.com>
> Signed-off-by: Filipe Brandenburger <filbranden@...gle.com>
> ---
>  lib/e2p/Makefile.in | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/lib/e2p/Makefile.in b/lib/e2p/Makefile.in
> index 83f04ff5c04e..d109bf190051 100644
> --- a/lib/e2p/Makefile.in
> +++ b/lib/e2p/Makefile.in
> @@ -68,12 +68,12 @@ e2p.pc: $(srcdir)/e2p.pc.in $(top_builddir)/config.status
>  tst_ostype: $(srcdir)/ostype.c
>         $(E) "  LD $@"
>         $(Q) $(CC) -DTEST_PROGRAM -I$(top_srcdir)/lib -o tst_ostype \
> -               $(srcdir)/ostype.c $(ALL_CFLAGS)
> +               $(srcdir)/ostype.c $(ALL_CFLAGS) $(ALL_LDFLAGS)
>
>  tst_feature: $(srcdir)/feature.c
>         $(E) "  LD $@"
>         $(Q) $(CC) -DTEST_PROGRAM -I$(top_srcdir)/lib -o tst_feature \
> -               $(srcdir)/feature.c $(ALL_CFLAGS)
> +               $(srcdir)/feature.c $(ALL_CFLAGS) $(ALL_LDFLAGS)
>
>  check::        tst_ostype tst_feature
>         ./tst_ostype
> --
> 2.8.0.rc3.226.g39d4020
>
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html