- Convert bsd.sdl.mk into Uses feature
- Simplify code a bit
Details
Details
- Reviewers
- None
- Group Reviewers
portmgr O5: Ports Framework (Owns No Changed Paths) - Commits
- rP492696: Convert bsd.sdl.mk to Uses/sdl.mk
Exp-run is needed
Diff Detail
Diff Detail
- Repository
- rP FreeBSD ports repository
- Lint
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 22365 Build 21537: arc lint + arc unit
Event Timeline
Mk/Uses/sdl.mk | ||
---|---|---|
78 | Agreed, already working on this. |
Mk/bsd.port.mk | ||
---|---|---|
1426–1429 | Yep, I'd prefer to convert ports gradually like it's done for USE_GNOME and USE_GL. Number of USE_SDL ports is comparable to USE_GL ports. |
Mk/bsd.port.mk | ||
---|---|---|
1426–1429 | Well, thing is, having it done gradually is not the nice thing to do, because you end up doing half the job and asking the other ports to finish it :-) Technically, the way it was done for USE_GNOME or USE_GL was ok, but it was humanly wrong. |
Comment Actions
Could you replace all the tests doing:
.if ${foo:Mbar} != ""
with a much simpler:
.if ${foo:Mbar}
and the:
.if ${foo:Mbar} == ""
with:
.if empty(foo:Mbar)
The time when our make required these constructions is long gone.
Comment Actions
@mat sorry, I though you've marked this as accepted. Exp-run was OK anyway: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=235561