Change:
I don't know where the at the beginning of the target comes from, but probably from assumptions about sed version that are wrong. I had to update to gnu-sed 4.2.2 to build.
Also ootb on a Mac doesn't work with the rules specified (on el capitan):
does work
I take it back - updating sed did not appear to work. EDIT: I take that back, I had to clean after fixing paste.
most recent OSX version is UNIX certified, a POSIX requires as file argument when reading input from stdin. For , is only guaranteed to work in GNU sed, you need to insert literal newline and tab character there for portability.
Can someone with a mac send a patch? I don't have a mac...
It's update and every should end in instead of having no arguments. , Tim Hockin wrote:
(user should update sed, make needs changes to paste) , Clayton Coleman wrote:
I think we should stick to portable way for , it's too risky for relying on GNUism. If you don't mind, I will make a PR.
Sure, thanks , Cuong Manh Le wrote:
Are you sure the worked? I pull the latest from master, then run the same command above, I got: File is empty.
Make sure you are in your GOPATH in the correct spot for Kube On Jul 14, 2016, at 2:47 AM, Cuong Manh Le wrote: Are you sure the Makefile worked? I pull the latest from master, then run the same command above, I got: F0714 13:36:50. ] Error: Failed making a parser: unable to add directory "": unable to import "": expected 'package', found 'EOF' make: * [Makefile.generatedfiles:201: gendeepcopy] Error 255 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub , or mute the thread .
If you get the "expected 'package', found 'EOF'" message it is usually because something somewhere failed, and left an empty .go file. make cleangenerated , Cuong Manh Le wrote:
From BSD sed man page: Maybe try that?
It's not about ERE ( enables it), it's about expanding escape sequences in RHS of search and replace .

Discussion:
gen_deepcopy: $(DEEPCOPY_FILES) if [[ -f $(META_DIR)/$(DEEPCOPY_GEN).todo ]]; then                 $(DEEPCOPY_GEN)                                                <del> -i $$(cat $(META_DIR)/$(DEEPCOPY_GEN).todo | paste -sd,)   </del> <ins> -i $$(cat $(META_DIR)/$(DEEPCOPY_GEN).todo | paste -sd, -)   </ins> --bounding-dirs $(PRJ_SRC_PATH)                            -O $(DEEPCOPY_BASENAME);                                   fi

./cmd/libs/go2idl/deepcopy-gen                     | grep "^$(PRJ_SRC_PATH)"                          | xargs go list -e -f '{{.Dir}} {{.Dir}}/*.go');   <del> echo $$DIRECT $$INDIRECT | sed 's/ / nt/g';        </del> <ins> echo $$DIRECT $$INDIRECT | sed 's/ / =,/g' | tr '=,' 'nt';  </ins> ) | sed "s|$$(pwd -P)/||" > $@.tmp;                     cmp -s $@.tmp $@ || cat $@.tmp > $@ && rm -f $@.tmp

gen_conversion: $(CONVERSION_FILES) if [[ -f $(META_DIR)/$(CONVERSION_GEN).todo ]]; then                 $(CONVERSION_GEN)                                                <del> -i $$(cat $(META_DIR)/$(CONVERSION_GEN).todo | paste -sd,)   </del> <ins> -i $$(cat $(META_DIR)/$(CONVERSION_GEN).todo | paste -sd, -)   </ins> -O $(CONVERSION_BASENAME);                                   fi

./cmd/libs/go2idl/conversion-gen                   | grep "^$(PRJ_SRC_PATH)"                          | xargs go list -e -f '{{.Dir}} {{.Dir}}/*.go');   <del> echo $$DIRECT $$INDIRECT | sed 's/ / nt/g';        </del> <ins> echo $$DIRECT $$INDIRECT | sed 's/ / =,/g' | tr '=,' 'nt';  </ins> ) | sed "s|$$(pwd -P)/||" > $@.tmp;                     cmp -s $@.tmp $@ || cat $@.tmp > $@ && rm -f $@.tmp