Page MenuHomeFreeBSD

crunchgen: quiet -Wmissing-prototypes
ClosedPublic

Authored by rlibby on Aug 23 2017, 7:54 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 22, 8:31 PM
Unknown Object (File)
Fri, Mar 22, 8:31 PM
Unknown Object (File)
Fri, Mar 22, 8:31 PM
Unknown Object (File)
Fri, Mar 22, 8:31 PM
Unknown Object (File)
Mar 9 2024, 7:32 AM
Unknown Object (File)
Feb 9 2024, 10:25 AM
Unknown Object (File)
Jan 8 2024, 4:53 AM
Unknown Object (File)
Jan 4 2024, 6:48 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.