badly written mscore launcher script

Bug #207565 reported by orbisvicis
6
Affects Status Importance Assigned to Milestone
mscore (Ubuntu)
Fix Released
Undecided
Toby Smithe

Bug Description

Binary package hint: mscore

affects hardy packages
two issues:
a) if pulseaudio outputs to jackd, then the mscore launch script wont work because it suspends PA, bad idea. Instead check if the jack-sink module is loaded into PulseAudio.
b) script currently precludes any arguments.
*Solution (simple)
$MSCORE "$@";

Revision history for this message
Toby Smithe (tsmithe) wrote :

Yeah, this script was written in a bit of a tearing hurry when rushing to get the package ready for FeatureFreeze. Rewriting this script to make sure Pulse is only suspended in the worst case scenario has been on my ToDo list for a while.

Changed in mscore:
assignee: nobody → tsmithe
status: New → Confirmed
Revision history for this message
Toby Smithe (tsmithe) wrote :

I have attached an updated version of the script. Please test, and if successful, I will upload a fixed package.

Revision history for this message
Toby Smithe (tsmithe) wrote :

Please bear in mind that since 0.9.5-3ubuntu0, the PulseAudio jack plugins have been disabled (as a package in main cannot build-depend on a package in universe).

Changed in mscore:
status: Confirmed → In Progress
Revision history for this message
Toby Smithe (tsmithe) wrote :

Please test the newly attached script, which fixes a security flaw (via unPATHed "which" calls) and checks for pactl.

Revision history for this message
Daniel T Chen (crimsun) wrote :

(Sorry, was at work when you replied.)

11:08 crimsun > tsmithe: (RE: pastebin) I don't think it makes sense to invoke mscore.real if you can't suspend PA. Surely you meant to remove its invocation on line 20?

(WRT http://paste.ubuntu-nl.org/61417/)

Revision history for this message
Toby Smithe (tsmithe) wrote : Re: [Bug 207565] Re: badly written mscore launcher script

On Sat, Mar 29, 2008 at 3:10 PM, Daniel T Chen <email address hidden> wrote:
> (Sorry, was at work when you replied.)
>
> 11:08 crimsun > tsmithe: (RE: pastebin) I don't think it makes
> sense to invoke mscore.real if you can't suspend PA. Surely you meant
> to remove its invocation on line 20?

Nah, it does make sense: mScore will still run, and still accept MIDI
input; it just won't produce any sound.

Revision history for this message
orbisvicis (orbisvicis) wrote :

thanks for updating the script!
On the surface it works fine/as advertised, however:
$ mscore.real
Couldn't resolve property: 2

$ mscore
/usr/bin/mscore: 28: list: not found
Using JACK for audio
Couldn't resolve property: 2
JACK ERROR: cannot complete execution of the processing graph (Resource temporarily unavailable)
JACK ERROR: zombified - calling shutdown handler
JACK SHUTDOWN

(jackd didnt shut-down, it shutdown the mscore socket)

Now I have no idea why this happens. Like I said, mscore runs fine.

thanks

Revision history for this message
Toby Smithe (tsmithe) wrote :

Thanks for the confirmation, I'll update the package and forward the
JACK bug upstream when I get home.

On 3/31/08, orbisvicis <email address hidden> wrote:
> thanks for updating the script!
> On the surface it works fine/as advertised, however:
> $ mscore.real
> Couldn't resolve property: 2
>
> $ mscore
> /usr/bin/mscore: 28: list: not found
> Using JACK for audio
> Couldn't resolve property: 2
> JACK ERROR: cannot complete execution of the processing graph (Resource
> temporarily unavailable)
> JACK ERROR: zombified - calling shutdown handler
> JACK SHUTDOWN
>
> (jackd didnt shut-down, it shutdown the mscore socket)
>
> Now I have no idea why this happens. Like I said, mscore runs fine.
>
> thanks
>
> --
> badly written mscore launcher script
> https://bugs.launchpad.net/bugs/207565
> You received this bug notification because you are a direct subscriber
> of the bug.
>

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package mscore - 0.9.1d+dfsg-0ubuntu4

---------------
mscore (0.9.1d+dfsg-0ubuntu4) hardy; urgency=low

  * mscore.sh: + Check for module-jack-sink in PulseAudio, rather than
          just suspend blindly.
        + Clean up output.
        + Closes LP: #207565
  * Enable PortAudio support.

 -- Toby Smithe <email address hidden> Mon, 31 Mar 2008 22:44:12 +0100

Changed in mscore:
status: In Progress → Fix Released
Revision history for this message
Motin (motin) wrote :

@orbisvicis: Increase the timeout for jackd. I use 2000ms ( jackd ... -t2000 ...) which is enough to avoid at least for my setup.

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

Other bug subscribers

Bug attachments

Remote bug watches

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