Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

LameXP v4.16 [2018-04-30] now failing when converting #67

Closed
smileystv opened this issue May 21, 2018 · 9 comments
Closed

LameXP v4.16 [2018-04-30] now failing when converting #67

smileystv opened this issue May 21, 2018 · 9 comments
Labels
Fixed! Investigating

Comments

@smileystv
Copy link

Hello,

I have no idea what changed, but I have been batch converting MP4s and all of them fail within milliseconds.

Nothing has changed on my end. I was converting and then an update appeared and then everything failed thereafter.

I have installed and reinstalled... Same issue.

Any advice?
@smileystv
Copy link
Author

A log file output:

LameXP v4.16 (Build #2134), compiled on 2018-04-30 at 14:48:13

C:/Users/STEVEV1/AppData/Local/Temp/813c3f6bdd675e0b/lxp_faad.exe -o C:\Users\XXXXX1\AppData\Local\Temp\813c3f6bdd675e0b\01e2464ec9f6f3c4.wav "C:\Users\XXXXXXX\Desktop\New folder\Kacey Musgraves High Horse (Live).mp4"

*********** Ahead Software MPEG-4 AAC Decoder V2.8.6 ******************
Build: Dec 16 2017
Copyright 2002-2004: Ahead Software AG
http://www.audiocoding.com
bug tracking: https://sourceforge.net/p/faac/bugs/
Floating point version
This program is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License.

**** MP4 header ****
Brand: isom(version 512)
Compatible brands: isomiso2avc1mp41
*track media type: 'vide': unsupported, skipping
*track media type: 'soun': OK
parse:-1
Error opening file: C:\Users\XXXXXX\Desktop\New folder\Kacey Musgraves High Horse (Live).mp4

Exited with code: 0x0000

@lordmulder
Copy link
Owner

lordmulder commented May 23, 2018 •
edited
Loading

Hello,

can you please provide an example MP4 file that I can use to reproduce the issue?

(Looks like FAAD fails to decode your specific MP4 file. Maybe regression in latest FAAD version?)

Regards
MuldeR

@smileystv
Copy link
Author

smileystv commented May 23, 2018 via email

@lordmulder
Copy link
Owner

lordmulder commented May 23, 2018 •
edited
Loading

The file is 33MB and sent via Dropbox.

Okay. Can you please send me the Dropbox link via e-mail then, so I can try to reproduce?

I even changed the file name to "1" and it still doesn't work.

I don't think the file name is the problem (but yeah, you never know). Maybe some property of the file.

I reverted back to the previous version though and all is well no matter the naming?

Well, I updated FAAD to latest version in the new LameXP release. So, maybe, a regression in FAAD.

I would like to check this with the sample file...

Regards.

@lordmulder
Copy link
Owner

It appears that FAAD v2.8 doesn't like MP4 files containing a video stream!

I have reverted to FAAD v2.7 for now:
https://forum.doom9.org/showpost.php?p=1843480&postcount=1547

I also filed a bug-report for FAAD, so let's see what happens:
knik0/faad2#13

Regards.

@smileystv
Copy link
Author

smileystv commented Jun 4, 2018 via email

@smileystv
Copy link
Author

smileystv commented Jun 27, 2018 via email

@lordmulder
Copy link
Owner

lordmulder commented Jun 27, 2018 •
edited
Loading

Unfortunately, no. There is no reply from the FAAD developer to my report yet 😩

I probably will stick with FAAD v2.7 for the next release of LameXP, as in the latest Alpha version:
LameXP-ALPHA.2018-06-03.Release-Static.Build-2142.exe

@lordmulder
Copy link
Owner

Should be fixed in LameXP v4.17 RC-4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Fixed! Investigating
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lordmulder @smileystv