wrong parameter to open startBristol process [patch]

Bug #660627 reported by Takashi Sakamoto
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
monobristol (Ubuntu)
Fix Released
Undecided
Stefan Ebner
Maverick
Fix Released
Undecided
Stefan Ebner

Bug Description

Binary package hint: monobristol

In MainWindow.cs, this code is used to start bristrol process with some parameters. In this case, one of parameter is "mini".

System.Diagnostics.Process.Start ("startBristol "+param+" -mini");

Then I got this error message below.

gvfs-open: file:///home/mocchi/test/hoge/startBristol%20-alsa-alsa%20-mini: error opening location: Error stating file '/home/mocchi/test/hoge/startBristol -alsa-alsa -mini': No such file or directory

To refer to this developer document, the sigle parameter show the file path, not path with parameters.

http://msdn.microsoft.com/en-us/library/53ezey2s%28v=VS.80%29.aspx

There is a need to give two parameters to start method, not one parameter.

http://msdn.microsoft.com/en-us/library/h6ak8zt5%28v=VS.80%29.aspx

So I change the code below. Here I define startBristol as "/usr/lib/startBristol" in advance.

System.Diagnostics.Process.Start (startBristol, param+" -mini");

Then I got no error.
Would you include this bug fix?

ProblemType: Bug
DistroRelease: Ubuntu 10.10
Package: monobristol 0.60.1-1
ProcVersionSignature: Ubuntu 2.6.35-22.34-generic 2.6.35.4
Uname: Linux 2.6.35-22-generic i686
Architecture: i386
Date: Fri Oct 15 01:02:21 2010
InstallationMedia: Ubuntu-Studio 10.10 "Maverick Meerkat" - Beta i386 (20100902.1)
PackageArchitecture: all
ProcEnviron:
 LANGUAGE=ja_JP:ja:en
 LANG=ja_JP.utf8
 SHELL=/bin/bash
SourcePackage: monobristol

Revision history for this message
Takashi Sakamoto (mocchi) wrote :
description: updated
tags: added: patch
Stefan Ebner (sebner)
Changed in monobristol (Ubuntu):
status: New → In Progress
Changed in monobristol (Ubuntu Maverick):
status: New → In Progress
assignee: nobody → Stefan Ebner (sebner)
Changed in monobristol (Ubuntu):
assignee: nobody → Stefan Ebner (sebner)
Revision history for this message
Stefan Ebner (sebner) wrote :

Uploaded fixed version to Natty (0.60.1-2)
Uploaded fixed version to maverick-proposed (0.60.1-2ubuntu0.1)

Changed in monobristol (Ubuntu):
status: In Progress → Confirmed
Changed in monobristol (Ubuntu Maverick):
status: In Progress → Confirmed
Stefan Ebner (sebner)
Changed in monobristol (Ubuntu):
status: Confirmed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

Stefan,

please use a lower version number than natty's for the SRU (like 2~maverick1). Also, please add the missing space in "LP: #xxx", so that the .changes will refer to this bug and it will autoclose. Thanks!

Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted monobristol into maverick-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in monobristol (Ubuntu Maverick):
status: Confirmed → Fix Committed
tags: added: verification-needed
Stefan Ebner (sebner)
Changed in monobristol (Ubuntu Maverick):
status: Fix Committed → 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.