Opened 8 years ago

Closed 4 years ago

#5281 closed defect (fixed)

FFmpeg doesn't care about existing output file

Reported by: Xenos Owned by:
Priority: important Component: ffmpeg
Version: git-master Keywords: win
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Executing this command

ffmpeg.exe -i "H:\Musiques\Musique Papa\BACH JS\BACH JS - Intégrale Brilliiant - Volume IV - Cantates II\BACH JS - Cantantes II - 1 - 110 198\02 Piste 2.wma" -vn -acodec vorbis -strict -2 -ac 2 "H:\Musiques\Musique Papa\BACH JS\BACH JS - Intégrale Brilliiant - Volume IV - Cantates II\BACH JS - Cantantes II - 1 - 110 198\02 Piste 2.ogg"

With ffmpeg "N-77870-gb58cfa6" or "N-50911-g9efcfbe" (seriously?!) and while both files exist should output for an overwrite confirmation. It actually doesn't: FFmpeg overrides the output without a warning nor a confirm.

Hence, having the "-n" option doens't work either: FFMpeg overwrites the output file.

Change History (10)

comment:1 by Elon Musk, 8 years ago

Please post full ffmpeg output.

comment:2 by Carl Eugen Hoyos, 8 years ago

Keywords: win added; -n overwrite output removed
Version: unspecifiedgit-master

Sounds related to ticket #819
Please test with a simpler command line and provide command line including complete, uncut console output to make this a valid ticket.

$ ffmpeg -f lavfi -i sine=d=1 Intégrale.wav

comment:3 by Xenos, 8 years ago

$ C:\Windows\system32>ffmpeg -n -i "H:\Musiques\Musique Papa\BACH JS\BACH JS - Intégrale Brilliiant - Volume IV - Cantates II\BACH JS - Cantantes II - 1 - 110 198\02 Piste 2.wma" -vn -acodec vorbis -strict -2 -ac 2 "H:\Musiques\Musique Papa\BACH JS\BACH JS - Intégrale Brilliiant - Volume IV - Cantates II\BACH JS - Cantantes II - 1 - 110 198\02 Piste 2.ogg"
ffmpeg version N-50911-g9efcfbe Copyright (c) 2000-2013 the FFmpeg developers
  built on Mar 13 2013 21:26:48 with gcc 4.7.2 (GCC)
  configuration: --enable-gpl --enable-version3 --disable-w32threads --enable-avisynth --enable-bzlib --enable-fontconfig --enable-frei0r --enable-gnutls --enable-libass --enable-libbluray --enable-libcaca --enable-libfreetype --enable-libgsm --enable-libilbc --enable-libmp3lame --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-libopus --enable-librtmp --enable-libschroedinger --enable-libsoxr --enable-libspeex --enable-libtheora --enable-libtwolame --enable-libvo-aacenc --enable-libvo-amrwbenc --enable-libvorbis --enable-libvpx --enable-libx264 --enable-libxavs --enable-libxvid --enable-zlib
  libavutil      52. 19.100 / 52. 19.100
  libavcodec     55.  0.100 / 55.  0.100
  libavformat    55.  0.100 / 55.  0.100
  libavdevice    54.  4.100 / 54.  4.100
  libavfilter     3. 45.103 /  3. 45.103
  libswscale      2.  2.100 /  2.  2.100
  libswresample   0. 17.102 /  0. 17.102
  libpostproc    52.  2.100 / 52.  2.100
Guessed Channel Layout for  Input Stream #0.0 : stereo
Input #0, asf, from 'H:\Musiques\Musique Papa\BACH JS\BACH JS - Int├®grale Brilliiant - Volume IV - Cantates II\BACH JS - Cantantes II - 1 - 110 198\02 Piste 2.wma':
  Metadata:
    title           : Piste 2
    artist          : Artiste inconnu
    genre           : Inconnu
    album           : Album inconnu (15/11/2009 10:31:31)
    WM/Track        : 1
    track           : 2
    WMFSDKVersion   : 8.00.00.4487
    WMFSDKNeeded    : 0.0.0.0000
  Duration: 00:01:23.41, start: 0.000000, bitrate: 194 kb/s
    Stream #0:0: Audio: wmav2 (a[1][0][0] / 0x0161), 44100 Hz, stereo, fltp, 192 kb/s
Output #0, ogg, to 'H:\Musiques\Musique Papa\BACH JS\BACH JS - Int├®grale Brilliiant - Volume IV - Cantates II\BACH JS - Cantantes II - 1 - 110 198\02 Piste 2.ogg':
  Metadata:
    title           : Piste 2
    artist          : Artiste inconnu
    genre           : Inconnu
    album           : Album inconnu (15/11/2009 10:31:31)
    WM/Track        : 1
    TRACKNUMBER     : 2
    WMFSDKVersion   : 8.00.00.4487
    WMFSDKNeeded    : 0.0.0.0000
    encoder         : Lavf55.0.100
    Stream #0:0: Audio: vorbis, 44100 Hz, stereo, fltp
Stream mapping:
  Stream #0:0 -> #0:0 (wmav2 -> vorbis)
Press [q] to stop, [?] for help
[ogg @ 0241d2a0] st:0 PTS: 14333 DTS: 14333 < 15361 invalid, clipping
[ogg @ 0241d2a0] st:0 PTS: 15357 DTS: 15357 < 15362 invalid, clipping
size=    1822kB time=00:01:23.28 bitrate= 179.2kbits/s
video:0kB audio:1807kB subtitle:0 global headers:3kB muxing overhead 0.620156%

This might be an encoding related issue.
If that's the issue, there should be at least a warning in the ffmpeg documentation (--help option).

Last edited 8 years ago by Carl Eugen Hoyos (previous) (diff)

comment:4 by Carl Eugen Hoyos, 8 years ago

Please test the command line that I provided, you have to run twice to test.

comment:5 by Xenos, 8 years ago

For the old ffmpeg version:

[lavfi @ 01fcb9a0] No such filter: 'sine' sine=d=1: Invalid argument

For the newer
$ ffmpeg -f lavfi -i sine=d=1 "01 Piste 1.wma"

Detects the overwritable output file (hence, asks for Overwrite? [y/N]), but a relative or absolute path with "Intégrale" doesn't: the file is created at the right place (Intégrale/...ogg") but the checker fails to detect this file already exists (since it seems to check for "Int├®grale").

Either you should put a simple warning in the documentation saying that accent (or non-base-ASCII chars) will fail, or you should fix the checker to use the same encoding as the converters (since the converted file is at the right place, meaning that the converter uses the correct path but the "does-output-exist" checker doesn't).

comment:6 by Elon Musk, 8 years ago

Status: newopen

comment:7 by Carl Eugen Hoyos, 8 years ago

So is the issue reproducible with the following command line?

$ ffmpeg -f lavfi -i sine=d=1 .\Intégrale.wav

comment:8 by Xenos, 8 years ago

$ ffmpeg -f lavfi -i sine=d=1 MesGenoux.wav
Does work and creates a "MesGenoux.wav"; run twice and it warns for existing file

$ ffmpeg -f lavfi -i sine=d=1 Intégrale.wav
Does not work: no warning for overwriting, but gen file is at the right place of "Intégrale.wav"

$ ffmpeg -f lavfi -i sine=d=1 Intégrale\MesGenoux.wav
Does not work: no warning for overwriting, but gen file is at the right place of "Intégrale\MesGenoux.wav"

Last edited 8 years ago by Xenos (previous) (diff)

comment:9 by Christian, 8 years ago

maybe is a matter of special char? I see you are have "é" try to use a normal "e".

comment:10 by Marton Balint, 4 years ago

Resolution: fixed
Status: openclosed

This is already fixed in 37787f261639c53998487400e874741c17e85fc6.

Note: See TracTickets for help on using tickets.