You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tephra.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/09/26 18:57:20 UTC
[jira] [Commented] (TEPHRA-186) Prefix function and variable names
to reduce shell namespace collisions
[ https://issues.apache.org/jira/browse/TEPHRA-186?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15523886#comment-15523886 ]
ASF GitHub Bot commented on TEPHRA-186:
---------------------------------------
GitHub user wolf31o2 opened a pull request:
https://github.com/apache/incubator-tephra/pull/15
TEPHRA-186 Prefix shell variables and functions
Prefix the names of shell variables and functions with TEPHRA_ and tephra_, respectively. This prevents any potential namespace collisions in the shell environment.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/wolf31o2/incubator-tephra TEPHRA-186-prefix-shell-script
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-tephra/pull/15.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #15
----
commit fb927907b261ad569aa122679d6c656d3e5af703
Author: Chris Gianelloni <ch...@cask.co>
Date: 2016-09-26T18:30:47Z
TEPHRA-186 Prefix shell variables and functions
----
> Prefix function and variable names to reduce shell namespace collisions
> -----------------------------------------------------------------------
>
> Key: TEPHRA-186
> URL: https://issues.apache.org/jira/browse/TEPHRA-186
> Project: Tephra
> Issue Type: Improvement
> Reporter: Chris Gianelloni
> Assignee: Poorna Chandra
> Original Estimate: 2h
> Remaining Estimate: 2h
>
> The tephra shell script should use variables prefixed with TEPHRA_ and functions should be prefixed, where possible, to prevent namespace collisions in the shell environment.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)