Re: [FFmpeg-trac] #7310(build system:new): configure broken with GNU sed version 4.1.5

[FFmpeg]

#7310: configure broken with GNU sed version 4.1.5
-+-
 Reporter:  cehoyos  |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  build
  Version:  git-master   |  system
 Keywords:  regression   |   Resolution:
 Blocking:   |   Blocked By:
Analyzed by developer:  0|  Reproduced by developer:  0
-+-

Comment (by michael.kostylev):

 The above quick test passed on FreeBSD 11.2, OpenBSD 6.2, Solaris 10,11,
 OpenIndiana 2017.10, AIX 7.1, numerous ancient Linuxes.
 And FATE passed on Sol 11.

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-trac

Re: [FFmpeg-trac] #7310(build system:new): configure broken with GNU sed version 4.1.5

[FFmpeg]

#7310: configure broken with GNU sed version 4.1.5
-+-
 Reporter:  cehoyos  |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  build
  Version:  git-master   |  system
 Keywords:  regression   |   Resolution:
 Blocking:   |   Blocked By:
Analyzed by developer:  0|  Reproduced by developer:  0
-+-

Comment (by michael.kostylev):

 If a user cannot replace sed or does not want to patch configure, then one
 can override stat("usr/xpg4/bin/sed",...)" via LD_PRELOAD. Obviously, this
 is overkill, but it works. Probably, the LD_PRELOAD trick would be more
 useful when a user cannot replace (broken) /bin/sh to run the FATE
 scripts.
 Going back to sed. Can someone test the following:
 {{{
 % for sed in $(which -a sed) ; do echo "$sed:" ; for expr in av_AV av_AZ1
 avx_z2 avfrc_b34 aa_X5 avsin_Abc_09 ; do echo 'extern AVFilter
 ff_'$expr';' | $sed -n 's/^extern AVFilter
 ff_[avfsinkrc]\{2,5\}_\([a-zA-Z0-9_]\{1,\}\);/\1_filter/p' ; done ; echo ;
 done
 }}}
 The output should be like this:
 {{{
 /usr/xpg4/bin/sed:
 AV_filter
 AZ1_filter
 b34_filter
 Abc_09_filter

 /opt/csw/gnu/sed:
 AV_filter
 AZ1_filter
 b34_filter
 Abc_09_filter

 /usr/bin/sed:
 AV_filter
 AZ1_filter
 b34_filter
 Abc_09_filter

 }}}
 In that case we can replace the first ERE with the BRE:
 {{{
 -sed -E -n "s/^extern AVFilter
 ff_([avfsinkrc]{2,5})_([a-zA-Z0-9_]+);/\2_filter/p" $file
 +sed -n "s/^extern AVFilter
 ff_[avfsinkrc]\{2,5\}_\([a-zA-Z0-9_]\{1,\}\);/\1_filter/p" $file
 }}}

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-trac

Re: [FFmpeg-trac] #7310(build system:new): configure broken with GNU sed version 4.1.5

[FFmpeg]

#7310: configure broken with GNU sed version 4.1.5
-+-
 Reporter:  cehoyos  |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  build
  Version:  git-master   |  system
 Keywords:  regression   |   Resolution:
 Blocking:   |   Blocked By:
Analyzed by developer:  0|  Reproduced by developer:  0
-+-

Comment (by cehoyos):

 This is made worse by the fact that our configure script on Solaris
 insists on using `/usr/xpg4/bin/sed` and the user cannot change this.

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-trac

Re: [FFmpeg-trac] #7310(build system:new): configure broken with GNU sed version 4.1.5

[FFmpeg]

#7310: configure broken with GNU sed version 4.1.5
-+-
 Reporter:  cehoyos  |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  build
  Version:  git-master   |  system
 Keywords:  regression   |   Resolution:
 Blocking:   |   Blocked By:
Analyzed by developer:  0|  Reproduced by developer:  0
-+-

Comment (by cehoyos):

 No, it is neither available on Solaris nor aix, it is a gnu extension.

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-trac

Re: [FFmpeg-trac] #7310(build system:new): configure broken with GNU sed version 4.1.5

[FFmpeg]

#7310: configure broken with GNU sed version 4.1.5
-+-
 Reporter:  cehoyos  |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  build
  Version:  git-master   |  system
 Keywords:  regression   |   Resolution:
 Blocking:   |   Blocked By:
Analyzed by developer:  0|  Reproduced by developer:  0
-+-

Comment (by jamrial):

 The output you pasted reports -r as a valid option. Is it not the sed
 available in Solaris and/or AIX? Am i missing something?

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-trac

Re: [FFmpeg-trac] #7310(build system:new): configure broken with GNU sed version 4.1.5

[FFmpeg]

#7310: configure broken with GNU sed version 4.1.5
-+-
 Reporter:  cehoyos  |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  build
  Version:  git-master   |  system
 Keywords:  regression   |   Resolution:
 Blocking:   |   Blocked By:
Analyzed by developer:  0|  Reproduced by developer:  0
-+-

Comment (by cehoyos):

 That wouldn't help on Solaris or aix.

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-trac

Re: [FFmpeg-trac] #7310(build system:new): configure broken with GNU sed version 4.1.5

[FFmpeg]

#7310: configure broken with GNU sed version 4.1.5
-+-
 Reporter:  cehoyos  |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  build
  Version:  git-master   |  system
 Keywords:  regression   |   Resolution:
 Blocking:   |   Blocked By:
Analyzed by developer:  0|  Reproduced by developer:  0
-+-

Comment (by jamrial):

 Newer sed shows

 {{{
 $ sed --help
 Usage: sed [OPTION]... {script-only-if-no-other-script} [input-file]...

   -n, --quiet, --silent
  suppress automatic printing of pattern space
   -e script, --expression=script
  add the script to the commands to be executed
   -f script-file, --file=script-file
  add the contents of script-file to the commands to be
 executed
   --follow-symlinks
  follow symlinks when processing in place
   -i[SUFFIX], --in-place[=SUFFIX]
  edit files in place (makes backup if SUFFIX supplied)
   -b, --binary
  open files in binary mode (CR+LFs are not processed
 specially)
   -l N, --line-length=N
  specify the desired line-wrap length for the `l' command
   --posix
  disable all GNU extensions.
   -E, -r, --regexp-extended
  use extended regular expressions in the script
  (for portability use POSIX -E).
   -s, --separate
  consider files as separate rather than as a single,
  continuous long stream.
   --sandbox
  operate in sandbox mode (disable e/r/w commands).
   -u, --unbuffered
  load minimal amounts of data from the input files and
 flush
  the output buffers more often
   -z, --null-data
  separate lines by NUL characters
   --help display this help and exit
   --version  output version information and exit
 }}}

 So looks like it should be a matter of replacing that -E with -r, assuming
 it doesn't break any other target.

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-trac

Re: [FFmpeg-trac] #7310(build system:new): configure broken with GNU sed version 4.1.5

[FFmpeg]

#7310: configure broken with GNU sed version 4.1.5
-+-
 Reporter:  cehoyos  |Owner:
 Type:  defect   |   Status:  new
 Priority:  important|Component:  build
  Version:  git-master   |  system
 Keywords:  regression   |   Resolution:
 Blocking:   |   Blocked By:
Analyzed by developer:  0|  Reproduced by developer:  0
-+-
Changes (by cehoyos):

 * priority:  normal => important


Comment:

 Regression since 8f1382f80e0d4184c54c14afdda6482f050fbba7, fails at least
 with AIX sed and Solaris sed.

--
Ticket URL: 
FFmpeg 
FFmpeg issue tracker
___
FFmpeg-trac mailing list
FFmpeg-trac@avcodec.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-trac