Patchwork legal-info: quote csv file contents, so commas can be used

login
register
mail settings
Submitter Arnout Vandecappelle
Date Nov. 13, 2012, noon
Message ID <1352808046-28127-1-git-send-email-arnout@mind.be>
Download mbox | patch
Permalink /patch/198635/
State Accepted
Commit f876cb598b78d31ca480e0f0ad0eb61d4e6c146f
Headers show

Comments

Arnout Vandecappelle - Nov. 13, 2012, noon
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

---
 package/pkg-utils.mk |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Peter Korsgaard - Nov. 14, 2012, 9:44 p.m.
>>>>> "Arnout" == Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> writes:

 Arnout> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

Committed, thanks.

Patch

diff --git a/package/pkg-utils.mk b/package/pkg-utils.mk
index 293bf4f..835c588 100644
--- a/package/pkg-utils.mk
+++ b/package/pkg-utils.mk
@@ -94,7 +94,7 @@  legal-warning-pkg=echo "WARNING: $(1): $(2)" >>$(LEGAL_WARNINGS)
 define legal-warning-pkg-savednothing # pkg, {local|override}
 	$(call legal-warning-pkg,$(1),sources and license files not saved ($(2) packages not handled))
 endef
-legal-manifest=echo "$(1),$(2),$(3),$(4),$(5)" >>$(LEGAL_MANIFEST_CSV)
+legal-manifest=echo '"$(1)","$(2)","$(3)","$(4)","$(5)"' >>$(LEGAL_MANIFEST_CSV)
 define legal-license-header
 	echo -e "$(LEGAL_INFO_SEPARATOR)\n\t$(1):" \
 		"$(2)\n$(LEGAL_INFO_SEPARATOR)\n\n" >>$(LEGAL_LICENSES_TXT)