[PATCH 1/2] configure: remove 'T' flag from ar flags on OS/2
OS/2 linker does not support a thin archive. --- configure | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configure b/configure index 2e94bf1f0..9d21320ec 100755 --- a/configure +++ b/configure @@ -1854,6 +1854,8 @@ if os2 ; then # And ensure that $ld_static should be at first in a library list because it # has effects only libraries after it. test -n "$ld_static" && ld_static='-Wl,-static' + # OS/2 linker does not support a thin archive. Remove 'T' flag. + _arflags=$(echo $_arflags | tr -d T) fi if wine ; then -- 2.13.3
s/parser/parsers/ --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 9d21320ec..d0f619130 100755 --- a/configure +++ b/configure @@ -1554,7 +1554,7 @@ list_subparts_filters() { echocheck "ffmpeg/libavcodec/allcodecs.c" libavdecoders_all=$(list_subparts_extern AVCodec decoder codec/allcodecs.c) libavencoders_all=$(list_subparts_extern AVCodec encoder codec/allcodecs.c) -libavparsers_all=$(list_subparts_extern AVCodecParser parser codec/parser.c) +libavparsers_all=$(list_subparts_extern AVCodecParser parser codec/parsers.c) test $? -eq 0 && _list_subparts=found || _list_subparts="not found" echores "$_list_subparts" -- 2.13.3
Hi! On 2018-08-06 17:23 +0900, KO Myung-Hun wrote:
s/parser/parsers/ --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure b/configure index 9d21320ec..d0f619130 100755 --- a/configure +++ b/configure @@ -1554,7 +1554,7 @@ list_subparts_filters() { echocheck "ffmpeg/libavcodec/allcodecs.c" libavdecoders_all=$(list_subparts_extern AVCodec decoder codec/allcodecs.c) libavencoders_all=$(list_subparts_extern AVCodec encoder codec/allcodecs.c) -libavparsers_all=$(list_subparts_extern AVCodecParser parser codec/parser.c) +libavparsers_all=$(list_subparts_extern AVCodecParser parser codec/parsers.c)
Committed as SVN r38114 Thanks Alexander
Hi! On 2018-08-06 17:23 +0900, KO Myung-Hun wrote:
OS/2 linker does not support a thin archive. --- configure | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/configure b/configure index 2e94bf1f0..9d21320ec 100755 --- a/configure +++ b/configure @@ -1854,6 +1854,8 @@ if os2 ; then # And ensure that $ld_static should be at first in a library list because it # has effects only libraries after it. test -n "$ld_static" && ld_static='-Wl,-static' + # OS/2 linker does not support a thin archive. Remove 'T' flag. + _arflags=$(echo $_arflags | tr -d T)
If the build on OS/2 is broken and works again after this patch, commit please. (You can commit to MPlayer SVN, right?) Can you confirm that in your installation sh -x -c 'ar 2>&1 | grep "\[T\] " ; echo "RC $?"' works as expected? If so does it mean OS/2 "ar" can create thin arhives but the remaining tool chain does not support linking them? Alexander
fi
if wine ; then
Hi/2. Alexander Strasser wrote:
Hi!
On 2018-08-06 17:23 +0900, KO Myung-Hun wrote:
OS/2 linker does not support a thin archive. --- configure | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/configure b/configure index 2e94bf1f0..9d21320ec 100755 --- a/configure +++ b/configure @@ -1854,6 +1854,8 @@ if os2 ; then # And ensure that $ld_static should be at first in a library list because it # has effects only libraries after it. test -n "$ld_static" && ld_static='-Wl,-static' + # OS/2 linker does not support a thin archive. Remove 'T' flag. + _arflags=$(echo $_arflags | tr -d T)
If the build on OS/2 is broken and works again after this patch, commit please. (You can commit to MPlayer SVN, right?)
Yes. But currently, my svn commit does not work due to symbolic link. I'll appreciate if you commit instead. ^______^
Can you confirm that in your installation
sh -x -c 'ar 2>&1 | grep "\[T\] " ; echo "RC $?"'
works as expected?
If so does it mean OS/2 "ar" can create thin arhives but the remaining tool chain does not support linking them?
Exactly. -- KO Myung-Hun Using Mozilla SeaMonkey 2.7.2 Under OS/2 Warp 4 for Korean with FixPak #15 In VirtualBox v4.1.32 on Intel Core i7-3615QM 2.30GHz with 8GB RAM Korean OS/2 User Community : http://www.os2.kr/
On 2018-08-13 18:03 +0900, KO Myung-Hun wrote:
Alexander Strasser wrote:
On 2018-08-06 17:23 +0900, KO Myung-Hun wrote:
OS/2 linker does not support a thin archive. --- configure | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/configure b/configure index 2e94bf1f0..9d21320ec 100755 --- a/configure +++ b/configure @@ -1854,6 +1854,8 @@ if os2 ; then # And ensure that $ld_static should be at first in a library list because it # has effects only libraries after it. test -n "$ld_static" && ld_static='-Wl,-static' + # OS/2 linker does not support a thin archive. Remove 'T' flag. + _arflags=$(echo $_arflags | tr -d T)
If the build on OS/2 is broken and works again after this patch, commit please. (You can commit to MPlayer SVN, right?)
Yes. But currently, my svn commit does not work due to symbolic link.
I'll appreciate if you commit instead. ^______^
Committed. [...] Alexander
On 2018-08-13 18:03 +0900, KO Myung-Hun wrote:
Alexander Strasser wrote:
On 2018-08-06 17:23 +0900, KO Myung-Hun wrote:
OS/2 linker does not support a thin archive. --- configure | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/configure b/configure index 2e94bf1f0..9d21320ec 100755 --- a/configure +++ b/configure @@ -1854,6 +1854,8 @@ if os2 ; then # And ensure that $ld_static should be at first in a library list because it # has effects only libraries after it. test -n "$ld_static" && ld_static='-Wl,-static' + # OS/2 linker does not support a thin archive. Remove 'T' flag. + _arflags=$(echo $_arflags | tr -d T)
If the build on OS/2 is broken and works again after this patch, commit please. (You can commit to MPlayer SVN, right?)
Yes. But currently, my svn commit does not work due to symbolic link.
I'll appreciate if you commit instead. ^______^
Committed as SVN r38115 [...] Alexander
Thanks a lot ! Alexander Strasser wrote:
On 2018-08-13 18:03 +0900, KO Myung-Hun wrote:
Alexander Strasser wrote:
On 2018-08-06 17:23 +0900, KO Myung-Hun wrote:
OS/2 linker does not support a thin archive. --- configure | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/configure b/configure index 2e94bf1f0..9d21320ec 100755 --- a/configure +++ b/configure @@ -1854,6 +1854,8 @@ if os2 ; then # And ensure that $ld_static should be at first in a library list because it # has effects only libraries after it. test -n "$ld_static" && ld_static='-Wl,-static' + # OS/2 linker does not support a thin archive. Remove 'T' flag. + _arflags=$(echo $_arflags | tr -d T)
If the build on OS/2 is broken and works again after this patch, commit please. (You can commit to MPlayer SVN, right?)
Yes. But currently, my svn commit does not work due to symbolic link.
I'll appreciate if you commit instead. ^______^
Committed as SVN r38115
[...]
Alexander _______________________________________________ MPlayer-dev-eng mailing list MPlayer-dev-eng@mplayerhq.hu https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
-- KO Myung-Hun Using Mozilla SeaMonkey 2.7.2 Under OS/2 Warp 4 for Korean with FixPak #15 In VirtualBox v4.1.32 on Intel Core i7-3615QM 2.30GHz with 8GB RAM Korean OS/2 User Community : http://www.os2.kr/
participants (2)
-
Alexander Strasser -
KO Myung-Hun