On 04/24/16 05:53 PM, Michael Niedermayer wrote:
On Sun, Apr 24, 2016 at 11:55:06AM -0700, Dave Yeo wrote:
On 04/23/16 12:53 PM, Michael Niedermayer wrote:
On Wed, Apr 20, 2016 at 11:15:39PM -0700, Dave Yeo wrote:
On 04/20/16 03:48 PM, Dmitriy Kuminov wrote:
I do care about consistency, collaboration and prevention of artificial
entropy growth.

One option is to make ln_s a configure option, something like the
attached, which is a work in progress as out of tree builds are
broken when using cp -p. Almost seems a failure in the code that
decides whether symlinks work.
Dave

this seems not working

./configure
./configure: 4790: ./configure: /tmp/dest_sJUY97zC: Permission denied
rm: cannot remove `/tmp/name_cM7p9K1e': No such file or directory


It's the correct output (I get similar even before this patch) when
$ln_s="cp -p" as it tries to touch a file in a symlinked directory
and correctly fails.

there are no error messages before here, also the ln_s and ln_s_default
are not connected i think
set_default or similar is missing

OK, thanks, that's what I missed.



Did you run make && make install after configuring?
Here's the version I'll post later in a new thread, please test.

this seems to apply only "in reverse"

Strange, wonder how I did that.
New patch which I think along with Dmitriys could be applied if KOMH has no objections
Dave
Dave
From 341e82add973c1ff54f93b0eca7278d5958e6b55 Mon Sep 17 00:00:00 2001
From: Dave Yeo <daveryeo@telus.net>
Date: Mon, 25 Apr 2016 07:39:08 -0700
Subject: [PATCH 2/2] configure: Allow choice in choosing a symlink command

Signed-off-by: Dave Yeo <daveryeo@telus.net>
---
 configure | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/configure b/configure
index c96b845..d75ff04 100755
--- a/configure
+++ b/configure
@@ -314,6 +314,7 @@ Toolchain options:
   --nm=NM                  use nm tool NM [$nm_default]
   --ar=AR                  use archive tool AR [$ar_default]
   --as=AS                  use assembler AS [$as_default]
+  --ln_s=LN_S              use symbolic link tool LN_S [$ln_s_default]
   --strip=STRIP            use strip tool STRIP [$strip_default]
   --windres=WINDRES        use windows resource compiler WINDRES [$windres_default]
   --yasmexe=EXE            use yasm-compatible assembler EXE [$yasmexe_default]
@@ -2137,6 +2138,7 @@ CMDLINE_SET="
     host_os
     install
     ld
+    ln_s
     logfile
     malloc_prefix
     nm
@@ -3078,7 +3080,7 @@ host_cc_default="gcc"
 cp_f="cp -f"
 doxygen_default="doxygen"
 install="install"
-ln_s="ln -s -f"
+ln_s_default="ln -s -f"
 nm_default="nm -g"
 objformat="elf"
 pkg_config_default=pkg-config
@@ -4001,7 +4003,7 @@ test -n "$cc_type" && enable $cc_type ||
 : ${dep_cc_default:=$cc}
 : ${ld_default:=$cc}
 : ${host_ld_default:=$host_cc}
-set_default ar as objcc dep_cc ld host_ld windres
+set_default ar as objcc dep_cc ld ln_s host_ld windres
 
 probe_cc as "$as"
 asflags_filter=$_flags_filter
-- 
2.0.0

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to