FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

`install-powershell.sh`: Switch from `/etc/lsb-release` to `/etc/os-release` for Debian by Himura2la · Pull Request #10773 · PowerShell/PowerShell · GitHub

install-powershell.sh: Switch from /etc/lsb-release to /etc/os-release for Debian - #10773

Merged
1 commit merged into
PowerShell:masterfrom
Himura2la:patch-1
Oct 12, 2019
Merged

install-powershell.sh: Switch from /etc/lsb-release to /etc/os-release for Debian#10773
1 commit merged into
PowerShell:masterfrom
Himura2la:patch-1

Conversation

Vladislav Glagolev (Himura2la) commented Oct 11, 2019
edited
Loading

Copy link
Copy Markdown
Contributor

PR Summary

The file /etc/lsb-release does not exist on Debian 9 and 10. This is not a standard Linux file.
Use /etc/os-release instead.
Plus, the /etc/os-release file is standard and may exist on many other distributives, which may eliminate the need of checking some other distributive-related files.

PR Context

PowerShell installation on Debian

PR Checklist

/etc/lsb-release does not exist on Debians
Travis Plunk (TravisEz13) changed the title install-powershell.sh: Switch from /etc/lsb-release to /etc/os-release install-powershell.sh: Switch from /etc/lsb-release to /etc/os-release for Debian Oct 11, 2019
Travis Plunk (TravisEz13) added the CL-Tools Indicates that a PR should be marked as a tools change in the Change Log label Oct 11, 2019
Travis Plunk (TravisEz13) added the AutoMerge informs the bot to automerge the PR label Oct 11, 2019

Copy link
Copy Markdown

Hello Travis Plunk (@TravisEz13)!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 24 hours, a condition that will be fulfilled in about 19 hours. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

ghost merged commit d200ba0 into PowerShell:master Oct 12, 2019

Copy link
Copy Markdown

🎉v7.0.0-preview.5 has been released which incorporates this pull request.:tada:

Handy links:

Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
This pull request was closed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AutoMerge informs the bot to automerge the PR CL-Tools Indicates that a PR should be marked as a tools change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL