View Ticket
Ticket Hash: 47edc4cb21e0add7eb7c78fac02ca1ff6ab01473
Title: Avoid unnecessary temporary files at build time
Status: Closed Type: Build Problem
Severity: Minor Priority: Immediate
Subsystem: Resolution: Rejected
Last Modified: 2017-01-17 15:44:50
Version Found In:
User Comments:
anonymous added on 2017-01-17 11:00:04:
I suggest the following change, to avoid unnecessary temporary files at build time.

Index: Makefile.in
==================================================================
--- Makefile.in
+++ Makefile.in
@@ -36,14 +36,12 @@

 # Create a C-source-ified version of the script resources
 # for TclTLS so that we only need a single file to enable
 # this extension
 tls.tcl.h: @srcdir@/tls.tcl Makefile
-       od -A n -v -t xC < '@srcdir@/tls.tcl' > tls.tcl.h.new.1
-       sed 's@  *@@g;s@..@0x&, @g' < tls.tcl.h.new.1 > tls.tcl.h.new.2
-       rm -f tls.tcl.h.new.1
-       mv tls.tcl.h.new.2 tls.tcl.h
+       od -A n -v -t xC < '@srcdir@/tls.tcl' | \
+       sed 's@  *@@g;s@..@0x&, @g' > tls.tcl.h

 # Create default DH parameters
 dh_params.h: @srcdir@/gen_dh_params Makefile
        @srcdir@/gen_dh_params @GEN_DH_PARAMS_ARGS@ > dh_params.h.new
        mv dh_params.h.new dh_params.h

rkeene added on 2017-01-17 14:42:59:

The reason it's done this way is because the exit status is only of the last command in the pipeline. Your solution will only fail if 'sed' fails, not if 'od' fails. The result will appear to be successful but be a zero byte file.