Your message dated Thu, 12 May 2011 11:32:18 +0000
with message-id <e1qku7u-0002rq...@franck.debian.org>
and subject line Bug#626481: fixed in ladish 0.3-5
has caused the Debian Bug report #626481,
regarding ladish: can't use rxvt / others as terminal emulators, shell not 
respected
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
626481: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=626481
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: ladish
Version: 0.3-4
Severity: normal
Tags: patch


Ladish tries to start terminal-bound commands as <terminal> -e 'command 
args'. In general, "x-terminal-emulator -e" expects an executable + 
arguments, NOT a command string, as daemon/loader.c provides. The attached 
(if trivial) patch fixes that.

More generally, this fix has the benefit of telling the terminal emulator 
which shell to use (the one configured in ladish), as opposed to letting it 
decide on its own.


-- System Information:
Debian Release: squeeze/sid
  APT prefers lucid-updates
  APT policy: (500, 'lucid-updates'), (500, 'lucid-security'), (500, 
'lucid-proposed'), (500, 'lucid-backports'), (500, 'lucid')
Architecture: i386 (i686)

Kernel: Linux 2.6.38-5.dmz.1-liquorix-686 (SMP w/2 CPU cores; PREEMPT)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages ladish depends on:
ii  la 1:1.0~rc2+20100724.gitf4d4a2-1~lucid2 set of tools to control and monito
ii  li 2.3.20-2build1                        Library of functions for 2D graphi
ii  li 1.30.0-0ubuntu2.1                     The ATK accessibility toolkit
ii  li 2.11.1-0ubuntu7.8                     Embedded GNU C Library: Shared lib
ii  li 1.8.10-2ubuntu1                       The Cairo 2D vector graphics libra
ii  li 1.8.4-0ubuntu1                        C++ wrappers for Cairo (shared lib
ii  li 1.2.16-2ubuntu4.2                     simple interprocess messaging syst
ii  li 0.84-1                                simple interprocess messaging syst
ii  li 2.0.1-7ubuntu1                        XML parsing C library - runtime li
ii  li 0.7.1-0ubuntu1~lucid1                 interactive widget for “boxes and 
ii  li 2.8.0-2ubuntu1                        generic font configuration library
ii  li 2.3.11-1ubuntu2.4                     FreeType 2 font engine, shared lib
ii  li 1:4.5.1-7ubuntu2danmppa1~lucid1       GCC support library
ii  li 2.24.1-0ubuntu1                       The GLib library of C routines
ii  li 2.24.2-0ubuntu1                       C++ wrapper for the GLib toolkit (
ii  li 2.30.1-0ubuntu1                       A powerful object-oriented display
ii  li 2.26.0-1build1                        C++ wrappers for libgnomecanvas2 (
ii  li 2.20.1-0ubuntu2                       The GTK+ graphical user interface 
ii  li 1:2.20.3-0ubuntu1                     C++ wrappers for GTK+ (shared libr
ii  li 0.120                                 Dummy package to quench libjack0 d
ii  li 1.28.0-0ubuntu2.2                     Layout and rendering of internatio
ii  li 2.26.2-0ubuntu1                       C++ Wrapper for pango (shared libr
ii  li 2.2.4.2-1                             type-safe Signal Framework for C++
ii  li 4.5.1-7ubuntu2danmppa1~lucid1         The GNU Standard C++ Library v3
ii  li 2.17.2-0ubuntu1.10.04.2               Universally Unique ID library

ladish recommends no packages.

ladish suggests no packages.

-- no debconf information
>From 5e07f29e9d863608581093313a4a780d38f1a3db Mon Sep 17 00:00:00 2001
From: Dan A. Muresan <danm...@gmail.com>
Date: Sat, 7 May 2011 14:01:14 +0300
Subject: [PATCH] Fix in-terminal execution for non-xterm
Forwarded: http://ladish.org/ticket/179

---
 daemon/loader.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/daemon/loader.c b/daemon/loader.c
index ad9b983..d8c7fa1 100644
--- a/daemon/loader.c
+++ b/daemon/loader.c
@@ -317,7 +317,7 @@ loader_exec_program(
     argv[i++] = "-e";
   }
 
-  if (!run_in_terminal || strchr(commandline, '$') != NULL)
+  if (!run_in_terminal || 1 /* -e expects an executable not a shell command */)
   {
     if (!conf_get(LADISH_CONF_KEY_DAEMON_SHELL, argv + i))
     {
-- 
1.7.0.4


--- End Message ---
--- Begin Message ---
Source: ladish
Source-Version: 0.3-5

We believe that the bug you reported is fixed in the latest version of
ladish, which is due to be installed in the Debian FTP archive:

gladish_0.3-5_amd64.deb
  to main/l/ladish/gladish_0.3-5_amd64.deb
ladish-dbg_0.3-5_amd64.deb
  to main/l/ladish/ladish-dbg_0.3-5_amd64.deb
ladish_0.3-5.debian.tar.gz
  to main/l/ladish/ladish_0.3-5.debian.tar.gz
ladish_0.3-5.dsc
  to main/l/ladish/ladish_0.3-5.dsc
ladish_0.3-5_amd64.deb
  to main/l/ladish/ladish_0.3-5_amd64.deb
liblash-compat-1_0.3-5_amd64.deb
  to main/l/ladish/liblash-compat-1_0.3-5_amd64.deb
liblash-compat-dev_0.3-5_all.deb
  to main/l/ladish/liblash-compat-dev_0.3-5_all.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 626...@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Alessio Treglia <ales...@debian.org> (supplier of updated ladish package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmas...@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Thu, 12 May 2011 13:05:39 +0200
Source: ladish
Binary: ladish gladish liblash-compat-1 liblash-compat-dev ladish-dbg
Architecture: source amd64 all
Version: 0.3-5
Distribution: experimental
Urgency: low
Maintainer: Debian Multimedia Maintainers 
<pkg-multimedia-maintainers@lists.alioth.debian.org>
Changed-By: Alessio Treglia <ales...@debian.org>
Description: 
 gladish    - graphical interface for LADI Session Handler
 ladish     - session management system for JACK applications
 ladish-dbg - Debugging symbols for ladish
 liblash-compat-1 - LASH compatibility library
 liblash-compat-dev - LASH compatibility library (development files)
Closes: 626481
Changes: 
 ladish (0.3-5) experimental; urgency=low
 .
   * Fix in-terminal commands execution (Closes: #626481).
     - Ladish tries to start commands as $terminal -e "$command $argument"
       but x-terminal-emulator generally expects an executable + arguments,
       not a command string. Plus, this tells the terminal emulator which
       shell to use, rather than letting it decide on its own.
       Patch provided by Dan Muresan.
Checksums-Sha1: 
 e5bc9cb554f354d94977c502b6964a282f470175 1518 ladish_0.3-5.dsc
 0ac9ac3e191e47175b1980e9b5df324d41902b0c 11355 ladish_0.3-5.debian.tar.gz
 c73570dd5986197878ca313a0ebdc92bd72a9ec4 179446 ladish_0.3-5_amd64.deb
 f58516147eff7b42b6f840f2ea51d0e60df682fc 155674 gladish_0.3-5_amd64.deb
 bb9e5790fb0135fc0deffceab78799bb4aa2751c 14032 liblash-compat-1_0.3-5_amd64.deb
 20b7cf7675f9f1c4d6b3351a3ba5d05929bcf86f 10458 liblash-compat-dev_0.3-5_all.deb
 85b3e8e2bfd7f0472deec13093d10ec84c856cae 185138 ladish-dbg_0.3-5_amd64.deb
Checksums-Sha256: 
 8f9a1ba60ee6f1729efeb9fade505ad755854d726d81627b8b2cceb937494c77 1518 
ladish_0.3-5.dsc
 e14df7885d380944a4b6395a5d27fdd6a76fa54643b8a76a8bd9f669aff06ddd 11355 
ladish_0.3-5.debian.tar.gz
 1c7fc902de4a593183d697d5212018e8fb8f9ed3e1607c0992a88db6f40db2bf 179446 
ladish_0.3-5_amd64.deb
 b9481311d1e665ed596d911c611e9b2fc0605cff88d1fffa8d11adc92fba0ff2 155674 
gladish_0.3-5_amd64.deb
 bd24b7db70e2e22a013b634a331f40adf3d3d3de5def292b8c09dd18e03d4424 14032 
liblash-compat-1_0.3-5_amd64.deb
 cc5fb3c64da3a7b24d7a3596f6774be3b43e0b20901516f725589bab548edad1 10458 
liblash-compat-dev_0.3-5_all.deb
 ea3a08129a2e2ada8a82f2afeb93cea1a59b6458f9dc7761a05b5c28378ed8b9 185138 
ladish-dbg_0.3-5_amd64.deb
Files: 
 b0ecac1b73d6793d78421eacfd144544 1518 sound optional ladish_0.3-5.dsc
 d5251e018494965c6862a231ad10d8d6 11355 sound optional 
ladish_0.3-5.debian.tar.gz
 39b444f2d61c964dc7dbbabe910c1aa5 179446 sound optional ladish_0.3-5_amd64.deb
 a74d580ed00b8cdb16e7609c68a97dda 155674 sound optional gladish_0.3-5_amd64.deb
 513bf338de71512b8ed8cdf9cb9d25a3 14032 libs optional 
liblash-compat-1_0.3-5_amd64.deb
 cd1e55eab65084177851338565e5e630 10458 libdevel optional 
liblash-compat-dev_0.3-5_all.deb
 fde00d6ccc2af2ea7d719052ce323cc1 185138 debug extra ladish-dbg_0.3-5_amd64.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)

iEYEARECAAYFAk3LwnQACgkQRdSMfNz8P9BywgCghdA4oZkltma+6ghOBiQQH4X9
e/0AniPrNi78ASQjr2KNBaeVmRrWlPI1
=Y5kC
-----END PGP SIGNATURE-----



--- End Message ---
_______________________________________________
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers

Reply via email to