acidrip x264 strangness

[Brian Neltner]

Sorry, I wasn't on this email list.

I wrote patches years ago to include x264 and xvid support, but I think
the developer moved on to better pastures. I submitted it as a patch to
the ubuntu project, but in case anyone else wants them, I have attached
the patches.

Unfortunately, the code seems to have changed slightly since I updated
it, so there are some changes in the patch that I don't recognize
making. However, basically all that needed to be done was include x264
and xvid in the sanity checking in acidrip.pm and signals.pm. 

If someone is so inclined, they should isolate just those changes. I
also made other changes for my benefit, like recording the first pass to
temp.avi instead of /dev/null (I like to be able to check the video as
it's encoding to get some idea of if the bitrates are sane), and
including good high-quality defaults for x264. Sorry for the messy
patch, but the software works fine for me, so I don't see a need to do a
lot of work "re-breaking" it for what I need it to do.

When you patch it, it will rename the files. You'll have to move the
acidrip-updated.pm to acidrip.pm and signals-updated.pm to signals.pm
(sorry, I'm new at this submitting patches thing).

Cheers,
Brian Neltner

Back in January 2007, Ben Edwards wrote:
> 
> I have been playing around with x264 in acidrip as it is supposed to
> give better results to xvid (although encoding time is longer).  
> 
> When I select x264 a lot of the option boxes are greyed out, well
> almost all in fact.  The strangest thing is it is no longer possible
> to set a target file size of a bit-rate!  It also does not allow the 
> number of passes to be set.  Basically all you can use is the
> defaults ;(.
> 
> Any idea what is going on?
> 
> Ben
-------------- next part --------------
A non-text attachment was scrubbed...
Name: acidrip.pm.patch
Type: text/x-patch
Size: 5993 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-users/attachments/20080114/02890f71/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signals.pm.patch
Type: text/x-patch
Size: 609 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-users/attachments/20080114/02890f71/attachment-0001.bin>