-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: dnf
module
#377
Draft
fiftydinar
wants to merge
52
commits into
main
Choose a base branch
from
dnf-module
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+488
−1
Draft
feat: dnf
module
#377
Changes from 42 commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
fd4ad28
feat: `dnf` module
fiftydinar cd91b05
chore: Add support for native installation of repos
fiftydinar cf3d495
docs: Revise the note about %OS_VERSION% in repos
fiftydinar 2f392e7
fix: missing quote in echo
fiftydinar 6d1047f
fix: Get raw COPR repos array elements without JSON brackets
fiftydinar 9d13c87
chore: Auto-confirm prompts (`-y`) when adding repos
fiftydinar 1396a18
fix: Accidental syntax error for `done`
fiftydinar b898af2
chore: Remove useless echo
fiftydinar 6523979
chore: Install & remove using `dnf` in single operation cases
fiftydinar 97566b8
docs: Add note that URL is not preferred for copr repos
fiftydinar 0bb79d0
fix: `/opt` symlink issue & migrate removal + install to `dnf`
fiftydinar 70c5766
chore: Document code some more
fiftydinar e01ed40
docs: Fix note about using install & remove at the same time
fiftydinar 7d4aff9
chore: Add `dnf` to `modules.json`
fiftydinar a4e9ba4
chore: Add support for replacing packages
fiftydinar 7da24f5
chore: Add option to enable or disable installation of weak dependencies
fiftydinar d495b6a
docs: Style backtick fix
fiftydinar 1137c8b
fix(rpm-ostree): Symlinking `/opt/` multiple times caused an error
fiftydinar eb522e3
chore: Rename `weak-dependencies` to `install-weak-dependencies` & ad…
fiftydinar 57f8b22
chore: Change `copr: user/project` formt to `COPR user/project`
fiftydinar 7a5bcea
fix: Syntax error due to redundant `fi`
fiftydinar c7ae430
fix: Restore logic about not removing any newlines to COPR repos
fiftydinar b1a7c6f
feat: Add removal & installation of RPM groups
fiftydinar b7c0206
docs: Style fix
fiftydinar feb0d71
chore: Make sorting order in typespec better
fiftydinar fba3840
typespec: Minor leftover fix
fiftydinar e27ba39
chore: Fix typespec replace definition
fiftydinar a53dd5b
chore: Add mutter-patched COPR to module.yml
fiftydinar 1a81609
fix: Forgot to replace `rpm-ostree` to `dnf` when checking for local …
fiftydinar a72ae5c
fix: Adding local file repos
fiftydinar 2790123
fix: Replacing packages
fiftydinar ecffeae
chore: Make error message consistent
fiftydinar 84f89d0
chore: Begin implementation of specific flag options for every instal…
fiftydinar 5d7f34b
chore: Simplify printing log message for install packages
fiftydinar 7fc5a24
feat: Add dnf flags to all operations
fiftydinar 63f745b
feat: Add `remove-unused-dependencies` option for package removal
fiftydinar b474a5a
chore: Fix group-remove typespec
fiftydinar 770bb33
chore: Fix group-install typespec
fiftydinar d8a9006
chore: Fix remove typespec
fiftydinar 33065b4
chore: Fix install typespec
fiftydinar 3423190
chore: Remove leftover global `install-weak-dependencies` option
fiftydinar f176120
chore: Remove redundant dnf5 plugin check
fiftydinar 7458b4b
fix: dnf group remove command missed `group` prefix
fiftydinar ce4163c
fix: Don't quote the flags, else dnf will see it as an empty argument…
fiftydinar c8043a4
fix: Don't quote the flags, else dnf will see it as an empty argument…
fiftydinar aa92eff
fix: Don't quote the flags, else dnf will see it as an empty argument…
fiftydinar 8265b85
fix: Don't quote the flags, else dnf will see it as an empty argument…
fiftydinar c9f4fb0
fix: Don't quote the flags, else dnf will see it as an empty argument…
fiftydinar 92df6a0
fix: Don't quote the flags, else dnf will see it as an empty argument…
fiftydinar e66b176
Merge branch 'main' into dnf-module
fiftydinar 4920cc4
chore: Add separate `copr` array
fiftydinar 7612cc2
fix: Assume yes for copr enable
gmpinder File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,70 @@ | ||
# `dnf` | ||
|
||
The [`dnf`](https://docs.fedoraproject.org/en-US/quick-docs/dnf/) module offers pseudo-declarative package and repository management using `dnf`. | ||
|
||
The module first downloads the repository files from URLs declared under `repos:` into `/etc/yum.repos.d/`. The magic string `%OS_VERSION%` can be substituted with the current VERSION_ID (major Fedora version), which can be used, for example, for pulling correct versions of repositories which have fixed Fedora version in the URL. | ||
|
||
You can also add repository files directly into your git repository if URLs are not provided. For example: | ||
```yml | ||
repos: | ||
- my-repository.repo # copies in .repo file from files/dnf/my-repository.repo to /etc/yum.repos.d/ | ||
``` | ||
|
||
Specific COPR repositories can also be specified in `COPR user/project` format & is prefered over using direct COPR URL. | ||
|
||
If you use a repo that requires adding custom keys (eg. Brave Browser), you can import the keys by declaring the key URLs under `keys:`. The magic string acts the same as it does in `repos`. | ||
|
||
Then the module installs the packages declared under `install:` using `dnf -y install --refresh`, it removes the packages declared under `remove:` using `dnf -y remove`. If there are packages declared under both `install:` and `remove:` then removal is performed 1st & install 2nd. | ||
|
||
Installing RPM packages directly from a `http(s)` url that points to the RPM file is also supported, you can just put the URLs under `install:` and they'll be installed along with the other packages. The magic string `%OS_VERSION%` is substituted with the current VERSION_ID (major Fedora version) like with the `repos:` property. | ||
|
||
If an RPM is not available in a repository or as an URL, you can also install it directly from a file in your git repository. For example: | ||
```yml | ||
install: | ||
- weird-package.rpm # tries to install files/dnf/weird-package.rpm | ||
``` | ||
|
||
Additionally, the `dnf` module supports a fix for packages that install into `/opt/`. Installation for packages that install into folder names declared under `optfix:` are fixed using some symlinks. Directory path in `/opt/` for those packages should be provided in recipe, like in Example Configuration. | ||
|
||
There is an option to install & remove RPM groups if desired in `group-install:` & `group-remove:`. RPM groups removal & installation always run before packages removal & installation. To see the list of all available RPM groups, you can use `dnf group list` command. | ||
|
||
The module can also replace base RPM packages with packages from any repo. Under `replace:`, the module finds every pair of keys `- from-repo:` and `packages:`. (Multiple pairs are supported.) The module uses `- from-repo:` key to gather the repo for package replacement, then it replaces packages declared under `packages:` using the command `dnf -y distro-sync --refresh --repo "${repo}" "${packages}"`. The magic string `%OS_VERSION%` is substituted with the current VERSION_ID (major Fedora version) as already said above. You need to assure that you provided the repo in `repos:` before using replacement functionality. To gather the repo ID that you need to input, you can use `dnf repo list` command. | ||
|
||
:::note | ||
[Removed packages are still present in the underlying ostree repository](https://coreos.github.io/rpm-ostree/administrator-handbook/#removing-a-base-package), what `remove` does is kind of like hiding them from the system, it doesn't free up storage space. | ||
::: | ||
|
||
## `dnf` behavior options | ||
|
||
There are several options that can be enabled during the package/group install + removal & during package replace, which modify the behavior of the package manager during those operations. | ||
|
||
Those include: | ||
|
||
Install operation: | ||
- `install-weak-dependencies` (`--setopt=install_weak_deps=True/False` flag) | ||
- `skip-unavailable-packages` (`--skip-unavailable` flag) | ||
- `skip-broken-packages` (`--skip-broken` flag) | ||
- `allow-erasing-packages` (`--allowerasing` flag) | ||
|
||
Remove operation: | ||
- `remove-unused-dependencies` (`--no-autoremove` flag) | ||
|
||
|
||
### `dnf` install/replace behavior options | ||
|
||
#### `install-weak-dependencies` | ||
`install-weak-dependencies` option is used to enable or disable installation of weak dependencies for every install & replace operation. By default, this option is true, which means that weak dependencies are installed by default. Which kind of dependencies are considered weak can be seen [here](https://docs.fedoraproject.org/en-US/packaging-guidelines/WeakDependencies/). | ||
|
||
#### `skip-unavailable-packages` | ||
`skip-unavailable-packages` option is used to continue or abort install/replace operation if there are no packages available in the repo in install operation, or if they are not available in the system in replace operation. By default, this option is false, which means that install/replace operation aborts in case of unavailable packages. | ||
|
||
#### `skip-broken-packages` | ||
`skip-broken-packages` option is used to continue or abort install/replace operation if there are broken packages in the system. By default, this option is false, which means that install/replace operation aborts in case of broken packages. | ||
|
||
#### `allow-erasing-packages` | ||
`allow-erasing-packages` option is used to allow erasing/removing problematic packages if they cause issues in install/replace operation. By default, this option is false, which means that problematic packages won't be removed & operation will be aborted. | ||
|
||
### `dnf` package (non-group) removal behavior options | ||
|
||
#### `remove-unused-dependencies` | ||
`remove-unused-dependencies` option is used to control the behavior of removing unused dependencies when some main packages are removed. By default, this option is true. Only compatible with removing packages, not compatible with removing RPM groups. |
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,294 @@ | ||
#!/usr/bin/env bash | ||
|
||
# Tell build process to exit if there are any errors. | ||
set -euo pipefail | ||
|
||
# Fail the build if dnf5 isn't installed | ||
if ! rpm -q dnf5 &>/dev/null; then | ||
echo "ERROR: Main dependency 'dnf5' is not installed. Install 'dnf5' before using this module to solve this error." | ||
exit 1 | ||
fi | ||
|
||
# Pull in repos | ||
get_json_array REPOS 'try .["repos"][]' "${1}" | ||
if [[ ${#REPOS[@]} -gt 0 ]]; then | ||
echo "Adding repositories" | ||
# Substitute %OS_VERSION% & remove newlines/whitespaces from all repo entries | ||
for i in "${!REPOS[@]}"; do | ||
repo="${REPOS[$i]}" | ||
repo="${repo//%OS_VERSION%/${OS_VERSION}}" | ||
REPOS[$i]="${repo//[$'\t\r\n ']}" | ||
# Remove spaces/newlines for all repos other than COPR | ||
if [[ "${repo}" != "COPR "* ]]; then | ||
REPOS[$i]="${repo//[$'\t\r\n ']}" | ||
else | ||
REPOS[$i]="${repo}" | ||
fi | ||
done | ||
# dnf config-manager & dnf copr don't support adding multiple repositories at once, hence why for/done loop is used | ||
for repo in "${REPOS[@]}"; do | ||
if [[ "${repo}" =~ ^https?:\/\/.* ]]; then | ||
echo "Adding repository URL: '${repo}'" | ||
dnf -y config-manager addrepo --from-repofile="${repo}" | ||
elif [[ "${repo}" == *".repo" ]] && [[ -f "${CONFIG_DIRECTORY}/dnf/${repo}" ]]; then | ||
echo "Adding repository file: '${repo##*/}'" | ||
dnf -y config-manager addrepo --from-repofile="${CONFIG_DIRECTORY}/dnf/${repo}" | ||
elif [[ "${repo}" == "COPR "* ]]; then | ||
echo "Adding COPR repository: '${repo#COPR }'" | ||
dnf -y copr enable "${repo#COPR }" | ||
fi | ||
done | ||
fi | ||
|
||
# Install RPM keys if they are provided | ||
get_json_array KEYS 'try .["keys"][]' "${1}" | ||
if [[ ${#KEYS[@]} -gt 0 ]]; then | ||
echo "Adding keys" | ||
for KEY in "${KEYS[@]}"; do | ||
KEY="${KEY//%OS_VERSION%/${OS_VERSION}}" | ||
rpm --import "${KEY//[$'\t\r\n ']}" | ||
done | ||
fi | ||
|
||
# Create symlinks to fix packages that create directories in /opt | ||
get_json_array OPTFIX 'try .["optfix"][]' "${1}" | ||
if [[ ${#OPTFIX[@]} -gt 0 ]]; then | ||
echo "Creating symlinks to fix packages that install to /opt" | ||
# Create symlink for /opt to /var/opt since it is not created in the image yet | ||
mkdir -p "/var/opt" | ||
ln -snf "/var/opt" "/opt" | ||
# Create symlinks for each directory specified in recipe.yml | ||
for OPTPKG in "${OPTFIX[@]}"; do | ||
OPTPKG="${OPTPKG%\"}" | ||
OPTPKG="${OPTPKG#\"}" | ||
mkdir -p "/usr/lib/opt/${OPTPKG}" | ||
ln -s "../../usr/lib/opt/${OPTPKG}" "/var/opt/${OPTPKG}" | ||
echo "Created symlinks for ${OPTPKG}" | ||
done | ||
fi | ||
|
||
# Install & remove group packages | ||
get_json_array GROUP_INSTALL 'try .["group-install"].["packages"][]' "${1}" | ||
get_json_array GROUP_REMOVE 'try .["group-remove"].["packages"][]' "${1}" | ||
|
||
# Get if 'install-weak-dependencies' is provided for group-install | ||
WEAK_DEPENDENCIES=$(echo "${1}" | jq -r 'try .["group-install"].["install-weak-dependencies"]') | ||
|
||
if [[ -z "${WEAK_DEPENDENCIES}" ]] || [[ "${WEAK_DEPENDENCIES}" == "null" ]] || [[ "${WEAK_DEPENDENCIES}" == "true" ]]; then | ||
WEAK_DEPS_FLAG="--setopt=install_weak_deps=True" | ||
elif [[ "${WEAK_DEPENDENCIES}" == "false" ]]; then | ||
WEAK_DEPS_FLAG="--setopt=install_weak_deps=False" | ||
fi | ||
|
||
# Get if 'skip-unavailable-packages' is provided for group-install | ||
SKIP_UNAVAILABLE=$(echo "${1}" | jq -r 'try .["group-install"].["skip-unavailable-packages"]') | ||
|
||
if [[ -z "${SKIP_UNAVAILABLE}" ]] || [[ "${SKIP_UNAVAILABLE}" == "null" ]] || [[ "${SKIP_UNAVAILABLE}" == "false" ]]; then | ||
SKIP_UNAVAILABLE_FLAG="" | ||
elif [[ "${SKIP_UNAVAILABLE}" == "true" ]]; then | ||
SKIP_UNAVAILABLE_FLAG="--skip-unavailable" | ||
fi | ||
|
||
# Get if 'skip-broken-packages' is provided for group-install | ||
SKIP_BROKEN=$(echo "${1}" | jq -r 'try .["group-install"].["skip-broken-packages"]') | ||
|
||
if [[ -z "${SKIP_BROKEN}" ]] || [[ "${SKIP_BROKEN}" == "null" ]] || [[ "${SKIP_BROKEN}" == "false" ]]; then | ||
SKIP_BROKEN_FLAG="" | ||
elif [[ "${SKIP_BROKEN}" == "true" ]]; then | ||
SKIP_BROKEN_FLAG="--skip-broken" | ||
fi | ||
|
||
# Get if 'allow-erasing-packages' is provided for group-install | ||
ALLOW_ERASING=$(echo "${1}" | jq -r 'try .["group-install"].["allow-erasing-packages"]') | ||
|
||
if [[ -z "${ALLOW_ERASING}" ]] || [[ "${ALLOW_ERASING}" == "null" ]] || [[ "${ALLOW_ERASING}" == "false" ]]; then | ||
ALLOW_ERASING_FLAG="" | ||
elif [[ "${ALLOW_ERASING}" == "true" ]]; then | ||
ALLOW_ERASING_FLAG="--allowerasing" | ||
fi | ||
|
||
if [[ ${#GROUP_INSTALL[@]} -gt 0 && ${#GROUP_REMOVE[@]} -gt 0 ]]; then | ||
echo "Removing & Installing RPM groups" | ||
echo "Removing: ${GROUP_REMOVE[*]}" | ||
echo "Installing: ${GROUP_INSTALL[*]}" | ||
dnf -y group remove "${GROUP_REMOVE[@]}" | ||
dnf -y "${WEAK_DEPS_FLAG}" group install --refresh "${SKIP_UNAVAILABLE_FLAG}" "${SKIP_BROKEN_FLAG}" "${ALLOW_ERASING_FLAG}" "${GROUP_INSTALL[@]}" | ||
fiftydinar marked this conversation as resolved.
Show resolved
Hide resolved
|
||
elif [[ ${#GROUP_INSTALL[@]} -gt 0 ]]; then | ||
echo "Installing RPM groups" | ||
echo "Installing: ${GROUP_INSTALL[*]}" | ||
dnf -y "${WEAK_DEPS_FLAG}" group install --refresh "${SKIP_UNAVAILABLE_FLAG}" "${SKIP_BROKEN_FLAG}" "${ALLOW_ERASING_FLAG}" "${GROUP_INSTALL[@]}" | ||
fiftydinar marked this conversation as resolved.
Show resolved
Hide resolved
|
||
elif [[ ${#GROUP_REMOVE[@]} -gt 0 ]]; then | ||
echo "Removing RPM groups" | ||
echo "Removing: ${GROUP_REMOVE[*]}" | ||
dnf -y remove "${GROUP_REMOVE[@]}" | ||
fiftydinar marked this conversation as resolved.
Show resolved
Hide resolved
|
||
fi | ||
|
||
get_json_array INSTALL_PKGS 'try .["install"].["packages"][]' "${1}" | ||
get_json_array REMOVE_PKGS 'try .["remove"].["packages"][]' "${1}" | ||
|
||
# Get if 'install-weak-dependencies' is provided for package install | ||
WEAK_DEPENDENCIES=$(echo "${1}" | jq -r 'try .["install"].["install-weak-dependencies"]') | ||
|
||
if [[ -z "${WEAK_DEPENDENCIES}" ]] || [[ "${WEAK_DEPENDENCIES}" == "null" ]] || [[ "${WEAK_DEPENDENCIES}" == "true" ]]; then | ||
WEAK_DEPS_FLAG="--setopt=install_weak_deps=True" | ||
elif [[ "${WEAK_DEPENDENCIES}" == "false" ]]; then | ||
WEAK_DEPS_FLAG="--setopt=install_weak_deps=False" | ||
fi | ||
|
||
# Get if 'skip-unavailable-packages' is provided for package install | ||
SKIP_UNAVAILABLE=$(echo "${1}" | jq -r 'try .["install"].["skip-unavailable-packages"]') | ||
|
||
if [[ -z "${SKIP_UNAVAILABLE}" ]] || [[ "${SKIP_UNAVAILABLE}" == "null" ]] || [[ "${SKIP_UNAVAILABLE}" == "false" ]]; then | ||
SKIP_UNAVAILABLE_FLAG="" | ||
elif [[ "${SKIP_UNAVAILABLE}" == "true" ]]; then | ||
SKIP_UNAVAILABLE_FLAG="--skip-unavailable" | ||
fi | ||
|
||
# Get if 'skip-broken-packages' is provided for package install | ||
SKIP_BROKEN=$(echo "${1}" | jq -r 'try .["install"].["skip-broken-packages"]') | ||
|
||
if [[ -z "${SKIP_BROKEN}" ]] || [[ "${SKIP_BROKEN}" == "null" ]] || [[ "${SKIP_BROKEN}" == "false" ]]; then | ||
SKIP_BROKEN_FLAG="" | ||
elif [[ "${SKIP_BROKEN}" == "true" ]]; then | ||
SKIP_BROKEN_FLAG="--skip-broken" | ||
fi | ||
|
||
# Get if 'allow-erasing-packages' is provided for package install | ||
ALLOW_ERASING=$(echo "${1}" | jq -r 'try .["install"].["allow-erasing-packages"]') | ||
|
||
if [[ -z "${ALLOW_ERASING}" ]] || [[ "${ALLOW_ERASING}" == "null" ]] || [[ "${ALLOW_ERASING}" == "false" ]]; then | ||
ALLOW_ERASING_FLAG="" | ||
elif [[ "${ALLOW_ERASING}" == "true" ]]; then | ||
ALLOW_ERASING_FLAG="--allowerasing" | ||
fi | ||
|
||
# Get if 'remove-unused-dependencies' is provided for package removal | ||
REMOVE_UNUSED_DEPS=$(echo "${1}" | jq -r 'try .["remove"].["remove-unused-dependencies"]') | ||
|
||
if [[ -z "${REMOVE_UNUSED_DEPS}" ]] || [[ "${REMOVE_UNUSED_DEPS}" == "null" ]] || [[ "${REMOVE_UNUSED_DEPS}" == "true" ]]; then | ||
REMOVE_UNUSED_DEPS_FLAG="" | ||
elif [[ "${REMOVE_UNUSED_DEPS}" == "false" ]]; then | ||
REMOVE_UNUSED_DEPS_FLAG="--no-autoremove" | ||
fi | ||
|
||
CLASSIC_INSTALL=false | ||
HTTPS_INSTALL=false | ||
LOCAL_INSTALL=false | ||
|
||
# Sort classic, URL & local install packages | ||
if [[ ${#INSTALL_PKGS[@]} -gt 0 ]]; then | ||
for i in "${!INSTALL_PKGS[@]}"; do | ||
PKG="${INSTALL_PKGS[$i]}" | ||
if [[ "${PKG}" =~ ^https?:\/\/.* ]]; then | ||
INSTALL_PKGS[$i]="${PKG//%OS_VERSION%/${OS_VERSION}}" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
HTTPS_INSTALL=true | ||
HTTPS_PKGS+=("${INSTALL_PKGS[$i]}") | ||
elif [[ ! "${PKG}" =~ ^https?:\/\/.* ]] && [[ -f "${CONFIG_DIRECTORY}/dnf/${PKG}" ]]; then | ||
LOCAL_INSTALL=true | ||
LOCAL_PKGS+=("${CONFIG_DIRECTORY}/dnf/${PKG}") | ||
else | ||
CLASSIC_INSTALL=true | ||
CLASSIC_PKGS+=("${PKG}") | ||
fi | ||
done | ||
fi | ||
|
||
# Function to inform the user about which type of packages is he installing | ||
echo_rpm_install() { | ||
if ${CLASSIC_INSTALL}; then | ||
echo "Installing: ${CLASSIC_PKGS[*]}" | ||
fi | ||
if ${HTTPS_INSTALL}; then | ||
echo "Installing package(s) directly from URL: ${HTTPS_PKGS[*]}" | ||
fi | ||
if ${LOCAL_INSTALL}; then | ||
echo "Installing local package(s): ${LOCAL_PKGS[*]}" | ||
fi | ||
} | ||
|
||
# Remove & install RPM packages | ||
if [[ ${#INSTALL_PKGS[@]} -gt 0 && ${#REMOVE_PKGS[@]} -gt 0 ]]; then | ||
echo "Removing & Installing RPMs" | ||
echo "Removing: ${REMOVE_PKGS[*]}" | ||
echo_rpm_install | ||
dnf -y remove "${REMOVE_UNUSED_DEPS_FLAG}" "${REMOVE_PKGS[@]}" | ||
dnf -y "${WEAK_DEPS_FLAG}" install --refresh "${SKIP_UNAVAILABLE_FLAG}" "${SKIP_BROKEN_FLAG}" "${ALLOW_ERASING_FLAG}" "${INSTALL_PKGS[@]}" | ||
fiftydinar marked this conversation as resolved.
Show resolved
Hide resolved
|
||
elif [[ ${#INSTALL_PKGS[@]} -gt 0 ]]; then | ||
echo "Installing RPMs" | ||
echo_rpm_install | ||
dnf -y "${WEAK_DEPS_FLAG}" install --refresh "${SKIP_UNAVAILABLE_FLAG}" "${SKIP_BROKEN_FLAG}" "${ALLOW_ERASING_FLAG}" "${INSTALL_PKGS[@]}" | ||
fiftydinar marked this conversation as resolved.
Show resolved
Hide resolved
|
||
elif [[ ${#REMOVE_PKGS[@]} -gt 0 ]]; then | ||
echo "Removing RPMs" | ||
echo "Removing: ${REMOVE_PKGS[*]}" | ||
dnf -y remove "${REMOVE_UNUSED_DEPS_FLAG}" "${REMOVE_PKGS[@]}" | ||
fiftydinar marked this conversation as resolved.
Show resolved
Hide resolved
|
||
fi | ||
|
||
get_json_array REPLACE 'try .["replace"][]' "$1" | ||
|
||
# Replace RPM packages from any repository | ||
if [[ ${#REPLACE[@]} -gt 0 ]]; then | ||
for REPLACEMENT in "${REPLACE[@]}"; do | ||
|
||
# Get repository | ||
REPO=$(echo "${REPLACEMENT}" | jq -r 'try .["from-repo"]') | ||
REPO="${REPO//%OS_VERSION%/${OS_VERSION}}" | ||
REPO="${REPO//[$'\t\r\n ']}" | ||
|
||
# Ensure repository is provided | ||
if [[ "${REPO}" == "null" ]] || [[ -z "${REPO}" ]]; then | ||
echo "ERROR: Key 'from-repo' was declared, but repository URL was not provided." | ||
exit 1 | ||
fi | ||
|
||
# Get packages to replace | ||
get_json_array PACKAGES 'try .["packages"][]' "${REPLACEMENT}" | ||
|
||
# Ensure packages are provided | ||
if [[ ${#PACKAGES[@]} -eq 0 ]]; then | ||
echo "ERROR: No packages were provided for repository '${REPO}'." | ||
exit 1 | ||
fi | ||
|
||
# Get if 'install-weak-dependencies' is provided for package replace | ||
WEAK_DEPENDENCIES=$(echo "${REPLACEMENT}" | jq -r 'try .["install-weak-dependencies"]') | ||
|
||
if [[ -z "${WEAK_DEPENDENCIES}" ]] || [[ "${WEAK_DEPENDENCIES}" == "null" ]] || [[ "${WEAK_DEPENDENCIES}" == "true" ]]; then | ||
WEAK_DEPS_FLAG="--setopt=install_weak_deps=True" | ||
elif [[ "${WEAK_DEPENDENCIES}" == "false" ]]; then | ||
WEAK_DEPS_FLAG="--setopt=install_weak_deps=False" | ||
fi | ||
|
||
# Get if 'skip-unavailable-packages' is provided for package replace | ||
SKIP_UNAVAILABLE=$(echo "${REPLACEMENT}" | jq -r 'try .["skip-unavailable-packages"]') | ||
|
||
if [[ -z "${SKIP_UNAVAILABLE}" ]] || [[ "${SKIP_UNAVAILABLE}" == "null" ]] || [[ "${SKIP_UNAVAILABLE}" == "false" ]]; then | ||
SKIP_UNAVAILABLE_FLAG="" | ||
elif [[ "${SKIP_UNAVAILABLE}" == "true" ]]; then | ||
SKIP_UNAVAILABLE_FLAG="--skip-unavailable" | ||
fi | ||
|
||
# Get if 'skip-broken-packages' is provided for package replace | ||
SKIP_BROKEN=$(echo "${REPLACEMENT}" | jq -r 'try .["skip-broken-packages"]') | ||
|
||
if [[ -z "${SKIP_BROKEN}" ]] || [[ "${SKIP_BROKEN}" == "null" ]] || [[ "${SKIP_BROKEN}" == "false" ]]; then | ||
SKIP_BROKEN_FLAG="" | ||
elif [[ "${SKIP_BROKEN}" == "true" ]]; then | ||
SKIP_BROKEN_FLAG="--skip-broken" | ||
fi | ||
|
||
# Get if 'allow-erasing-packages' is provided for package replace | ||
ALLOW_ERASING=$(echo "${REPLACEMENT}" | jq -r 'try .["allow-erasing-packages"]') | ||
|
||
if [[ -z "${ALLOW_ERASING}" ]] || [[ "${ALLOW_ERASING}" == "null" ]] || [[ "${ALLOW_ERASING}" == "false" ]]; then | ||
ALLOW_ERASING_FLAG="" | ||
elif [[ "${ALLOW_ERASING}" == "true" ]]; then | ||
ALLOW_ERASING_FLAG="--allowerasing" | ||
fi | ||
|
||
echo "Replacing packages from repository: '${REPO}'" | ||
echo "Replacing: ${PACKAGES[*]}" | ||
|
||
dnf -y "${WEAK_DEPS_FLAG}" distro-sync --refresh "${SKIP_UNAVAILABLE_FLAG}" "${SKIP_BROKEN_FLAG}" "${ALLOW_ERASING_FLAG}" --repo "${REPO}" "${PACKAGES[@]}" | ||
fiftydinar marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
done | ||
fi |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[shellcheck] reported by reviewdog 🐶
$/$ {} is unnecessary on arithmetic variables. SC2004