Skip to content

Commit

Permalink
remove screen option, seems too hokey
Browse files Browse the repository at this point in the history
  • Loading branch information
ddollar committed Jul 20, 2010
1 parent 51513dc commit 151ddb4
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 20 deletions.
3 changes: 0 additions & 3 deletions lib/foreman/cli.rb
Expand Up @@ -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"'

Expand All @@ -18,8 +17,6 @@ def start(process=nil)

if process
engine.execute(process, options)
elsif options[:screen]
engine.screen
else
engine.start(options)
end
Expand Down
13 changes: 0 additions & 13 deletions lib/foreman/engine.rb
Expand Up @@ -58,19 +58,6 @@ def start(options={})
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/
Expand Down
4 changes: 0 additions & 4 deletions man/foreman.1.ronn
Expand Up @@ -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
Expand Down

0 comments on commit 151ddb4

Please sign in to comment.