Bug #1171: ri: if "pager" exists, use it before trying less.

• Status: Closed
• Assignee: drbrain (Eric Hodel)
• Target version: -
• ruby -v: 1.9.1p0

[lucas (Lucas Nussbaum)]

=begin
Hi,

On Debian/Ubuntu, "pager" points to the preferred system pager.
It should be tried before "less".

This patch is used in the Debian package.

diff -urNad ruby1.9-1.9.0.1~/lib/rdoc/ri/display.rb ruby1.9-1.9.0.1/lib/rdoc/ri/display.rb
--- ruby1.9-1.9.0.1~/lib/rdoc/ri/display.rb 2008-01-31 15:48:35.000000000 +0900
+++ ruby1.9-1.9.0.1/lib/rdoc/ri/display.rb 2008-03-06 11:35:27.000000000 +0900
@@ -192,7 +192,7 @@

def setup_pager
  unless @use_stdout then

•  for pager in [ ENV['PAGER'], "less", "more", 'pager' ].compact.uniq
  

•  for pager in [ ENV['PAGER'], "pager", "less", "more" ].compact.uniq
     return IO.popen(pager, "w") rescue nil
   end
   @use_stdout = true
  

=end

[yugui (Yuki Sonoda)]

• Assignee set to akira (akira yamada)
• Priority changed from Normal to 3

=begin
It is a matter on the debian package. < akira
=end

[yugui (Yuki Sonoda)]

• Status changed from Open to Third Party's Issue

=begin

=end

[drbrain (Eric Hodel)]

• Assignee changed from akira (akira yamada) to drbrain (Eric Hodel)

=begin
Taking this, I think it is acceptable.
=end

[drbrain (Eric Hodel)]

=begin
I applied this patch to the RDoc source.
=end

[drbrain (Eric Hodel)]

• Status changed from Third Party's Issue to Closed

=begin
Now checked in
=end