[Notes] [Git][BuildStream/buildstream][master] 2 commits: Fix infinite recursion in default strip debug command

[Valentin David <gitlab mg gitlab com>]

Title: GitLab

Valentin David pushed to branch master at BuildStream / buildstream

Commits:

• e28f668a
  by Valentin David at 2018-11-08T09:34:11Z

  Fix infinite recursion in default strip debug command
  
  Fixes #645.
  

• 83d15350
  by Valentin David at 2018-11-08T10:01:39Z

  Merge branch 'valentindavid/fix-recursive-strip-debug' into 'master'
  
  Fix infinite recursion in default strip debug command
  
  Closes #645
  
  See merge request BuildStream/buildstream!918

4 changed files:

• buildstream/data/projectconfig.yaml
• tests/cachekey/project/elements/build1.expected
• tests/cachekey/project/elements/build2.expected
• tests/cachekey/project/target.expected

Changes:

buildstream/data/projectconfig.yaml

---

...	...	@@ -62,6 +62,11 @@ variables:
62	62	-o -name '*.cmxs' -o -name '*.node' ')' \
63	63	-exec sh -ec \
64	64	'read -n4 hdr <"$1" # check for elf header
	65	+ case "$1" in
	66	+ %{install-root}%{debugdir}/*)
	67	+ exit 0
	68	+ ;;
	69	+ esac
65	70	if [ "$hdr" != "$(printf \\x7fELF)" ]; then
66	71	exit 0
67	72	fi

tests/cachekey/project/elements/build1.expected

---

1		-dd5e29baefb84f68eb4abac3a1befc332077ec4c97bb2572e57f3ca98ba46707
		\ No newline at end of file
	1	+ce0ddf7126d45d14f5ec1a525337c39ec8ddbbe4b0ec2ef51bae777619ed39bb
		\ No newline at end of file

tests/cachekey/project/elements/build2.expected

---

1		-99d80454cce44645597c885800edf0bf254d1c3606d869f2ccdd5043ec7685cb
		\ No newline at end of file
	1	+5e2a48dbeae43f6bab84071dbd02345a3aa32a473c189645ab26f3d5d6cfe547
		\ No newline at end of file

tests/cachekey/project/target.expected

---

1		-29a1252ec30dd6ae73c772381f0eb417e3874c75710d08be819f5715dcaa942b
		\ No newline at end of file
	1	+125d9e7dcf4f49e5f80d85b7f144b43ed43186064afc2e596e57f26cce679cf5
		\ No newline at end of file

—
View it on GitLab.
You're receiving this email because of your account on gitlab.com. If you'd like to receive fewer emails, you can adjust your notification settings.