Re: [FFmpeg-devel] PATCH for building with nasm
On Mon, Jan 02, 2017 at 10:54:18AM -0500, Ronald S. Bultje wrote: > Hi, > > On Sun, Jan 1, 2017 at 8:36 PM, Michael Niedermayer > wrote: > > > On Sun, Jan 01, 2017 at 08:16:00AM +, Rostislav Pehlivanov wrote: > > > On 1 January 2017 at 07:52, John Comeau wrote: > > > > > > > fixes `operation size not specified` errors as described here: > > > > http://stackoverflow.com/questions/36854583/compiling- > > > > ffmpeg-for-kali-linux-2 > > > > > > > > I rebuilt again with yasm and made sure it didn't break that. > > > > -- > > > > John Comeau KE5TFZ j...@unternet.net http://jc.unternet.net/ > > > > "A place for everything, and everything all over the place" > > > > > > > > ___ > > > > ffmpeg-devel mailing list > > > > ffmpeg-devel@ffmpeg.org > > > > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > > > > > > > > Odd, I've been using nasm to build ffmpeg for years now. > > > What version do you use? It sure works on nasm 2.12.01 here. > > > > do you see a disadvantage in the change by the patch ? > > if not, then i think it should be applied as it seems improving > > support for some nasm version be that the only issue or one of > > several > > > I think it's fine to merge. applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB In a rich man's house there is no place to spit but his face. -- Diogenes of Sinope signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] PATCH for building with nasm
Hi, On Sun, Jan 1, 2017 at 8:36 PM, Michael Niedermayer wrote: > On Sun, Jan 01, 2017 at 08:16:00AM +, Rostislav Pehlivanov wrote: > > On 1 January 2017 at 07:52, John Comeau wrote: > > > > > fixes `operation size not specified` errors as described here: > > > http://stackoverflow.com/questions/36854583/compiling- > > > ffmpeg-for-kali-linux-2 > > > > > > I rebuilt again with yasm and made sure it didn't break that. > > > -- > > > John Comeau KE5TFZ j...@unternet.net http://jc.unternet.net/ > > > "A place for everything, and everything all over the place" > > > > > > ___ > > > ffmpeg-devel mailing list > > > ffmpeg-devel@ffmpeg.org > > > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > > > > > Odd, I've been using nasm to build ffmpeg for years now. > > What version do you use? It sure works on nasm 2.12.01 here. > > do you see a disadvantage in the change by the patch ? > if not, then i think it should be applied as it seems improving > support for some nasm version be that the only issue or one of > several I think it's fine to merge. Ronald ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] PATCH for building with nasm
On Sun, Jan 01, 2017 at 08:16:00AM +, Rostislav Pehlivanov wrote: > On 1 January 2017 at 07:52, John Comeau wrote: > > > fixes `operation size not specified` errors as described here: > > http://stackoverflow.com/questions/36854583/compiling- > > ffmpeg-for-kali-linux-2 > > > > I rebuilt again with yasm and made sure it didn't break that. > > -- > > John Comeau KE5TFZ j...@unternet.net http://jc.unternet.net/ > > "A place for everything, and everything all over the place" > > > > ___ > > ffmpeg-devel mailing list > > ffmpeg-devel@ffmpeg.org > > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > > Odd, I've been using nasm to build ffmpeg for years now. > What version do you use? It sure works on nasm 2.12.01 here. do you see a disadvantage in the change by the patch ? if not, then i think it should be applied as it seems improving support for some nasm version be that the only issue or one of several [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The real ebay dictionary, page 1 "Used only once"- "Some unspecified defect prevented a second use" "In good condition" - "Can be repaird by experienced expert" "As is" - "You wouldnt want it even if you were payed for it, if you knew ..." signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] PATCH for building with nasm
On Sun, Jan 1, 2017 at 11:08 AM, Carl Eugen Hoyos wrote: > 2017-01-01 17:02 GMT+01:00 John Comeau : >> nasm version 2.11.05, which is current in Debian stable. I could use >> 2.12.01 from sid, but should that be a requirement? > > I thought 2.11 has more issues with FFmpeg, or was that only on osx? > > Please avoid top-posting here, Carl Eugen sorry about the top-posting, I haven't used a listserv for so long I'd forgotten the etiquette. there were no other errors after fixing those 3 lines. whether or not it actually built correctly I can't say. I didn't run `make check` after the nasm build, and on my slow netbook I don't want to do it over again and waste a few more hours. I have no emotional attachment to the patches I submitted, so feel free to ignore. my fork is up on github if anybody wants to take a look. I'm unsubscribing for now, so if there's any more commentary on this or my Floureon ipcam patch, please CC me. thanks -- jc -- John Comeau KE5TFZ j...@unternet.net http://jc.unternet.net/ "A place for everything, and everything all over the place" ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] PATCH for building with nasm
2017-01-01 17:02 GMT+01:00 John Comeau : > nasm version 2.11.05, which is current in Debian stable. I could use > 2.12.01 from sid, but should that be a requirement? I thought 2.11 has more issues with FFmpeg, or was that only on osx? Please avoid top-posting here, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] PATCH for building with nasm
nasm version 2.11.05, which is current in Debian stable. I could use 2.12.01 from sid, but should that be a requirement? On 1/1/17, Rostislav Pehlivanov wrote: > On 1 January 2017 at 07:52, John Comeau wrote: > >> fixes `operation size not specified` errors as described here: >> http://stackoverflow.com/questions/36854583/compiling- >> ffmpeg-for-kali-linux-2 >> >> I rebuilt again with yasm and made sure it didn't break that. >> -- >> John Comeau KE5TFZ j...@unternet.net http://jc.unternet.net/ >> "A place for everything, and everything all over the place" >> >> ___ >> ffmpeg-devel mailing list >> ffmpeg-devel@ffmpeg.org >> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel >> >> > Odd, I've been using nasm to build ffmpeg for years now. > What version do you use? It sure works on nasm 2.12.01 here. > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > -- John Comeau http://jc.unternet.net/ "A place for everything, and everything all over the place" ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] PATCH for building with nasm
On 1 January 2017 at 07:52, John Comeau wrote: > fixes `operation size not specified` errors as described here: > http://stackoverflow.com/questions/36854583/compiling- > ffmpeg-for-kali-linux-2 > > I rebuilt again with yasm and made sure it didn't break that. > -- > John Comeau KE5TFZ j...@unternet.net http://jc.unternet.net/ > "A place for everything, and everything all over the place" > > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > Odd, I've been using nasm to build ffmpeg for years now. What version do you use? It sure works on nasm 2.12.01 here. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel