Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On Wed, Apr 27, 2016 at 05:31:43PM -0700, Dave Yeo wrote: > On 04/27/16 04:05 PM, Michael Niedermayer wrote: > >On Tue, Apr 26, 2016 at 06:15:26PM +0300, Dmitriy Kuminov wrote: > >>On 2016-04-26 11:51:28 +, KO Myung-Hun said: > >> > >>>However, ln_s is always set to 'cp -f' on OS/2. Please

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On 04/27/16 04:05 PM, Michael Niedermayer wrote: On Tue, Apr 26, 2016 at 06:15:26PM +0300, Dmitriy Kuminov wrote: On 2016-04-26 11:51:28 +, KO Myung-Hun said: However, ln_s is always set to 'cp -f' on OS/2. Please remove ln_s overriding codes as well. Yes, but my initial patch already

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On Tue, Apr 26, 2016 at 06:15:26PM +0300, Dmitriy Kuminov wrote: > On 2016-04-26 11:51:28 +, KO Myung-Hun said: > > >However, ln_s is always set to 'cp -f' on OS/2. Please remove ln_s > >overriding codes as well. > > Yes, but my initial patch already removes this override (check the > first

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On 2016-04-26 11:51:28 +, KO Myung-Hun said: However, ln_s is always set to 'cp -f' on OS/2. Please remove ln_s overriding codes as well. Yes, but my initial patch already removes this override (check the first hunk) so just applying it + Dave's should get it all done. -- Kind regards,

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

Dave Yeo wrote: > 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:

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

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,

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

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

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

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

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

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

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On 2016-04-21 00:05:57 +, Michael Niedermayer said: On Thu, Apr 21, 2016 at 01:48:22AM +0300, Dmitriy Kuminov wrote: you all want the same, a perfect solution iam sure you can reach some consensus about what to do and work together to do it. Iam happy to apply whatever patch the OS/2

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

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.

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On Thu, Apr 21, 2016 at 01:48:22AM +0300, Dmitriy Kuminov wrote: > On 2016-04-20 11:48:33 +, KO Myung-Hun said: [...] > A question to maintainers with commit rights: please clarify, what > should I do to have my patches applied? you all want the same, a perfect solution iam sure you can reach

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On 2016-04-20 11:48:33 +, KO Myung-Hun said: No, it's not because ln_s overriding is already there. And there is just no need to replace it. Wrong. "There is no need" sounds like if it were the truth but in fact it's not, it's just your opinion. Did you present any real-life cases

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On 2016-04-19 12:56:23 +, KO Myung-Hun said: I don't understand why you insist on using symlink. Even if without it, current FFmpeg works well, maybe better in according to Dave. I don't know what is the benefit from using symlink. Likewise, I don't understand why you insist on not using

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On 2016-04-19 06:03:52 +, Dave Yeo said: The problem is that symlinks on OS/2 (unless using TVFS.IFS) are inherently fragile. It just takes one utility that is not aware of them to break things and not everything is linked against kLIBC. In FFmpegs case we use lxlite, which is a Pascal

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

Hi/2. Dmitriy Kuminov wrote: > On 2016-04-18 11:52:15 +, KO Myung-Hun said: > >> Strange conclusion. Anyway not important. > > In Dave's case the symlink functionality check fails because it is > performed in TMPDIR which is located on ramfs.ifs (which fails to use > symlinks). But I

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On 04/16/16 11:00 AM, Dmitriy Kuminov wrote: On 2016-04-16 17:24:12 +, Dave Yeo said: Actually I now get this at the beginning of the configure run, using a ramfs.ifs volume for $TMPDIR, [K:\usr\local\src\ffmpeg.obj]sh ../ffmpeg/configure --enable-gpl --disable-doc

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On 2016-04-18 11:52:15 +, KO Myung-Hun said: Strange conclusion. Anyway not important. In Dave's case the symlink functionality check fails because it is performed in TMPDIR which is located on ramfs.ifs (which fails to use symlinks). But I believe his real build directory is not on

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

Dmitriy Kuminov wrote: > On 2016-04-17 05:21:31 +, KO Myung-Hun said: > >> Even if it's a bug of ramfs.ifs, its bug should be considered when using >> ln -s. > > Yes, and in this particular case configure handles it safely. It results > into a false fallback though because the symlink dir

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On 2016-04-17 05:21:31 +, KO Myung-Hun said: Even if it's a bug of ramfs.ifs, its bug should be considered when using ln -s. Yes, and in this particular case configure handles it safely. It results into a false fallback though because the symlink dir check is performed in TMPDIR which

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

Dmitriy Kuminov wrote: > On 2016-04-16 17:24:12 +, Dave Yeo said: > >> Actually I now get this at the beginning of the configure run, using a >> ramfs.ifs volume for $TMPDIR, >> >> [K:\usr\local\src\ffmpeg.obj]sh ../ffmpeg/configure --enable-gpl >> --disable-doc

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On 2016-04-16 17:24:12 +, Dave Yeo said: Actually I now get this at the beginning of the configure run, using a ramfs.ifs volume for $TMPDIR, [K:\usr\local\src\ffmpeg.obj]sh ../ffmpeg/configure --enable-gpl --disable-doc --samples=/usr/local/share/fate-suite --cpu=i686 --extra-libs=-lpoll

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On 04/16/16 09:29 AM, Dmitriy Kuminov wrote: On 2016-04-16 04:52:23 +, KO Myung-Hun said: diff --git a/configure b/configure index ff80aee..36a23f4 100755 --- a/configure +++ b/configure @@ -4716,7 +4716,6 @@ case $target_os in ;; os/2*) strip="lxlite -CS" -ln_s="cp -f" Please do

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On 2016-04-16 04:52:23 +, KO Myung-Hun said: diff --git a/configure b/configure index ff80aee..36a23f4 100755 --- a/configure +++ b/configure @@ -4716,7 +4716,6 @@ case $target_os in ;; os/2*) strip="lxlite -CS" -ln_s="cp -f" Please do not remove this. Copying is more compatible

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

Hi/2. Dave Yeo wrote: > On 04/14/16 07:29 PM, Dave Yeo wrote: >>> From fbff28a583cc9fd7144dd4f9dc37b7b5f8aa Mon Sep 17 00:00:00 2001 >> From: Dmitriy Kuminov >> Date: Thu, 14 Apr 2016 01:45:01 +0400 >> Subject: [PATCH 2/3] configure: Do not create/install versioned DLLs on

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On 04/14/16 07:29 PM, Dave Yeo wrote: From fbff28a583cc9fd7144dd4f9dc37b7b5f8aa Mon Sep 17 00:00:00 2001 From: Dmitriy Kuminov Date: Thu, 14 Apr 2016 01:45:01 +0400 Subject: [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2. Attached Dave From

Re: [FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

On Thu, Apr 14, 2016 at 07:29:26PM -0700, Dave Yeo wrote: > >From fbff28a583cc9fd7144dd4f9dc37b7b5f8aa Mon Sep 17 00:00:00 2001 > From: Dmitriy Kuminov > Date: Thu, 14 Apr 2016 01:45:01 +0400 > Subject: [PATCH 2/3] configure: Do not create/install versioned DLLs > on OS/2. >