Fails to mux files whose name or path contain spaces

Bug #1484397 reported by Carlos Maddela
This bug affects 1 person
Affects Status Importance Assigned to Milestone
bombono-dvd (Ubuntu)

Bug Description

Description: Ubuntu 15.04
Release: 15.04

  Installed: 1.2.2-0ubuntu4
  Candidate: 1.2.2-0ubuntu4
  Version table:
 *** 1.2.2-0ubuntu4 0
        500 vivid/universe amd64 Packages
        100 /var/lib/dpkg/status

Attempting to mux files whose name or path contain spaces will result in the following error dialog:

    Muxing error

    The reason is "exit code = 1" (see Details)

The textview dialog contains:

mplex -f 8 -o /tmp/Folder with Space/out 1.mpg /tmp/Folder with Space/out 1.m2v /tmp/Folder with Space/out 1.mp2
   INFO: [mplex] mplex version 2.1.0 (2.2.7 $Date: 2012/11/17 01:55:16 $)
**ERROR: [mplex] Unable to open file with for reading.

It is clear that the error occurs since the arguments are not quoted. I have attached a patch which fixes this.

Related branches

Revision history for this message
Carlos Maddela (e7appew) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "0002-Allow-muxing-of-files-with-spaces.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Changed in bombono-dvd (Ubuntu):
importance: Undecided → Low
Mathew Hodson (mhodson)
tags: added: vivid
Revision history for this message
Mathew Hodson (mhodson) wrote :

In order for this to be SRUed, it needs to be made compliant with (Please add a rational, test case, and regression potential to the description of this bug.)

Revision history for this message
Daniel Holbach (dholbach) wrote :

Maybe the patch could be forwarded here:

Revision history for this message
Brian Murray (brian-murray) wrote :

As I mentioned in the Wily merge proposal the escaping of %2% is inconsistent in the patch.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers