Page MenuHomeFreeBSD

Check for USE_XORG being non-empty instead of just defined
AbandonedPublic

Authored by AMDmi3 on Dec 27 2019, 11:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 25, 5:47 AM
Unknown Object (File)
Wed, Mar 18, 1:59 PM
Unknown Object (File)
Tue, Mar 17, 11:40 PM
Unknown Object (File)
Tue, Mar 17, 7:03 AM
Unknown Object (File)
Thu, Mar 12, 12:52 PM
Unknown Object (File)
Wed, Mar 11, 7:06 PM
Unknown Object (File)
Wed, Mar 11, 3:41 AM
Unknown Object (File)
Sun, Mar 8, 11:55 PM
Subscribers

Details

Reviewers
bapt
Group Reviewers
O5: Ports Framework(Owns No Changed Paths)
portmgr
Summary

The problem is that USE_XORG may be defined by other Uses modules, such as gl, but still empty.

For example

USES=gl
USE_GL=glut

leads to USE_XORG set via (in Uses/gl.mk):

USE_XORG+=  ${_GL_${_component}_USE_XORG}

to empty value, as _GL_glut_USE_XORG is not defined

This reproduces on graphics/devil when X11 option is off.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 28341
Build 26438: arc lint + arc unit

Event Timeline

I am off my computer to really check but at quick glance it should probably be if defined and !empty

In D22939#502604, @bapt wrote:

I am off my computer to really check but at quick glance it should probably be if defined and !empty

Why, wouldn't it be redundant?

back on it almost 3 years later... sorry about that, after rereading, I think your proposal is ok, it would require at least an exp-run, but looks good to me.
Note I haven't tested it!!!

This revision is now accepted and ready to land.Jul 18 2022, 12:17 PM