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

Support Ubuntu and Debian-derived OS using ID_LIKE in /etc/os-release #343

Merged
merged 2 commits into from Jul 20, 2022

Conversation

onepunchdan
Copy link
Contributor

Tested on Pop!_OS 22.04, this should extend support to Ubuntu and Debian-derived distros that reference either in the ID_LIKE variable of /etc/os-release. Pop!_OS 22.04 in particular lists both ID_LIKE="ubuntu debian" in which case only the first is used.

@onepunchdan
Copy link
Contributor Author

Fixes #328

deb-get Outdated Show resolved Hide resolved
Copy link

@dominix dominix left a comment

Choose a reason for hiding this comment

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

works for me on pop!_os 22.04

@philclifford
Copy link
Member

philclifford commented Jul 18, 2022

Confirmed working on Linux Mint
closes #352
closes #348

@philclifford
Copy link
Member

philclifford commented Jul 19, 2022

should also work for #327 because neon has

ID_LIKE="ubuntu debian"

@flexiondotorg flexiondotorg merged commit ff77019 into wimpysworld:main Jul 20, 2022
@ymauray
Copy link
Contributor

ymauray commented Jul 20, 2022

This broke deb-get 0.3.2, at least on Ubuntu MATE 22.04. /etc/os-release has

ID=ubuntu
ID_LIKE=debian

The code provided by this PR then replaces UPSTREAM_ID with debian, which breaks all packages that install from a PPA, given they all look like this :

function deb_copyq() {
    if [ "${UPSTREAM_ID}" == "ubuntu" ]; then
        PPA="ppa:hluk/copyq"
        PRETTY_NAME="CopyQ"
        WEBSITE="https://hluk.github.io/CopyQ/"
        SUMMARY="An advanced clipboard manager with editing and scripting features."
    fi
}

making it impossible to update or install packages from PPAs :

sudo deb-get install copyq
  [!] ERROR! Download URL for copyq is empty. Skipping.

mrm48 pushed a commit to mrm48/deb-get that referenced this pull request Jul 21, 2022
…wimpysworld#343)

* Support Ubuntu and Debian-derived OS using ID_LIKE in /etc/os-release

* Revert end of case correction.
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.

None yet

6 participants