Skip to content
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.

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

CI: Missing dependencies #68

Merged
merged 1 commit into from
Jun 4, 2024
Merged

CI: Missing dependencies #68

merged 1 commit into from
Jun 4, 2024

Conversation

me00001
Copy link
Collaborator

@me00001 me00001 commented Jun 4, 2024

@me00001 me00001 merged commit 1e07dd9 into master Jun 4, 2024
3 checks passed
@me00001 me00001 deleted the CI branch June 4, 2024 10:28
@xlmnxp
Copy link
Owner

xlmnxp commented Jun 4, 2024

thank you @me00001 @hamirmahal 🙏

@hamirmahal
Copy link
Contributor

You're welcome @xlmnxp.

Comment on lines 11 to +22
sudo apt-get install \
librust-gstreamer-sys-dev \
librust-gtk4-sys-dev \
libunwind-dev
libunwind-dev \
librust-gstreamer-sys-dev \
librust-gdk-sys-dev \
librust-atk-dev \
librust-gtk4-sys-dev \
libgdk-pixbuf-2.0-dev \
librust-pango-dev \
librust-gobject-sys-dev \
ffmpeg
Copy link
Contributor

Choose a reason for hiding this comment

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

This seems to work with just

          sudo apt-get install \
            librust-gdk-sys-dev \
            librust-gtk4-sys-dev \
            librust-gstreamer-sys-dev \
            libunwind-dev

@hamirmahal
Copy link
Contributor

https://github.com/hamirmahal/blue-recorder/commit/b798069dccce5f10ea85749ef17f2c9ab1abcd84 is the commit that works with just

          sudo apt-get install \
            librust-gdk-sys-dev \
            librust-gtk4-sys-dev \
            librust-gstreamer-sys-dev \
            libunwind-dev

I will probably delete that branch soon. We may want to simplify the dependencies to just these four, to speed up CI workflow times and reduce code complexity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants