Page MenuHomeFreeBSD

newvers.sh: Unbreak building in Git repositories.
ClosedPublic

Authored by 0mp on Sep 21 2018, 12:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 10, 9:50 PM
Unknown Object (File)
Wed, Nov 27, 2:55 AM
Unknown Object (File)
Oct 15 2024, 8:56 PM
Unknown Object (File)
Oct 2 2024, 8:08 AM
Unknown Object (File)
Sep 30 2024, 3:11 PM
Unknown Object (File)
Sep 30 2024, 2:35 AM
Unknown Object (File)
Sep 30 2024, 2:35 AM
Unknown Object (File)
Sep 30 2024, 2:35 AM

Details

Summary

Building the kernel in Git repositories when git-svn is not available and
the "help.autocorrect" Git parameter is enabled results in Git trying to
replace the "svn" command it does not know with "serve". As a result
the output of the "git server" command is appended to the values of the
environmental variables VERINFO, which causes the auto generated vers.c
file to contain invalid C syntax (missing newline escapes):

#define "@(#)FreeBSD 12.0-ALPHA7  r000eversion 2
0015agent=git/2.19.0
000cls-refs
0012fetch=shallow
0012server-option
0000=5e2272613fa(splash-vt)"
#define VERSTR "FreeBSD 12.0-ALPHA7  r000eversion 2
0015agent=git/2.19.0
000cls-refs
0012fetch=shallow
0012server-option
0000=5e2272613fa(splash-vt)\n"

Using -c help.autocorrect=0 seems to be a good solution as it does not
modify user's environment. I am not sure, however, if we should use
programs (or Git commands), which we are not sure exist (we never check if
git-svn is available on the host), as there may be more unexpected
behaviors like this one.

Sponsored by: Bally Wulff Games & Entertainment GmbH

Diff Detail

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

Event Timeline

0mp edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Sep 21 2018, 12:47 PM
0mp requested review of this revision.Sep 21 2018, 12:54 PM
0mp added reviewers: krion, mat.
0mp added subscribers: krion, mat.

@krion, @mat. May I have your approval? :)

This revision is now accepted and ready to land.Sep 21 2018, 12:55 PM
This revision was automatically updated to reflect the committed changes.