Bug 448209 - totem-video-thumbnailer do not accept arguments alike " --size XXXxYYY"
totem-video-thumbnailer do not accept arguments alike " --size XXXxYYY"
Status: CLOSED NOTABUG
Product: Fedora
Classification: Fedora
Component: totem (Show other bugs)
9
All Linux
medium Severity medium
: ---
: ---
Assigned To: Bastien Nocera
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2008-05-24 09:22 EDT by Miao ZhiCheng
Modified: 2008-05-24 09:51 EDT (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-05-24 09:51:38 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Miao ZhiCheng 2008-05-24 09:22:56 EDT
Description of problem:
I have some scripts use totem-video-thumbnailer to get thumbnails of videos,
there was a long time that it accepted arguments alike :
[code]
 totem-video-thumbnailer --size ${HEIGHT}x${WIDTH} "$IN" "$OUT"
[/code]

Version-Release number of selected component (if applicable):
$ rpm -qf /usr/bin/totem-video-thumbnailer totem-2.23.2-2.fc9.i386


How reproducible:
Every time

Steps to Reproduce:
1. find a video
2. run something like 
$ totem-video-thumbnailer --size 100x200 1.ogm 2.png

  
Actual results:
couldn't parse command-line options: Cannot parse integer value '100x200' for --size


Expected results:
Get proper thumbnail

Additional info:
Comment 1 Bastien Nocera 2008-05-24 09:51:38 EDT
You can see the port to GOption here:
http://svn.gnome.org/viewvc/totem/trunk/src/totem-video-thumbnailer.c?r1=3757&r2=3770

Basically, what happened was that the second part of the size was completely
ignored:
size = g_strtod (argv[++i], NULL);

So:
$ totem-video-thumbnailer --size 100x200 1.ogm 2.png
would have been the equivalent of this:
$ totem-video-thumbnailer --size 100 1.ogm 2.png

Also note that we never advertised supporting passing both the height and the width:
g_print ("Usage: %s [-s <size>] [-j] <input> <output> [backend options]\n", cmd);

I'm afraid you'll have to change your scripts.

Note You need to log in before you can comment on or make changes to this bug.