diff mbox

kms_flip: Add -lrt back

Message ID 1355842287-19442-1-git-send-email-ville.syrjala@linux.intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Ville Syrjälä Dec. 18, 2012, 2:51 p.m. UTC 
From: Ville Syrjälä <ville.syrjala@linux.intel.com>

kms_flip doesn't build due to missing -lrt.

The build was broken due to a typo in this commit:

commit acdf7171264a5139caabb78127c9eef5080c0682
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date:   Sat Dec 15 00:48:47 2012 +0100

    tests: s/flip_test/kms_flip

    OCD wins!

    Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

For some reason libudev.pc brings in -lrt as well, which probably
caused the typo to go unnoticed.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
 tests/Makefile.am |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Daniel Vetter Dec. 18, 2012, 3:04 p.m. UTC | #1 
On Tue, Dec 18, 2012 at 04:51:27PM +0200, ville.syrjala@linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> 
> kms_flip doesn't build due to missing -lrt.
> 
> The build was broken due to a typo in this commit:
> 
> commit acdf7171264a5139caabb78127c9eef5080c0682
> Author: Daniel Vetter <daniel.vetter@ffwll.ch>
> Date:   Sat Dec 15 00:48:47 2012 +0100
> 
>     tests: s/flip_test/kms_flip
> 
>     OCD wins!
> 
>     Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> 
> For some reason libudev.pc brings in -lrt as well, which probably
> caused the typo to go unnoticed.
> 
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

Blargh, I suck, thanks for fixing this up.
-Daniel
diff mbox

Patch

diff --git a/tests/Makefile.am b/tests/Makefile.am
index 9cafe4c..0476908 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -180,7 +180,7 @@  gem_fence_thrash_LDADD = $(LDADD) -lpthread
 gem_threaded_access_tiled_LDADD = $(LDADD) -lpthread
 
 gem_wait_render_timeout_LDADD = $(LDADD) -lrt
-kms_flip = $(LDADD) -lrt
+kms_flip_LDADD = $(LDADD) -lrt
 
 gem_ctx_basic_LDADD = $(LDADD) -lpthread