Bug 15896

Summary: monitor-probe-using-X is broken since X dropped support for its -probeonly option
Product: Mageia Reporter: Rémi Verschelde <rverschelde>
Component: RPM PackagesAssignee: Mageia tools maintainers <mageiatools>
Status: NEW --- QA Contact:
Severity: normal    
Priority: Normal CC: thierry.vignaud
Version: Cauldron   
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard: MGA5TOO
Source RPM: monitor-edid CVE:
Status comment:

Description Rémi Verschelde 2015-05-09 20:13:20 CEST
As discussed in this thread [1], monitor-probe-using-X is broken because X does not recognise the -probeonly option anyway (and as usual, gives somewhat verbose error messages...).

# monitor-probe-using-X intel
[...]
(EE) 
Fatal server error:
(EE) Unrecognized option: -probeonly
(EE) 
(EE) 
Please consult the The X.Org Foundation support 
         at http://bugs.mageia.org
 for help. 
(EE) 
X probe failed


It looks like upstream dropped support for -probeonly and the rest of the PCI-related code 6 years ago [2], so I guess monitor-probe-using-X has always been broken in Mageia.

I haven't studied the code, but judging by the script name I guess that monitor-probe-using-X is just obsolete nowadays, so we should probably drop it.

--
[1] https://ml.mageia.org/l/arc/dev/2015-05/msg00231.html
[2] http://lists.x.org/archives/xorg-devel/2009-July/001504.html

Reproducible: 

Steps to Reproduce:
Comment 1 Rémi Verschelde 2015-05-09 20:14:01 CEST
Note that the -probeonly switch is used here: http://gitweb.mageia.org/software/monitor-edid/tree/monitor-probe-using-X#n103
Comment 2 Mageia Robot 2015-05-09 20:17:45 CEST
commit c1e6181d4a72e1eaec4b636b51c2b44e5cedbd92
Author: Rémi Verschelde <rverschelde@...>
Date:   Sat May 9 20:16:17 2015 +0200

    Use monitor-edid directly instead of monitor-probe
    
    monitor-probe falls back to monitor-probe-using-X which is broken since X dropped its -probeonly switch (mga#15896)
    The logic used in mga-bg-res to parse the resolution was probably not adapted to the expected output from monitor-probe-using-X anyway.
---
 Commit Link:
   http://gitweb.mageia.org/software/design/theme/commit/?id=c1e6181d4a72e1eaec4b636b51c2b44e5cedbd92
Rémi Verschelde 2015-06-06 15:23:58 CEST

CC: (none) => thierry.vignaud

Samuel Verschelde 2015-06-06 17:08:32 CEST

Whiteboard: (none) => MGA5TOO

Rémi Verschelde 2016-01-13 11:49:22 CET

Blocks: (none) => 15527

Comment 3 Rémi Verschelde 2016-01-13 11:49:38 CET
Sorry, wrong bug.

Blocks: 15527 => (none)

Comment 4 Rémi Verschelde 2016-01-13 11:50:55 CET
@Thierry: I think we can obsolete monitor-probe-using-X?
Rémi Verschelde 2016-11-01 14:44:40 CET

Assignee: bugsquad => mageiatools