Page MenuHomeFreeBSD

Delete sys/dir.h which was deprecated since 1997.
ClosedPublic

Authored by ota_j.email.ne.jp on May 31 2019, 12:43 AM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 22 2024, 12:13 PM
Unknown Object (File)
Jan 12 2024, 10:01 AM
Unknown Object (File)
Dec 23 2023, 9:38 AM
Unknown Object (File)
Dec 22 2023, 11:57 AM
Unknown Object (File)
Dec 20 2023, 4:10 AM
Unknown Object (File)
Sep 10 2023, 7:20 PM
Unknown Object (File)
Sep 6 2023, 4:10 AM
Unknown Object (File)
Jul 22 2023, 5:18 AM
Subscribers

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

LGTM.

Should request an exp-run for completeness sake.

What do you mean by "an exp-run?" I had made buildworld and buildkernel successful on i386. Universe will be better but I don't have enough disk space.

Original PR suggest visit and fix all of ports. It's been over 20 years since taken out of POSIX mentioned PR, I hope most of ports are sys/dir.h clean.

What do you mean by "an exp-run?" I had made buildworld and buildkernel successful on i386. Universe will be better but I don't have enough disk space.

Original PR suggest visit and fix all of ports. It's been over 20 years since taken out of POSIX mentioned PR, I hope most of ports are sys/dir.h clean.

Exp-run info: https://www.freebsd.org/doc/en/articles/committers-guide/ports.html#ports-exp-run
For example https://bugs.freebsd.org/214864 is the exp-run I requested for lld.
I agree that there shouldn't be any ports using sys/dir.h still but it is still good to confirm that prior to the commit. (Here it's almost certainly been long enough that it's not necessary, but is a best practice.)

I'll ask in the #bsdports channel if anyone there thinks it's necessary.

Exp-run info: https://www.freebsd.org/doc/en/articles/committers-guide/ports.html#ports-exp-run
For example https://bugs.freebsd.org/214864 is the exp-run I requested for lld.
I agree that there shouldn't be any ports using sys/dir.h still but it is still good to confirm that prior to the commit. (Here it's almost certainly been long enough that it's not necessary, but is a best practice.)

I'll ask in the #bsdports channel if anyone there thinks it's necessary.

Thank you for explanations.

If FreeBSD foundation has resource to build all of ports, that's the best practice and approach.
Even in FreeBSD src, we can find lots of matches with <sys/dir.h>.
After grepping, I got worried.
The catch is all of them are behind ifdef HAS_DIRENT else cases or in README, configure, or config.h files.
All of them were false positive.

This revision is now accepted and ready to land.Aug 8 2019, 6:34 AM
This revision was automatically updated to reflect the committed changes.