remove screen option, seems too hokey

This commit is contained in:
David Dollar
2010-07-19 17:11:42 -07:00
parent 51513dcb6d
commit 151ddb45c8
3 changed files with 0 additions and 20 deletions

View File

@@ -9,7 +9,6 @@ class Foreman::CLI < Thor
desc "start [PROCESS]", "Start the application, or a specific process"
method_option :screen, :type => :boolean, :aliases => "-s"
method_option :concurrency, :type => :string, :aliases => "-c",
:banner => '"alpha=5,bar=3"'
@@ -18,8 +17,6 @@ class Foreman::CLI < Thor
if process
engine.execute(process, options)
elsif options[:screen]
engine.screen
else
engine.start(options)
end

View File

@@ -58,19 +58,6 @@ class Foreman::Engine
watch_for_termination
end
def screen
tempfile = Tempfile.new("foreman")
tempfile.puts "sessionname foreman"
processes.each do |name, process|
tempfile.puts "screen -t #{name} #{process.command}"
end
tempfile.close
system "screen -c #{tempfile.path}"
tempfile.delete
end
def execute(name, options={})
processes(options[:concurrency]).values.select do |process|
process.name =~ /\A#{name}\.?\d*\Z/

View File

@@ -29,10 +29,6 @@ The following options control how the application is run:
Specify the number of each process type to run. The value passed in
should be in the format `process=num,process=num`
* `-s`, `--screen`:
Run the application as a series of screen windows rather than interleaved
in stdout.
## EXPORTING
`foreman export` is used to export your application to another process