Skip to content

Debian free fix#15895

Merged
daschuer merged 6 commits intomixxxdj:2.5from
blackhold:debian_free_fix
Feb 2, 2026
Merged

Debian free fix#15895
daschuer merged 6 commits intomixxxdj:2.5from
blackhold:debian_free_fix

Conversation

@blackhold
Copy link
Copy Markdown
Contributor

Fixing debian_buildenv.sh file.

@github-actions github-actions Bot added library ui preferences packaging developer experience Issues, bugs and PRs related to the development process, development environment & developer docs labels Jan 23, 2026
@daschuer daschuer changed the base branch from main to 2.5 January 23, 2026 13:24
Copy link
Copy Markdown
Member

@daschuer daschuer left a comment

Choose a reason for hiding this comment

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

Cool, we are almost there. Thanks.

Comment thread tools/debian_buildenv.sh Outdated
Comment on lines +70 to +81
echo ""
read -p "Do you want to enable the non-free repository and install fonts-ubuntu? (y/n) " -n 1 -r
echo
if [[ $REPLY =~ ^[Yy]$ ]]; then
echo "Enabling non-free repository..."
# Add non-free to sources.list if not already present
if ! grep -q " non-free$" /etc/apt/sources.list; then
sudo sed -i 's/^\(deb.*\) \(main\|contrib\|non-free-firmware\)$/\1 \2 non-free/' /etc/apt/sources.list
fi
echo "Updating package list..."
sudo apt-get update
FONTS_UBUNTU_AVAILABLE=true
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.

messing with the sources sources.list feels risky. I don't want to be blamed for destroying users system because of a non standard sources.list file. How about this.

Suggested change
echo ""
read -p "Do you want to enable the non-free repository and install fonts-ubuntu? (y/n) " -n 1 -r
echo
if [[ $REPLY =~ ^[Yy]$ ]]; then
echo "Enabling non-free repository..."
# Add non-free to sources.list if not already present
if ! grep -q " non-free$" /etc/apt/sources.list; then
sudo sed -i 's/^\(deb.*\) \(main\|contrib\|non-free-firmware\)$/\1 \2 non-free/' /etc/apt/sources.list
fi
echo "Updating package list..."
sudo apt-get update
FONTS_UBUNTU_AVAILABLE=true
echo "See also: https://wiki.debian.org/SourcesList"
echo ""
read -p "Would you like to exit to enable 'non-free' now? (y = Exit / n = Continue without fonts): " -n 1 -r
echo
if [[ $REPLY =~ ^[Yy]$ ]]; then
echo "Please edit your /etc/apt/sources.list, run 'sudo apt update', and restart this script."
exit 1

@blackhold
Copy link
Copy Markdown
Contributor Author

Done!

@acolombier
Copy link
Copy Markdown
Member

Hi, this the third PR you are opening to fix tools/debian_buildenv.sh. Could you please make sure:

  • Either the close previous, superseded PR
  • Push the update in already open PR

This will reduce the spam to the community.
Thanks.

@blackhold
Copy link
Copy Markdown
Contributor Author

Done! One question, can I delete debian_free_fix in my local branch? if I do that will delete the commit or not?

@blackhold blackhold requested a review from daschuer January 25, 2026 20:21
@acolombier
Copy link
Copy Markdown
Member

Thanks!

I delete debian_free_fix in my local branch?

If you mean your fork (with a git push -d origin debian_free_fix), no you cannot yet, otherwise it will close this PR. Once it is merged, you can however safely delete it.

Copy link
Copy Markdown
Member

@daschuer daschuer left a comment

Choose a reason for hiding this comment

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

LGTM, Thank you for this nice starter PR.

@daschuer daschuer merged commit 4f55fed into mixxxdj:2.5 Feb 2, 2026
13 checks passed
@acolombier acolombier added this to the 2.5.5 milestone Feb 25, 2026
@github-project-automation github-project-automation Bot moved this from In progress to Done in Releases Feb 25, 2026
@github-project-automation github-project-automation Bot moved this to In progress in Releases Feb 25, 2026
@daschuer
Copy link
Copy Markdown
Member

daschuer commented Mar 6, 2026

@blackhold I like to put your name into the contributor list in the Mixxx about box.
Shall I use your real name or you GitHub user name?

@blackhold
Copy link
Copy Markdown
Contributor Author

Lately I'm using my real name Laura Mora but Blackhold is easy to follow, so, if it's possible: Laura Mora (Blackhold). Thanks :)

I have pending to continue the stems development, I'm ending some other work jobs, when I finish them I'll continue!

@daschuer
Copy link
Copy Markdown
Member

daschuer commented Mar 7, 2026

You are always welcome, as your time allows.
Ok I will use your real name like most others.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

developer experience Issues, bugs and PRs related to the development process, development environment & developer docs library packaging preferences ui

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants