Skip to content

fix(dispatcharr): add comskip installation and build dependencies #16224

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
eXistC wants to merge 4 commits into
base: main
Choose a base branch
from
+31 −2
Open

fix(dispatcharr): add comskip installation and build dependencies #16224

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9768e7a
fix(dispatcharr): add comskip installation and build dependencies usi…
eXistC Aug 2, 2026
6d05a83
refactor(dispatcharr): use native fetch_and_deploy_from_url helper
eXistC Aug 5, 2026
b2f39f8
Merge branch 'community-scripts:main' into fix/dispatcharr-comskip-in…
eXistC Aug 5, 2026
ecba253
Merge branch 'community-scripts:main' into fix/dispatcharr-comskip-in…
eXistC Aug 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 15 additions & 1 deletion ct/dispatcharr.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,21 @@ EOF
msg_ok "Migrated Nginx Configuration"
fi

ensure_dependencies vlc-bin vlc-plugin-base
ensure_dependencies vlc-bin vlc-plugin-base build-essential autoconf libtool libargtable2-dev libavformat-dev libsdl2-dev libswscale-dev

if ! command -v comskip &> /dev/null; then
msg_info "Compiling Comskip"
if [[ ! -d "/opt/Comskip" ]]; then
Comment thread
asylumexp marked this conversation as resolved.
Outdated
mkdir -p /opt/Comskip
wget -qO- https://github.com/erikkaashoek/Comskip/archive/refs/heads/master.tar.gz | tar -xz -C /opt/Comskip --strip-components=1

@asylumexp asylumexp Aug 5, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

is the tar command this verbose upon running (eg does it need $STD?

@eXistC eXistC Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ah, i thought it was fine since i uses the -xz flag makes it silent. but now that you mention it, i realized definitely still needs $STD to handle exceptions properly. Thanks for pointing that out! I'll push a fix.

@eXistC eXistC Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Wait, actually, when @tremor021 mentioned using a release tarball earlier, I completely missed that there's actually a built-in helper function fetch_and_deploy_from_url exactly for this. I'll get that swapped over!

fi
cd /opt/Comskip
$STD ./autogen.sh
$STD ./configure
$STD make
$STD make install
msg_ok "Compiled and Installed Comskip"
fi

if check_for_gh_release "Dispatcharr" "Dispatcharr/Dispatcharr"; then
msg_info "Stopping Services"
Expand Down
18 changes: 17 additions & 1 deletion install/dispatcharr-install.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,25 @@ $STD apt install -y \
procps \
vlc-bin \
vlc-plugin-base \
streamlink
streamlink \
autoconf \
libtool \
libargtable2-dev \
libavformat-dev \
libsdl2-dev \
libswscale-dev
msg_ok "Installed Dependencies"

msg_info "Compiling Comskip"
mkdir -p /opt/Comskip
wget -qO- https://github.com/erikkaashoek/Comskip/archive/refs/heads/master.tar.gz | tar -xz -C /opt/Comskip --strip-components=1
cd /opt/Comskip
$STD ./autogen.sh
$STD ./configure
$STD make
$STD make install
msg_ok "Compiled and Installed Comskip"

setup_uv
NODE_VERSION="24" setup_nodejs
PG_VERSION="16" setup_postgresql
Expand Down