Bug 121428 - [patch] x11/xlockmore: fix Mesa/GL modes
Summary: [patch] x11/xlockmore: fix Mesa/GL modes
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-03-06 15:10 UTC by Oliver Fromme
Modified: 2008-03-13 11:40 UTC (History)
1 user (show)

See Also:


Attachments
file.diff (396 bytes, patch)
2008-03-06 15:10 UTC, Oliver Fromme
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Fromme 2008-03-06 15:10:06 UTC
The options switch "Enable Mesa 3D (for GL modes)" is broken.
In fact it's broken in a very interesting way.  :-)

It turns out that these lines are the culprit:

    OPTIONS=MESAGL   "Enable Mesa 3D (for GL modes)"   off

    .if ${ARCH} == amd64
    WITH_MESAGL=    no
    .else
    WITH_MESAGL?=   yes
    .endif
    .if ${WITH_MESAGL} == yes
    CONFIGURE_ARGS+=        --with-mesa

First case:  options MESAGL is *disabled*.  Then the options
framework sets the variable WITHOUT_MESAGL=true.  The above
code (running on non-amd64) then sets WITH_MESAGL=yes anyway
enabling Mesa support even though the user didn't want it.

Second case:  options MESAGL is *enabled*.  As a result, the
variable WITH_MESAGL is set to "true" (not "yes"!).  The
above code only checks for the value "yes", therefore *not*
enabling Mesa support.

How-To-Repeat: 
On a FreeBSD/i386 machine:

cd /usr/ports/x11/xlockmore
make config
make

Notice that Mesa/GL modes are *not* included if you enabled
them in the config menu.  Notice that they *are* included
if you disabled them.
Comment 1 Martin Wilke freebsd_committer freebsd_triage 2008-03-13 11:33:52 UTC
State Changed
From-To: open->closed

Committed. Thanks!
Comment 2 dfilter service freebsd_committer freebsd_triage 2008-03-13 11:33:54 UTC
miwi        2008-03-13 11:33:44 UTC

  FreeBSD ports repository

  Modified files:
    x11/xlockmore        Makefile 
  Log:
  - Fix Mesa/GL modes
  
  PR:             121428
  Submitted by:   Oliver Fromme <olli@secnetix.de>
  
  Revision  Changes    Path
  1.109     +4 -4      ports/x11/xlockmore/Makefile
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"