[PATCH 6/9] dim: prefix global variables for internal configuration with dim_
Jani Nikula
jani.nikula at intel.com
Thu Mar 29 08:45:58 UTC 2018
Simple renames, amend comments to clarify global variable naming style.
Signed-off-by: Jani Nikula <jani.nikula at intel.com>
---
dim | 50 +++++++++++++++++++++++++-------------------------
1 file changed, 25 insertions(+), 25 deletions(-)
diff --git a/dim b/dim
index 60d4d36ef25f..54cbfe956c18 100755
--- a/dim
+++ b/dim
@@ -31,8 +31,8 @@
set -e
#
-# User configuration. Set in environment or configuration file. See
-# dimrc.sample for an example.
+# User configuration. Global DIM_ prefixed uppercase variables. Set in
+# environment or configuration file. See dimrc.sample for an example.
#
# dim configuration file
@@ -68,7 +68,7 @@ DIM_TEMPLATE_SIGNATURE=${DIM_TEMPLATE_SIGNATURE:-$HOME/.dim.template.signature}
DIM_GPG_KEYID=${DIM_GPG_KEYID:+-u $DIM_GPG_KEYID}
#
-# Internal configuration.
+# Internal configuration. Global dim_ prefixed variables.
#
dim=$(basename $0)
@@ -78,7 +78,7 @@ dim_timestamp="$(date --utc +%Yy-%mm-%dd-%Hh-%Mm-%Ss) UTC"
# Recipients for all dim based pull requests.
# Add To: lines to the end, Cc: lines in the beginning with -c.
-pull_request_recipients=(
+dim_pull_request_recipients=(
-c "Daniel Vetter <daniel.vetter at ffwll.ch>"
-c "Jani Nikula <jani.nikula at linux.intel.com>"
-c "Joonas Lahtinen <joonas.lahtinen at linux.intel.com>"
@@ -92,7 +92,7 @@ pull_request_recipients=(
)
# Recipients for drm-intel-testing updates.
-test_request_recipients=(
+dim_test_request_recipients=(
-c "Jani Nikula <jani.nikula at linux.intel.com>"
-c "Joonas Lahtinen <joonas.lahtinen at linux.intel.com>"
-c "Rodrigo Vivi <rodrigo.vivi at intel.com>"
@@ -101,7 +101,7 @@ test_request_recipients=(
)
# integration configuration
-integration_config=nightly.conf
+dim_integration_config=nightly.conf
function read_integration_config
{
@@ -110,16 +110,16 @@ function read_integration_config
declare -g -A drm_tip_repos
declare -g -a drm_tip_config
- if [ -r $DIM_PREFIX/drm-rerere/$integration_config ]; then
+ if [ -r $DIM_PREFIX/drm-rerere/$dim_integration_config ]; then
# shellcheck source=/dev/null
- source $DIM_PREFIX/drm-rerere/$integration_config
+ source $DIM_PREFIX/drm-rerere/$dim_integration_config
if [[ "${#drm_tip_repos[@]}" = "0" ]] || [[ "${#drm_tip_config[@]}" = "0" ]]; then
- echoerr "$integration_config not set up correctly, please fix manually"
+ echoerr "$dim_integration_config not set up correctly, please fix manually"
exit 1
fi
else
- echoerr "$integration_config is missing, please check your configuration and/or run dim setup"
+ echoerr "$dim_integration_config is missing, please check your configuration and/or run dim setup"
exit 1
fi
@@ -164,7 +164,7 @@ function pause
# Variable naming convetion:
#
# repo:
-# symbolic git repository name from $integration_config
+# symbolic git repository name from $dim_integration_config
# remote:
# local remote name in the git repository for the current path
# branch:
@@ -643,12 +643,12 @@ function dim_rebuild_tip
cd $rerere
if git status --porcelain | grep -v "^.. rr-cache" | grep -q -v "^[ ?][ ?]"; then
- warn_or_fail "integration configuration file $integration_config not committed"
+ warn_or_fail "integration configuration file $dim_integration_config not committed"
fi
update_rerere_cache
- echo -n "Reloading $integration_config... "
+ echo -n "Reloading $dim_integration_config... "
read_integration_config
echo "Done."
@@ -1237,10 +1237,10 @@ function dim_create_branch
git push $DRY_RUN $remote +$branch --set-upstream
cd $DIM_PREFIX/drm-rerere
- $DRY sed -i "s/^\() # DO NOT CHANGE THIS LINE\)$/\t\"$repo\t\t${branch//\//\\\/}\"\n\1/" $integration_config
+ $DRY sed -i "s/^\() # DO NOT CHANGE THIS LINE\)$/\t\"$repo\t\t${branch//\//\\\/}\"\n\1/" $dim_integration_config
- $DRY git add $integration_config
- $DRY git commit --quiet -m "Add $repo $branch to $integration_config"
+ $DRY git add $dim_integration_config
+ $DRY git commit --quiet -m "Add $repo $branch to $dim_integration_config"
}
function dim_remove_branch
@@ -1265,7 +1265,7 @@ function dim_remove_branch
repo=$(branch_to_repo $branch)
if [[ $repo == "" ]] ; then
- echoerr "$branch not found in $integration_config"
+ echoerr "$branch not found in $dim_integration_config"
exit 1
fi
@@ -1274,10 +1274,10 @@ function dim_remove_branch
git push $DRY_RUN $remote --delete $branch
$DRY git fetch $remote --prune
cd $DIM_PREFIX/drm-rerere
- $DRY sed -i "/^[[:space:]]*\"${repo}[[:space:]]\+${branch//\//\\\/}.*$/d" $integration_config
+ $DRY sed -i "/^[[:space:]]*\"${repo}[[:space:]]\+${branch//\//\\\/}.*$/d" $dim_integration_config
- $DRY git add $integration_config
- $DRY git commit --quiet -m "Remove $repo $branch from $integration_config"
+ $DRY git add $dim_integration_config
+ $DRY git commit --quiet -m "Remove $repo $branch from $dim_integration_config"
dim_rebuild_tip
}
@@ -1312,7 +1312,7 @@ function dim_checkout
fi
if [[ $repo == "" ]] ; then
- echoerr "$branch not found in $integration_config"
+ echoerr "$branch not found in $dim_integration_config"
exit 1
fi
@@ -1755,7 +1755,7 @@ function dim_update_next_continue
prep_pull_mail_signature >> $req_file
$DRY $DIM_MUA -s "Updated drm-intel-testing" \
- -i $req_file "${test_request_recipients[@]}"
+ -i $req_file "${dim_test_request_recipients[@]}"
}
function dim_tag_branch
@@ -1827,7 +1827,7 @@ function dim_pull_request
git request-pull $upstream $git_url $tag >> $req_file
$DRY $DIM_MUA -s "[PULL] $branch" \
- -i $req_file "${pull_request_recipients[@]}"
+ -i $req_file "${dim_pull_request_recipients[@]}"
}
function dim_pull_request_next
@@ -2237,7 +2237,7 @@ check_for_updates
git_version_check
#
-# Command line options.
+# Command line options. Global short uppercase variables.
#
DRY_RUN=
@@ -2311,7 +2311,7 @@ export __dim_running=1
# Commands useful for developers don't need a full dim setup
if list_developer_commands | grep -qx $subcmd; then
- if [ -r $DIM_PREFIX/drm-rerere/$integration_config ]; then
+ if [ -r $DIM_PREFIX/drm-rerere/$dim_integration_config ]; then
read_integration_config
fi
else
--
2.11.0
More information about the dim-tools
mailing list