commit c09efadfa97762fa7b8af599d0ec39105eb67aee
parent cca1b550a8cb0fb27dfde0258a91027d33503c9f
Author: Suraj N. Kurapati <sunaku@gmail.com>
Date: Sat, 16 Sep 2006 00:31:52 -0700
[project @ 56c9d78697afeaac5422f42e180ac43bf54f5a99]
[project @ 97]
revert back to Kernel#system
Diffstat:
2 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/HISTORY b/HISTORY
@@ -4,7 +4,8 @@ This is a history of major changes to {my wmii configuration}[http://people.ucsc
* Added Wmii.get_view and Wmii.get_client methods, to further minimize hard-coded IXP file system paths. This will make it easier to upgrade to wmii-4 later on.
-* Implemented Constantine Evans' idea to use fork/exec instead of system when launching programs.
+* Implemented Constantine Evans' idea to use fork/exec instead of Kernel#system when launching programs.
+** Reverted back because not all programs in the $PATH were executable via Kernel#exec.
= 2006-09-14
diff --git a/wmiirc-config.rb b/wmiirc-config.rb
@@ -18,8 +18,6 @@
=end
$: << File.dirname(__FILE__)
-require 'fs'
-require 'wm'
require 'rc'
FS = Wmii.fs
@@ -224,13 +222,13 @@ SHORTCUTS = {
# launch an internal action by choosing from a menu
"#{MENU_SEQ}i" => lambda do
action = show_menu(ACTION_MENU)
- fork {exec action} unless action.empty?
+ system(action << '&') unless action.empty?
end,
# launch an external program by choosing from a menu
"#{MENU_SEQ}e" => lambda do
program = show_menu(PROGRAM_MENU)
- fork {exec program} unless program.empty?
+ system(program << '&') unless program.empty?
end,
# focus any view by choosing from a menu