Bug 195816 - [www/chromium] fix of handling of spaces in shell script
Summary: [www/chromium] fix of handling of spaces in shell script
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-chromium (Nobody)
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2014-12-09 03:19 UTC by Yuri Victorovich
Modified: 2015-07-24 05:48 UTC (History)
2 users (show)

See Also:

Attachments
patch (317 bytes, patch)
2014-12-09 03:19 UTC, Yuri Victorovich 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Yuri Victorovich freebsd_committer freebsd_triage 2014-12-09 03:19:39 UTC 
Created attachment 150377 [details]
patch

Current bin/chrome shell script messes up arguments when they contain spaces.
The way to pass all arguments with proper spaces in "$@" (and not $*)
Comment 1 Yuri Victorovich freebsd_committer freebsd_triage 2015-05-06 10:36:09 UTC 
Maintainer timeout.
Comment 2 Ed Maste freebsd_committer freebsd_triage 2015-06-15 13:22:41 UTC 
Patch looks good to me
Comment 3 commit-hook freebsd_committer freebsd_triage 2015-07-23 19:51:52 UTC 
A commit references this bug:

Author: rene
Date: Thu Jul 23 19:51:05 UTC 2015
New revision: 392749
URL: https://svnweb.freebsd.org/changeset/ports/392749

Log:
  Fix passing arguments containing spaces to the wrapper shell script.

  PR:		195816
  Submitted by:	yuri@rawbw.com
  MFH:		2015Q3

Changes:
  head/www/chromium/files/chrome.in
Comment 4 commit-hook freebsd_committer freebsd_triage 2015-07-24 05:48:12 UTC 
A commit references this bug:

Author: rene
Date: Fri Jul 24 05:47:48 UTC 2015
New revision: 392770
URL: https://svnweb.freebsd.org/changeset/ports/392770

Log:
  MFH: r392749

  Fix passing arguments containing spaces to the wrapper shell script.

  PR:		195816
  Submitted by:	yuri@rawbw.com
  Approved by:	ports-secteam (feld)

Changes:
_U  branches/2015Q3/
  branches/2015Q3/www/chromium/files/chrome.in