Page MenuHomeFreeBSD

Final removal of WITH_NEW_XORG
ClosedPublic

Authored by dumbbell on Dec 22 2014, 9:21 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 18 2024, 10:57 AM
Unknown Object (File)
Feb 15 2024, 2:17 PM
Unknown Object (File)
Feb 13 2024, 3:55 PM
Unknown Object (File)
Jan 27 2024, 6:44 AM
Unknown Object (File)
Jan 24 2024, 7:55 AM
Unknown Object (File)
Jan 14 2024, 7:44 PM
Unknown Object (File)
Jan 14 2024, 7:43 PM
Unknown Object (File)
Dec 25 2023, 3:20 PM
Subscribers

Details

Summary

The knob was turned on by default more than two months ago. Since then, it was removed from several places. Now that xserver 1.14 is in the tree, this is the final removal.

While here, add WITH_NEW_XORG and WITH_KMS to bsd.sanity.mk to make people remove them from their /etc/make.conf.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

dumbbell retitled this revision from to Final removal of WITH_NEW_XORG.
dumbbell updated this object.
dumbbell edited the test plan for this revision. (Show Details)
dumbbell added a reviewer: kwm.
bapt requested changes to this revision.Dec 22 2014, 9:46 AM
bapt added a reviewer: bapt.
This revision now requires changes to proceed.Dec 22 2014, 9:46 AM
kwm requested changes to this revision.Dec 22 2014, 10:30 AM
kwm edited edge metadata.
kwm added inline comments.
x11-servers/xephyr/Makefile
27 ↗(On Diff #2809)

these lines that where between the .else .endif block needs to be removed too.

Should there be a ports/CHANGES entry because the switches went away?

I don't think a changes is needed

dumbbell edited edge metadata.

Cleanup x11-servers/xephyr/Makefile: only keep the correct Xephyr.1 man page.

In D1351#9, @kwm wrote:

Should there be a ports/CHANGES entry because the switches went away?

I'm not sure it's necessary, the knob is already ineffective/broken.

antoine added a reviewer: antoine.
antoine added a subscriber: antoine.

Looks good to me

kwm edited edge metadata.
bapt edited edge metadata.
This revision is now accepted and ready to land.Dec 22 2014, 8:13 PM
dumbbell updated this revision to Diff 2823.

Closed by commit rP375286 (authored by @dumbbell).