Page MenuHomeFreeBSD

crunchgen: quiet -Wmissing-prototypes
ClosedPublic

Authored by rlibby on Aug 23 2017, 7:54 AM.
Tags
None
Referenced Files
Unknown Object (File)
May 6 2026, 2:27 AM
Unknown Object (File)
May 3 2026, 7:05 PM
Unknown Object (File)
Apr 30 2026, 5:10 PM
Unknown Object (File)
Apr 30 2026, 7:09 AM
Unknown Object (File)
Apr 29 2026, 7:50 AM
Unknown Object (File)
Apr 22 2026, 5:20 AM
Unknown Object (File)
Apr 20 2026, 12:44 AM
Unknown Object (File)
Apr 19 2026, 1:49 AM
Subscribers

Details

Test Plan
buildworld; installworld
/rescue/rescue (usage)
/rescue/rescue foo (usage)
/rescue/rescue ls

Diff Detail

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

Event Timeline

This revision is now accepted and ready to land.Aug 26 2017, 3:04 PM
usr.sbin/crunch/crunchgen/crunchgen.c
1108 ↗(On Diff #32333)

Whoops, that extra "\n" led to the following makefile template, from strings $(which crunchgen):

%s_stub.c:
        echo "extern int main(int argc, char **argv, char **envp);
int _crunched_%s_stub(int argc, char **argv, char **envp){return main(argc,argv,envp);}" >%s_stub.c
%s.lo: %s_stub.o $(%s_OBJPATHS)

I didn't notice earlier, apparently the build is using the installed crunchgen (rather than the one being built)?

New version replaces the line feed with a space and gives this from
strings /usr/obj/usr/src/freebsd/usr.sbin/crunch/crunchgen/crunchgen:

%s_stub.c:
        echo "extern int main(int argc, char **argv, char **envp); int _crunched_%s_stub(int argc, char **argv, char **envp){return main(argc,argv,envp);}" >%s_stub.c
%s.lo: %s_stub.o $(%s_OBJPATHS)
rlibby edited edge metadata.

Keep generated stub on one line.

This revision now requires review to proceed.Aug 26 2017, 11:24 PM
This revision was automatically updated to reflect the committed changes.