After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 349177 - add a way to start and stop the record, without needing notification icon
add a way to start and stop the record, without needing notification icon
Status: RESOLVED FIXED
Product: istanbul
Classification: Other
Component: general
HEAD CVS
Other Linux
: Normal enhancement
: ---
Assigned To: Istanbul maintainers
Istanbul QA maintainers
Depends on:
Blocks:
 
 
Reported: 2006-07-29 15:00 UTC by Zaheer Abbas Merali
Modified: 2006-08-01 13:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
start record from command line (4.27 KB, patch)
2006-07-31 20:08 UTC, Frederic Peters
none Details | Review
start record from command line (8.63 KB, patch)
2006-08-01 09:41 UTC, Frederic Peters
committed Details | Review

Description Zaheer Abbas Merali 2006-07-29 15:00:26 UTC
Add a way to start and stop the recording without needing notification icon.

ie either command line based or through a dbus interface.
Comment 1 Marc-Andre Lureau 2006-07-30 16:24:09 UTC
Dbus interface or cli means that you are now talking of Istanbul as a service. 

Hence, it won't be easy to correct this if we want to be able to make a screencast of screencast. Because having 2 screencast at the same time means 2 services with different name. And thus, you are not talking of a unique service anymore... a service as only name (one access point)  

The other solution is to seperate the tray icon and the screencast objects:

- one "screencast instance" that provide a DBus service like this

(org.gnome.Istanbul)

    <method name="StartRecording">
      <arg type="d" name="frame_rate" direction="in"/>
      <arg type="d" name="scale_factor" direction="in"/>
      <arg type="b" name="record_3d" direction="in"/>
      <arg type="b" name="record_mousepointer" direction="in"/>
      <arg type="s" name="audio_source" direction="in"/> <-- the audio pipe description 
      <arg type="s" name="tmpfile" direction="out"/> <-- the tmp file, needed to identify the recording
    </method>

    <method name="StopRecording">
      <arg type="s" name="tmpfile" direction="in"/>
    </method>

- and the tray icon that communicates with it..

what do you think?
Comment 2 Frederic Peters 2006-07-31 20:08:43 UTC
Created attachment 70000 [details] [review]
start record from command line

This is not a D-Bus service, just a new cmdline option.

`istanbul --record FILE` will immediately start a screencast, recording to FILE, and won't add a tray icon.

The screencast can be stopped by SIGTERM, istanbul will then stop recording,
wait for end of stream and quit.

This is the most basic cmdline support you can get but it works fine (actually
I failed to watch produced video because bug 347761 but it is unrelated).

Also my main use case is running it from inside a Xvfb session and it doesn't
work in that case, the output file is never written to.  This may be a
limitation of Xvfb or istximagesrc, I don't know.  Also unrelated.
Comment 3 Zaheer Abbas Merali 2006-08-01 08:41:30 UTC
Frederic,

Should all the preferences that are gotten from gconf, be still gotten from gconf on a command line or should it be overridden by command line parameters.  Also would you like to be able to specify co-ordinates of the area on the screen to record?
Comment 4 Frederic Peters 2006-08-01 09:41:34 UTC
Created attachment 70018 [details] [review]
start record from command line

Modified patch to support setting coordinates and options from cmdline,
this gives:

  --record FILENAME (as before)
  --coords X1,Y1,X2,Y2
  --video-framerate DD
  --video-size [full,half,quarter]
  --record-3d (and --no-record-3d)
  --record-mousepointer (and --no-record-mousepointer)
  --record-sound (and --no-record-sound)

Also SIGINT can also be used to interrupt a screencast (SIGTERM is still supported).
Comment 5 Zaheer Abbas Merali 2006-08-01 13:31:29 UTC
Awesome patch, just tested it and it works well.

I may clean up the command line options later.  Thank you Frederic.