Page MenuHomeFreeBSD

Fix newvers.sh with BUILD_WITH_STRICT_TMPPATH=1
ClosedPublic

Authored by arichardson on Nov 29 2018, 10:34 AM.
Tags
None
Referenced Files
F172997991: D18377.id.diff
Tue, Sep 22, 6:18 PM
Unknown Object (File)
Sun, Sep 13, 11:18 AM
Unknown Object (File)
Thu, Sep 10, 8:39 PM
Unknown Object (File)
Thu, Sep 10, 3:13 PM
Unknown Object (File)
Thu, Sep 10, 2:42 PM
Unknown Object (File)
Tue, Sep 8, 3:14 PM
Unknown Object (File)
Sun, Sep 6, 11:00 AM
Unknown Object (File)
Sun, Sep 6, 4:06 AM
Subscribers

Details

Summary

newvers.sh runs mkfifo which did not exist before this change.
However, I didn't notice before because it is run from a function
where a missing command does cause a noticeable failure.

Diff Detail

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

Event Timeline

emaste added subscribers: kib, markj.

LGTM. One of either @kib or @markj suggested eliminating the mkfifo but I believe we cannot without a more capable shell.

This revision is now accepted and ready to land.Nov 29 2018, 3:19 PM

Oh huh. What's the problem with:

$git_cmd ... | while read smode ...; do
...
done

? A quick test suggests that /bin/sh does indeed support that.

Anyway, LGTM.

Oh, maybe I'm conflating a different comment with something about mkfifo, but it was in D15968. In any case, the issue is that with cmd | while ... a return in the while loop returns from the subshell, not the function. There's a decent Stack Overflow discussion at https://stackoverflow.com/questions/13726764/while-loop-subshell-dilemma-in-bash

This revision was automatically updated to reflect the committed changes.