Page MenuHomeFreeBSD

depend-cleanup.sh: Simplify handling for stale syscall.S
ClosedPublic

Authored by jhb on Jul 24 2024, 4:33 PM.
Tags
None
Referenced Files
F93168140: D46103.id.diff
Sat, Sep 7, 8:08 PM
F93100160: D46103.id141317.diff
Sat, Sep 7, 8:27 AM
Unknown Object (File)
Thu, Sep 5, 12:26 AM
Unknown Object (File)
Fri, Aug 30, 12:50 PM
Unknown Object (File)
Fri, Aug 9, 4:13 PM
Unknown Object (File)
Aug 5 2024, 7:53 PM
Unknown Object (File)
Jul 31 2024, 4:04 AM
Unknown Object (File)
Jul 25 2024, 4:28 PM
Subscribers

Details

Summary

Remove the outer grep and depend on the greps in clean_dep instead.

Obtained from: CheriBSD

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb requested review of this revision.Jul 24 2024, 4:33 PM

This will remove a valid depends file for the lib32 builds on amd64 on each build. The issue I ran into was that when upgrading from a pre-libsys tree to a libsys tree on a arch with other compat arches, the other compat arches weren't cleaned and failed.

I was worried this would always force a rebuild of syscall.o.
But reading clean_dep I've convinced myself that this is a good change.
And reason to not also do it on i386? Why do we still need the MACHINE check?

This revision is now accepted and ready to land.Jul 24 2024, 5:02 PM
In D46103#1050896, @imp wrote:

I was worried this would always force a rebuild of syscall.o.
But reading clean_dep I've convinced myself that this is a good change.
And reason to not also do it on i386? Why do we still need the MACHINE check?

i386 still has syscall.S in its libc. Hence why lib32 on amd64 will also still have syscall.S.

We could decide that i386 doesn't need to be weird and drop the assembly. I didn't do it with the others because it is a modest optimization to call the intended syscall directly from userspace (vs other architectures where it was just a pointless extra file.)