hostapd is removed for now because it's a little less trivial with
recent wpa architecture. The rest is adopting to ncurses and ifconfig
changes, as well as a switch in PIE default.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG FreeBSD src repository
- Lint
Lint Skipped - Unit
Tests Skipped - Build Status
Buildable 54325 Build 51215: arc lint + arc unit
Event Timeline
| tools/bsdbox/Makefile | ||
|---|---|---|
| 75 | Why not -lifconfig like the others? | |
| tools/bsdbox/Makefile | ||
|---|---|---|
| 75 | I was just following what we did in rescue here, but I'm pretty sure it's because libifconfig is an INTERNALLIB so it doesn't end up in the place you'd expect in the sysroot we build against | |
Comment Actions
This looks fine... Did I see you had hostap added back somewhere else?
| tools/bsdbox/Makefile | ||
|---|---|---|
| 7 | Why no pie? Is a restriction of crunchgen? If so it should be in the mn page (separate commit is fine). | |
| 75 | Funny... I almost added 'because it's an internal lib' to the end of my comment, but didn't want to lead the witness :) | |
Comment Actions
Yeah, one of the later changes brings it back. I've already merged the crunchgen change, so I might just squash these two together.
| tools/bsdbox/Makefile | ||
|---|---|---|
| 7 | I'm not sure anymore. I had added it as a response to a build error, but I'm starting to think the build error was that I hadn't built the prerequisite libs first in that .OBJDIR- for a little bit I had forgotten that crunchgen will only rebuild the PROGS, not {SH,}LIBS. I'll axe it. | |