[bug #55301] grog: doesn't honor GROFF_TYPESETTER

[G. Branden Robinson]

Update of bug #55301 (project groff):

                  Status:             In Progress => Fixed                  
             Open/Closed:                    Open => Closed                 
         Planned Release:                    None => 1.23.0                 

    _______________________________________________________

Follow-up Comment #4:


commit 182942f0453fc138147780b237492b8c50e514c8
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
Date:   Mon Jun 28 21:08:09 2021 +1000

    [grog]: Refactor device (`-T` option) handling.
    
    * src/utils/grog/grog.pl: Redeclare `device` as a scalar instead of a
      list.
    
      (process_arguments): Generalize handling of `-T` and `-m` options to
      permit optional whitespace.  Rename scalar `was_T` to
      `delayed_option`.
    
      (infer_device): Stop unconditionally adding a `-T ps` argument to the
      generated groff command.  Remove logic that validates any given `-T`
      option argument; instead, let groff fail if a bad one is supplied (its
      own diagnostics in this scenario were improved in commit 5a721a30, 27
      May).  This makes grog agnostic about any differently configured
      default device in groff itself, and about the GROFF_TYPESETTER
      environment variable.
    
      Fixes <https://savannah.gnu.org/bugs/?55301> by getting out of the
      way.
    
    * src/utils/grog/tests/PF-does-not-start-pic-region.sh:
    * src/utils/grog/tests/smoke-test.sh: Update expected output.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?55301>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/