update-prod-static: Remove unused authors-not-required.

This argument hasn't done anything since we moved constructing the
/team page to a cron job.
This commit is contained in:
Tim Abbott
2020-04-24 13:26:32 -07:00
parent 7e0eeb20a3
commit 5187d5032c
4 changed files with 5 additions and 10 deletions

View File

@@ -403,7 +403,7 @@ if [ "$has_appserver" = 0 ]; then
# If we're installing from a git checkout, we need to run
# `tools/update-prod-static` in order to build the static
# assets.
su zulip -c '/home/zulip/deployments/current/tools/update-prod-static --authors-not-required'
su zulip -c '/home/zulip/deployments/current/tools/update-prod-static'
fi
fi

View File

@@ -142,7 +142,7 @@ elif args.from_git:
# update-prod-static with the git upgrade process. But it'll fail
# safely; this seems like a worthwhile tradeoff to minimize downtime.
logging.info("Building static assets...")
subprocess.check_call(["./tools/update-prod-static", "--authors-not-required", "--prev-deploy",
subprocess.check_call(["./tools/update-prod-static", "--prev-deploy",
os.path.join(DEPLOYMENTS_DIR, 'current')],
preexec_fn=su_to_zulip)
logging.info("Caching zulip git version...")

View File

@@ -2,18 +2,15 @@
set -eu
usage() {
echo "Usage: $0 <ZULIP_VERSION> [--authors-not-required]" >&2
echo "Usage: $0 <ZULIP_VERSION>" >&2
exit 1
}
args="$(getopt -o '' -l 'authors-not-required,help' -n "$0" -- "$@")"
args="$(getopt -o '' -l 'help' -n "$0" -- "$@")"
eval "set -- $args"
authors_not_required=
while true; do
case "$1" in
--authors-not-required)
authors_not_required=--authors-not-required; shift;;
--) shift; break;;
*) usage;;
esac
@@ -92,7 +89,7 @@ rabbitmq_password = 'not_used_here'
initial_password_salt = 'not_used_here'
EOF
./tools/update-prod-static $authors_not_required
./tools/update-prod-static
# We don't need duplicate copies of emoji with hashed paths, and they would break bugdown
find prod-static/serve/generated/emoji/images/emoji/ -regex '.*\.[0-9a-f]+\.png' -delete

View File

@@ -25,8 +25,6 @@ sanity_check.check_venv(__file__)
parser = argparse.ArgumentParser()
parser.add_argument('--prev-deploy', metavar='DIR',
help='a previous deploy from which to reuse files if possible')
parser.add_argument('--authors-not-required', action='store_true', default=False,
help='Authors files need not be updated')
args = parser.parse_args()
prev_deploy = args.prev_deploy