[Pidgin] #6476: Pidgin adds --display to the wm's session information, breaking remote terminals.

Pidgin trac at pidgin.im
Tue Aug 19 12:53:53 EDT 2008


#6476: Pidgin adds --display to the wm's session information, breaking remote
terminals.
----------------------+-----------------------------------------------------
 Reporter:  gdvieira  |        Owner:              
     Type:  defect    |       Status:  new         
Milestone:            |    Component:  pidgin (gtk)
  Version:  2.4.3     |   Resolution:              
 Keywords:            |  
----------------------+-----------------------------------------------------

Comment(by deryni):

 Given the patch that causes your current problem it would appear that at
 least one person has the expectation that windows restore to the display
 they were last on. Which may or may not be what your expectation is
 (assuming it doesn't fail when that display doesn't exist).

 You and the reporter of #3137 are likely going to be equally motivated to
 find the correct solution (in fact I'd argue you might even be more
 motivated as you said pidgin doesn't even start for you, whereas for them
 it does and just needs moving).

 But neither of those things is materially relevant to this discussion. And
 while it is all well and good to say "we don't fix bugs by introducing new
 problems" and that regression is always worse the question now is how long
 does a change have to have existed for it to stop being a regression. (The
 patch was in the 2.2.1 release which is just shy of 11 months old at this
 point.)

 Ideally someone who understands (and uses) session management stuff would
 find an application that tries to store display data (or find something
 that indicates applications shouldn't do that) and we would have a correct
 answer. Barring that, someone could (as I suggested in one of my previous
 comments) attempt to detect whether the display being requested actually
 exists and remove the argument when it doesn't (during an sm startup) in
 our getopt code.

-- 
Ticket URL: <http://developer.pidgin.im/ticket/6476#comment:5>
Pidgin <http://pidgin.im>
Pidgin


More information about the Tracker mailing list