On Sun, 24 Nov 2013, Diego Biurrun wrote:

On Sat, Nov 23, 2013 at 07:23:43PM +0100, Vittorio Giovara wrote:
--- a/configure
+++ b/configure
@@ -804,6 +808,19 @@ check_ldflags(){

+test_stripflags(){
+    log test_stripflags "$@"
+    cat > $TMPC <<EOF
+int main(void){ return 0; }
+EOF
+    check_cmd $strip $STRIPFLAGS "$@" $TMPO
+}

This does not do anything with TMPC.  What is your intention?

I think his intention was to make sure that $TMPO actually exists and is a valid object file - otherwise the strip tests will succeed or fail depending on whether the latest compile check prior to the strip tests actually succeeded. The patch obviously doesn't do quite that, though.

// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to