Closed GoogleCodeExporter closed 9 years ago
This should be a defect not an enhancment issue but I canont figure out how to
change
that in the issue report page.
Original comment by marcinp...@gmail.com
on 14 Dec 2008 at 7:28
Also please note that I changed the voribus quality setting to -q 10. But It
does
not work with the default as well.
Original comment by marcinp...@gmail.com
on 14 Dec 2008 at 7:29
I'll look into it. Thanks for the detailed bug report.
Original comment by rubyripp...@gmail.com
on 16 Dec 2008 at 7:05
I also had this problem and was able to solve it by removing the "-p 4"
parameter in
the call of oggenc in rr_lib.rb. I do not know what this parameter is intended
to do.
At least the two versions of oggenc I regularly use (OggEnc v1.0.2 and OggEnc
v1.0.2
aoTuV beta 4.51) do not know this parameter.
The attached diff removes the parameter and makes oggenc working for me with
rubyripper.
One general question: Would it not be better when only really needed parameters
(e.g., setting the tag values) are hard-coded in rrlib?
Additional parameters (such as -p 4 whatever it is intended to do or
--id3v2-only for
lame) IMHO should be defined by the user in the codec configuration. This would
avoid
such errors like this one, and the user would have more flexibility (e.g.,
allow also
id3v1 tags for mp3).
Original comment by peter...@yahoo.de
on 25 Dec 2008 at 2:16
Attachments:
I'm also experiencing the same error.
In reply to peter's comment on "-p 4":
I'm not sure what "-p 4" is being used for but after checking "man oggenc" I
found this:
-P, --name-replace=s
Replace characters removed by --name-remove with the characters
specified. If this string is shorter than the --name-remove
list, or is not specified, the extra characters are just
removed. The default settings for this option, and the -X option
above, are platform specific (and chosen to ensure legal file‐
names are generated for each platform).
I'm not sure if this would be the same as "-p" since the one I found from the
man
page is uppercase but, just the same, I thought I should mention it.
Original comment by user6035...@spamcorptastic.com
on 29 Dec 2008 at 10:34
I don't think it is the same because the oggenc's options are case-sensitive. My
oggenc version knows -P but does not know -p.
Original comment by peter...@yahoo.de
on 31 Dec 2008 at 2:49
The -p option is already removed on 12 november 2008. Closing the issue.
Original comment by rubyripp...@gmail.com
on 15 Jan 2009 at 8:11
Original issue reported on code.google.com by
marcinp...@gmail.com
on 14 Dec 2008 at 7:26