Message ID | 1462578003-25315-1-git-send-email-filbranden@google.com |
---|---|
State | Accepted, archived |
Headers | show |
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@google.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@google.com> > Signed-off-by: Filipe Brandenburger <filbranden@google.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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, May 26, 2016 at 10:47:54AM -0700, Filipe Brandenburger wrote: > 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... Applied, thanks. - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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
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@google.com> Signed-off-by: Filipe Brandenburger <filbranden@google.com> --- lib/e2p/Makefile.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)