me-tv fails to store recordings in a FAT32 partition

Bug #491602 reported by Marco
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Me TV
Fix Released
Low
Unassigned
1.1
Fix Released
Low
Unassigned
me-tv (Debian)
Fix Released
Undecided
Unassigned
me-tv (Ubuntu)
Fix Released
Undecided
Unassigned
Nominated for Lucid by Michael Lamothe

Bug Description

Binary package hint: me-tv

Package: me-tv 1.0.0-1ubuntu1
PackageArchitecture: i386
Uname: Linux 2.6.31-15-generic i686
Architecture: i386
DistroRelease: Ubuntu 9.10

me-tv appends date and time to the filename of the recorded video. The presence of the colon character in the timestamp is a problem if the file is stored in a fat32 partition. In this case, me-tv gives a "invalid argument" error message.

Even if I am *not* a glibc/c++ expert programmer, I've modified a little the code of me-tv so that any ":" would be changed to "_".
My patch is in the attachment.

Revision history for this message
Marco (flare) wrote :
Marco (flare)
description: updated
Revision history for this message
Brian Murray (brian-murray) wrote :

Thank you for providing a patch for this bug report. This particular package is synchronized with Debian so it would be quite helpful if you were to forward the patch to the Debian bug tracker. You can learn more about how to use the Debian bug tracker at https://wiki.ubuntu.com/Debian/Bugs. After you've forwarded the bug report and patch please add a bug watch, following the procedure at http://wiki.ubuntu.com/Bugs/Watches, so we can track the progress of the upstream bug report.

Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

Thanks Moire, I like the simplicity of your solution and have made it a configurable option in the preferences dialog. Right now, this is targeted for Me TV 1.1.0 due for release 1-1-2010.

Changed in me-tv:
status: New → Confirmed
importance: Undecided → Low
assignee: nobody → Michael Lamothe (lamothe)
Changed in me-tv (Ubuntu):
status: New → Confirmed
Changed in debian:
status: New → Confirmed
Revision history for this message
Michael Lamothe (lamothe-deactivatedaccount-deactivatedaccount) wrote :

Fix committed in revision 628 of lp:me-tv

Revision history for this message
Julian Andres Klode (juliank) wrote :

1.0.2-1 uploaded to Debian. Once it enters testing, it will be synced to lucid automatically.

affects: debian → me-tv (Debian)
Changed in me-tv (Debian):
status: Confirmed → Fix Released
Revision history for this message
Felix Geyer (debfx) wrote :

Marking bug as Fix Released as version 1.1.2-1 has been synced to Ubuntu Lucid.

Changed in me-tv (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.