Skip to content

Prevent tinypilot.postinst from appending to Janus config#1615

Merged
mtlynch merged 2 commits into
masterfrom
fix-double-append
Sep 1, 2023
Merged

Prevent tinypilot.postinst from appending to Janus config#1615
mtlynch merged 2 commits into
masterfrom
fix-double-append

Conversation

@mtlynch
Copy link
Copy Markdown
Contributor

@mtlynch mtlynch commented Sep 1, 2023

There was a bug in the tinypilot.postinst where we were appending to an existing janus.plugin.ustreamer.jcfg file instead of the intended behavior of recreating the file fresh on each install.
Review on CodeApprove

There was a bug in the tinypilot.postinst where we were appending to an existing janus.plugin.ustreamer.jcfg file instead of the intended behavior of recreating the file fresh on each install.
@mtlynch mtlynch requested review from db39 and jdeanwallace September 1, 2023 13:08
Copy link
Copy Markdown
Contributor Author

mtlynch commented Sep 1, 2023

Automated comment from CodeApprove ➜

@db39 please review this Pull Request

Copy link
Copy Markdown
Contributor Author

mtlynch commented Sep 1, 2023

Automated comment from CodeApprove ➜

@jdeanwallace please review this Pull Request

Copy link
Copy Markdown
Contributor

@jdeanwallace jdeanwallace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated comment from CodeApprove ➜

Approved on CodeApprove
✔️ Approved

LGTM!


👀 @db39, @mtlynch it's your turn please take a look

@mtlynch mtlynch merged commit ed0d1fa into master Sep 1, 2023
@mtlynch mtlynch deleted the fix-double-append branch September 1, 2023 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants