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

everything #160

Open
wants to merge 194 commits into
base: upstream
Choose a base branch
from
Open

everything #160

wants to merge 194 commits into from

Conversation

zen0bit
Copy link
Member

@zen0bit zen0bit commented Mar 29, 2024

No description provided.

zen0bit and others added 30 commits September 14, 2023 13:51
Added new configuration file option "sound_card" and new command-line
switch "--sound-card" to change the virtual sound hardware. Allowed
options are "intel-hda" (default), "ac97", "es1370", "sb16", and "none".
Also changed default sound card for Solaris to "ac97" and for FreeDOS to
"sb16".

quickemu-project#705
Co-authored-by: Chase Covello <[email protected]>
zen0bit and others added 25 commits January 16, 2024 13:26
Signed-off-by: zenobit <[email protected]>
Signed-off-by: zenobit <[email protected]>
list, list_csv and list_json Not mentioned!

Signed-off-by: zenobit <[email protected]>
Signed-off-by: zenobit <[email protected]>
Signed-off-by: zenobit <[email protected]>
Signed-off-by: zenobit <[email protected]>
Signed-off-by: zenobit <[email protected]>
@@ -1780,7 +1787,10 @@ if [ -n "${VM}" ] && [ -e "${VM}" ]; then
VMPATH=$(realpath "$(dirname "${VM}")")
VM_MONITOR_SOCKETPATH="${VMDIR}/${VMNAME}-monitor.socket"
VM_SERIAL_SOCKETPATH="${VMDIR}/${VMNAME}-serial.socket"

if [ ! -f "${disk_img}" ]; then
cd "${VMPATH}"

Choose a reason for hiding this comment

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

⚠️ [shellcheck] reported by reviewdog 🐶
Use 'cd ... || exit' or 'cd ... || return' in case cd fails. SC2164

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

shellcheck

quickget|1138 col 34| Double quote array expansions to avoid re-splitting elements. SC2068
quickget|1141 col 14| Double quote array expansions to avoid re-splitting elements. SC2068
quickget|1141 col 32| Double quote array expansions to avoid re-splitting elements. SC2068
quickget|1147 col 25| Prefer mapfile or read -a to split command output (or quote to avoid splitting). SC2207
quickget|1162 col 10| Double quote array expansions to avoid re-splitting elements. SC2068
quickget|1162 col 28| Double quote array expansions to avoid re-splitting elements. SC2068
quickget|1362 col 29| Quote this to prevent word splitting. SC2046
quickget|1362 col 52| Double quote to prevent globbing and word splitting. SC2086
quickget|1362 col 91| Double quote to prevent globbing and word splitting. SC2086
quickget|1635 col 24| Double quote to prevent globbing and word splitting. SC2086
quickget|1646 col 92| This { is literal. Check expression (missing ;/\n?) or quote it. SC1083
quickget|1646 col 106| This } is literal. Check expression (missing ;/\n?) or quote it. SC1083
quickget|1646 col 144| Double quote to prevent globbing and word splitting. SC2086
quickget|1646 col 152| Double quote to prevent globbing and word splitting. SC2086
quickget|1674 col 24| Double quote to prevent globbing and word splitting. SC2086
quickget|1676 col 24| Double quote to prevent globbing and word splitting. SC2086
quickget|1703 col 27| Double quote to prevent globbing and word splitting. SC2086
quickget|2256 col 9| read without -r will mangle backslashes. SC2162
quickget|2554 col 90| This { is literal. Check expression (missing ;/\n?) or quote it. SC1083
quickget|2554 col 104| This } is literal. Check expression (missing ;/\n?) or quote it. SC1083
quickget|2554 col 142| Double quote to prevent globbing and word splitting. SC2086
quickget|2554 col 150| Double quote to prevent globbing and word splitting. SC2086
quickget|2635 col 33| Double quote to prevent globbing and word splitting. SC2086
quickget|2737 col 19| Double quote to prevent globbing and word splitting. SC2086
quickget|2748 col 19| Double quote to prevent globbing and word splitting. SC2086
quickget|2749 col 20| Double quote to prevent globbing and word splitting. SC2086
quickget|2768 col 32| Double quote to prevent globbing and word splitting. SC2086
quickget|2769 col 25| Double quote to prevent globbing and word splitting. SC2086
quickget|2772 col 32| Double quote to prevent globbing and word splitting. SC2086
quickget|2773 col 25| Double quote to prevent globbing and word splitting. SC2086
quickget|2824 col 32| Double quote to prevent globbing and word splitting. SC2086
quickget|2825 col 25| Double quote to prevent globbing and word splitting. SC2086
quickget|2828 col 32| Double quote to prevent globbing and word splitting. SC2086
quickget|2829 col 25| Double quote to prevent globbing and word splitting. SC2086
quickget|3585 col 29| Quote this to prevent word splitting. SC2046
quickget|3591 col 34| Remove quotes from right-hand side of =~ to match as a regex rather than literally. SC2076
quickget|3593 col 29| Quote this to prevent word splitting. SC2046
quickget|3599 col 12| Expressions don't expand in single quotes, use double quotes for that. SC2016
quickget|3636 col 29| Quote this to prevent word splitting. SC2046
quickget|3651 col 36| Double quote to prevent globbing and word splitting. SC2086
quickget|3653 col 10| Double quote to prevent globbing and word splitting. SC2086
quickget|3655 col 17| Double quote to prevent globbing and word splitting. SC2086
quickget|3657 col 16| Double quote to prevent globbing and word splitting. SC2086
quickget|3659 col 31| Double quote to prevent globbing and word splitting. SC2086
quickget|3663 col 24| Double quote to prevent globbing and word splitting. SC2086
quickget|3667 col 17| Double quote to prevent globbing and word splitting. SC2086
quickget|3669 col 14| Double quote to prevent globbing and word splitting. SC2086
quickget|3714 col 10| Useless echo? Instead of 'echo $(cmd)', just use 'cmd'. SC2005
quickget|3714 col 48| Quote this to prevent word splitting. SC2046
quickget|3719 col 28| Double quote to prevent globbing and word splitting. SC2086
quickget|3722 col 33| Double quote to prevent globbing and word splitting. SC2086
quickget|3756 col 43| Remove quotes from right-hand side of =~ to match as a regex rather than literally. SC2076
quickget|3813 col 36| Remove quotes from right-hand side of =~ to match as a regex rather than literally. SC2076
quickget|3829 col 8| Quote this to prevent word splitting. SC2046
quickget|3829 col 19| Double quote to prevent globbing and word splitting. SC2086
quickget|3830 col 24| Double quote to prevent globbing and word splitting. SC2086
quickget|3862 col 37| Quote this to prevent word splitting. SC2046
quickget|3865 col 41| Quote this to prevent word splitting. SC2046

local SIMPLE_NAME=""
local PRETTY_NAME=""
SIMPLE_NAME="${1}"
PRETTY_NAME=$(os_info ${SIMPLE_NAME} | cut -d'|' -f1)

Choose a reason for hiding this comment

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

📝 [shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting. SC2086

local SIMPLE_NAME=""
local BASED=""
SIMPLE_NAME="${1}"
BASED=$(os_info ${SIMPLE_NAME} | cut -d'|' -f2)

Choose a reason for hiding this comment

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

📝 [shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting. SC2086

local SIMPLE_NAME=""
local CREDE=""
SIMPLE_NAME="${1}"
CREDE=$(os_info ${SIMPLE_NAME} | cut -d'|' -f3)

Choose a reason for hiding this comment

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

📝 [shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting. SC2086

local SIMPLE_NAME=""
local HOMEP=""
SIMPLE_NAME="${1}"
HOMEP=$(os_info ${SIMPLE_NAME} | cut -d'|' -f4)

Choose a reason for hiding this comment

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

📝 [shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting. SC2086

local SIMPLE_NAME=""
local ABOUT=""
SIMPLE_NAME="${1}"
ABOUT=$(os_info ${SIMPLE_NAME} | cut -d'|' -f5)

Choose a reason for hiding this comment

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

📝 [shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting. SC2086

# shellcheck disable=SC1001
local DEBOLD=$(wget -q https://cdimage.debian.org/cdimage/archive/ -O- |grep -e '>[1-9][0-9]\.'|grep -v 'live' | cut -d\> -f9|cut -d\/ -f1 )
local DEBOLD=$(wget -q https://cdimage.debian.org/cdimage/archive/ -O- | grep -e '>[1-9][0-9]\.' | grep -v 'live' | cut -d\> -f9 | cut -d\/ -f1 )

Choose a reason for hiding this comment

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

⚠️ [shellcheck] reported by reviewdog 🐶
DEBOLD appears unused. Verify use (or export if used externally). SC2034

edubuntu|ubuntu-unity|ubuntucinnamon)
echo "${INTERIM_SUPPORT[@]}" daily-live
edubuntu|ubuntu-unity|ubuntucinnamon)
echo ${INTERIM_SUPPORT[@]} daily-live

Choose a reason for hiding this comment

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

🚫 [shellcheck] reported by reviewdog 🐶
Double quote array expansions to avoid re-splitting elements. SC2068

## after 14.04
echo "${LTS_SUPPORT[@]:1}" "${INTERIM_SUPPORT[@]}" daily-live jammy-daily "${EOL_VERSIONS[@]/#/eol-}"
echo ${LTS_SUPPORT[@]:1} ${INTERIM_SUPPORT[@]} daily-live jammy-daily #${EOL_VERSIONS[@]/#/eol-}

Choose a reason for hiding this comment

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

🚫 [shellcheck] reported by reviewdog 🐶
Double quote array expansions to avoid re-splitting elements. SC2068

## after 14.04
echo "${LTS_SUPPORT[@]:1}" "${INTERIM_SUPPORT[@]}" daily-live jammy-daily "${EOL_VERSIONS[@]/#/eol-}"
echo ${LTS_SUPPORT[@]:1} ${INTERIM_SUPPORT[@]} daily-live jammy-daily #${EOL_VERSIONS[@]/#/eol-}

Choose a reason for hiding this comment

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

🚫 [shellcheck] reported by reviewdog 🐶
Double quote array expansions to avoid re-splitting elements. SC2068

#after 16.04
echo "${LTS_SUPPORT[@]:2}" "${INTERIM_SUPPORT[@]}" daily-live jammy-daily "${EOL_VERSIONS[@]/#/eol-}"
echo ${LTS_SUPPORT[@]:2} ${INTERIM_SUPPORT[@]} daily-live jammy-daily #${EOL_VERSIONS[@]/#/eol-}

Choose a reason for hiding this comment

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

🚫 [shellcheck] reported by reviewdog 🐶
Double quote array expansions to avoid re-splitting elements. SC2068

@@ -1780,7 +1787,10 @@ if [ -n "${VM}" ] && [ -e "${VM}" ]; then
VMPATH=$(realpath "$(dirname "${VM}")")
VM_MONITOR_SOCKETPATH="${VMDIR}/${VMNAME}-monitor.socket"
VM_SERIAL_SOCKETPATH="${VMDIR}/${VMNAME}-serial.socket"

if [ ! -f "${disk_img}" ]; then
cd "${VMPATH}"

Choose a reason for hiding this comment

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

[shellcheck (suggestion)] reported by reviewdog 🐶

Suggested change
cd "${VMPATH}"
cd "${VMPATH}" || exit

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

shellcheck (suggestion)

quickget|3663|
quickget|3667|
quickget|3669|
quickget|3719|
quickget|3722|
quickget|3829|

local SIMPLE_NAME=""
local PRETTY_NAME=""
SIMPLE_NAME="${1}"
PRETTY_NAME=$(os_info ${SIMPLE_NAME} | cut -d'|' -f1)

Choose a reason for hiding this comment

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

[shellcheck (suggestion)] reported by reviewdog 🐶

Suggested change
PRETTY_NAME=$(os_info ${SIMPLE_NAME} | cut -d'|' -f1)
PRETTY_NAME=$(os_info "${SIMPLE_NAME}" | cut -d'|' -f1)

local SIMPLE_NAME=""
local BASED=""
SIMPLE_NAME="${1}"
BASED=$(os_info ${SIMPLE_NAME} | cut -d'|' -f2)

Choose a reason for hiding this comment

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

[shellcheck (suggestion)] reported by reviewdog 🐶

Suggested change
BASED=$(os_info ${SIMPLE_NAME} | cut -d'|' -f2)
BASED=$(os_info "${SIMPLE_NAME}" | cut -d'|' -f2)

local SIMPLE_NAME=""
local CREDE=""
SIMPLE_NAME="${1}"
CREDE=$(os_info ${SIMPLE_NAME} | cut -d'|' -f3)

Choose a reason for hiding this comment

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

[shellcheck (suggestion)] reported by reviewdog 🐶

Suggested change
CREDE=$(os_info ${SIMPLE_NAME} | cut -d'|' -f3)
CREDE=$(os_info "${SIMPLE_NAME}" | cut -d'|' -f3)

local SIMPLE_NAME=""
local HOMEP=""
SIMPLE_NAME="${1}"
HOMEP=$(os_info ${SIMPLE_NAME} | cut -d'|' -f4)

Choose a reason for hiding this comment

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

[shellcheck (suggestion)] reported by reviewdog 🐶

Suggested change
HOMEP=$(os_info ${SIMPLE_NAME} | cut -d'|' -f4)
HOMEP=$(os_info "${SIMPLE_NAME}" | cut -d'|' -f4)

local SIMPLE_NAME=""
local ABOUT=""
SIMPLE_NAME="${1}"
ABOUT=$(os_info ${SIMPLE_NAME} | cut -d'|' -f5)

Choose a reason for hiding this comment

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

[shellcheck (suggestion)] reported by reviewdog 🐶

Suggested change
ABOUT=$(os_info ${SIMPLE_NAME} | cut -d'|' -f5)
ABOUT=$(os_info "${SIMPLE_NAME}" | cut -d'|' -f5)

OS="${1}"
os_supported
if command -v neofetch > /dev/null; then
neofetch -l --ascii_distro ${OS}

Choose a reason for hiding this comment

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

[shellcheck (suggestion)] reported by reviewdog 🐶

Suggested change
neofetch -l --ascii_distro ${OS}
neofetch -l --ascii_distro "${OS}"

if command -v neofetch > /dev/null; then
neofetch -l --ascii_distro ${OS}
fi
echo ${OS}

Choose a reason for hiding this comment

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

[shellcheck (suggestion)] reported by reviewdog 🐶

Suggested change
echo ${OS}
echo "${OS}"

fi
echo ${OS}
echo -n 'Pretty name: '
pretty_name ${OS}

Choose a reason for hiding this comment

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

[shellcheck (suggestion)] reported by reviewdog 🐶

Suggested change
pretty_name ${OS}
pretty_name "${OS}"

echo -n 'Pretty name: '
pretty_name ${OS}
echo -n 'Based of: '
os_basedof ${OS}

Choose a reason for hiding this comment

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

[shellcheck (suggestion)] reported by reviewdog 🐶

Suggested change
os_basedof ${OS}
os_basedof "${OS}"

echo -n 'Based of: '
os_basedof ${OS}

if [ -z "$(os_credentials ${OS})" ]; then

Choose a reason for hiding this comment

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

[shellcheck (suggestion)] reported by reviewdog 🐶

Suggested change
if [ -z "$(os_credentials ${OS})" ]; then
if [ -z "$(os_credentials "${OS}")" ]; then

Copy link

🦙 MegaLinter status: ❌ ERROR

Descriptor Linter Files Fixed Errors Elapsed time
❌ BASH shellcheck 2 132 5.78s
⚠️ BASH shfmt 2 2 0.15s
✅ COPYPASTE jscpd yes no 0.76s
✅ REPOSITORY gitleaks yes no 2.94s
✅ REPOSITORY git_diff yes no 0.01s
✅ REPOSITORY grype yes no 12.71s
✅ REPOSITORY secretlint yes no 0.74s
✅ REPOSITORY trivy yes no 4.47s
✅ REPOSITORY trivy-sbom yes no 4.2s
✅ REPOSITORY trufflehog yes no 6.42s
✅ YAML prettier 1 0 0.46s
✅ YAML v8r 1 0 3.89s
✅ YAML yamllint 1 0 0.26s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

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.