Bug 448209 - totem-video-thumbnailer do not accept arguments alike " --size XXXxYYY"
Summary: totem-video-thumbnailer do not accept arguments alike " --size XXXxYYY"
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: totem
Version: 9
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Bastien Nocera
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-05-24 13:22 UTC by Miao ZhiCheng
Modified: 2008-05-24 13:51 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-05-24 13:51:38 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Miao ZhiCheng 2008-05-24 13:22:56 UTC
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 13:51:38 UTC
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.