Skip to content

Commit b2e5a79

Browse files
authored
Remove temporary workarounds since reaching next checkpoint release (#1888)
Resolves tiny-pilot/tinypilot-pro#978 This PR removes the temporary workarounds no longer needed since reaching the next checkpoint release after 2.6.0 (i.e., the 2.7.0 checkpoint release). Notes 1. We're removing all the workarounds marked with the following issue URL: - [`https://github.com/tiny-pilot/tinypilot-pro/issues/978`](https://github.com/tiny-pilot/tinypilot-pro/issues/978) <a data-ca-tag href="https://codeapprove.com/pr/tiny-pilot/tinypilot/1888"><img src="https://codeapprove.com/external/github-tag-allbg.png" alt="Review on CodeApprove" /></a>
1 parent 56fe836 commit b2e5a79

File tree

2 files changed

+0
-34
lines changed

2 files changed

+0
-34
lines changed

debian-pkg/debian/tinypilot.postinst

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -60,16 +60,6 @@ runuser tinypilot \
6060
deactivate
6161
popd
6262

63-
# Workaround to restore the default NGINX config that has been previously
64-
# modified by ansible-role-nginx. We can remove this on TinyPilot's next
65-
# checkpoint release.
66-
# https://github.com/tiny-pilot/tinypilot-pro/issues/978
67-
if grep --silent 'tinypilot' /etc/nginx/nginx.conf; then
68-
cp \
69-
/usr/share/tinypilot/nginx.conf \
70-
/etc/nginx/nginx.conf
71-
fi
72-
7363
tc358743_overlay_enabled='false'
7464
if grep --silent '^dtoverlay=tc358743$' "${BOOT_CONFIG_PATH}" ; then
7565
tc358743_overlay_enabled='true'
@@ -118,24 +108,6 @@ acap: {
118108
EOF
119109
fi
120110

121-
# Clean up the legacy TinyPilot-specific config file for uStreamer. We can
122-
# remove this on TinyPilot's next checkpoint release.
123-
# https://github.com/tiny-pilot/tinypilot-pro/issues/978
124-
readonly USTREAMER_ANSIBLE_CONFIG='/home/ustreamer/config.yml'
125-
if [[ -f "${USTREAMER_ANSIBLE_CONFIG}" ]]; then
126-
rm "${USTREAMER_ANSIBLE_CONFIG}"
127-
fi
128-
129-
# Workaround to remove settings that are not user-configurable to avoid
130-
# inconsistent config. We can remove this on TinyPilot's next checkpoint
131-
# release.
132-
# https://github.com/tiny-pilot/tinypilot-pro/issues/978
133-
sed \
134-
--in-place \
135-
--expression '/ustreamer_port/d' \
136-
--expression '/ustreamer_persistent/d' \
137-
"${TINYPILOT_SETTINGS_FILE}"
138-
139111
# Create a directory for uStreamer launcher configs.
140112
readonly USTREAMER_LAUNCHER_BASE_DIR='/opt/ustreamer-launcher'
141113
readonly USTREAMER_LAUNCHER_CONFIG_DIR="${USTREAMER_LAUNCHER_BASE_DIR}/configs.d"

debian-pkg/debian/tinypilot.preinst

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,4 @@ if [[ -d /opt/tinypilot/.git ]]; then
99
rm -rf /opt/tinypilot
1010
fi
1111

12-
# Workaround to remove TinyPilot's legacy NGINX config. We can remove this on
13-
# TinyPilot's next checkpoint release.
14-
# https://github.com/tiny-pilot/tinypilot-pro/issues/978
15-
rm -f \
16-
/etc/nginx/sites-enabled/tinypilot.conf
17-
1812
#DEBHELPER#

0 commit comments

Comments
 (0)