Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(4964)

Issue 13254044: acme: execute commands containing '/'s through shell

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 7 months ago by marius.eriksen
Modified:
10 years, 7 months ago
Reviewers:
rsc
CC:
rsc, codebot
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M src/cmd/acme/exec.c View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4
marius.eriksen
10 years, 7 months ago (2013-09-02 18:07:03 UTC) #1
marius.eriksen
This allows ‘subdirectory’ commands to be executed from acme.
10 years, 7 months ago (2013-09-02 18:08:28 UTC) #2
rsc
LGTM
10 years, 7 months ago (2013-09-06 20:22:50 UTC) #3
rsc
10 years, 7 months ago (2013-09-06 20:23:51 UTC) #4
*** Submitted as
https://code.google.com/p/plan9port/source/detail?r=219380a98a91 ***

acme: execute commands with / using shell

This allows commands in bin subdirectories.

R=rsc
CC=plan9port.codebot
https://codereview.appspot.com/13254044

Committer: Russ Cox <rsc@swtch.com>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b