Skip to content

Added support for alts alternatives #29

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.

Sign up for GitHub

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

Merged
lslezak merged 2 commits into from
Jun 30, 2026
Merged

Added support for alts alternatives #29

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1bef33e
Added "alts" support
lslezak Jun 30, 2026
f3ab1cf
Added .editorconfig
lslezak Jun 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions .editorconfig
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
# EditorConfig is awesome: https://EditorConfig.org

# top-most EditorConfig file
root = true

# A special non-standard configuration for the `shfmt` tool which matches any
# file with a shell hashbang on the first line, it does not need the *.sh
# extension in the file name (needs shfmt >= 3.8.0).
#
# To reformat all files run `shfmt -w -l .` at the top-level directory.
[[shell]]
indent_style = space
indent_size = 2
switch_case_indent = true
space_redirects = true
trim_trailing_whitespace = true
2 changes: 1 addition & 1 deletion package/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
FROM opensuse/tumbleweed

# do not install the files marked as documentation (use "rpm --excludedocs")
RUN sed -i -e "s/^.*rpm.install.excludedocs.*/rpm.install.excludedocs = yes/" /etc/zypp/zypp.conf
RUN zypper --non-interactive install --no-recommends zypp-excludedocs

@mvidner mvidner Jun 30, 2026

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.

Just curious, where did you find out about zypp-exludedocs.rpm?

@lslezak lslezak Jun 30, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

In some bug where we discussed setting the "no-recommends" option in Agama with the libzypp team. JFYI there is also zypp-no-recommends package and some more.


# import the YaST OBS GPG key
COPY YaST:Head.pub /usr/share/gpg-keys/
Expand Down
22 changes: 17 additions & 5 deletions package/yast-ci-ruby
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ function set_defaults() {
RUN_RUBOCOP=1
else
RUN_RUBOCOP=0
fi;
fi

if [ -e .spell.yml ]; then
RUN_CHECK_SPELLING=1
Expand Down Expand Up @@ -126,6 +126,7 @@ function exclude() {
echo "ERROR: Unknown step: '$1'"
echo "Known steps: $ALL_STEPS"
exit 1
;;
esac
EXCLUDE_USED=1
}
Expand Down Expand Up @@ -168,6 +169,7 @@ function run_only() {
echo "ERROR: Unknown step: '$1'"
echo "Known steps: $ALL_STEPS"
exit 1
;;
esac
RUN_ONLY_USED=1
}
Expand All @@ -188,7 +190,17 @@ function dump_settings() {
function set_rubocop_version() {
# here we need to set path exactly as update-alternatives needs, so find proper path from its output
RUBOCOP_PATH=$(update-alternatives --display rubocop | sed "/^[^[:space:]]\+$1"'/!d;s/^\([^[:space:]]\+\).*$/\1/')
update-alternatives --set rubocop "$RUBOCOP_PATH"
if [ -n "$RUBOCOP_PATH" ]; then
update-alternatives --set rubocop "$RUBOCOP_PATH"
else
RUBOCOP_PATH=$(alts -l rubocop | grep "Target:" | sed "s/^.*Target:\s*\(\S.*\)$/\1/" | grep "$1")
if [ -n "$RUBOCOP_PATH" ]; then
alts -s -n rubocop "$RUBOCOP_PATH"
else
echo "ERROR: Cannot set the default Rubocop version to $1"
exit 1
fi
fi
}

# run parallel rubocop if it is available,
Expand All @@ -212,11 +224,11 @@ function check_perl() {
# Perl allows checking the syntax only for one file at once, we need to use -n1
# xargs option, to speed it up run the checks in parallel (-P option).
# If you need to specify an additional search path use the PERL5LIB environment variable.
find . -type f -name '*.p[ml]' -print0 \
| xargs -0 -P"$(nproc)" -n1 perl -I src/modules -I /usr/share/YaST2/modules -w -c
find . -type f -name '*.p[ml]' -print0 |
xargs -0 -P"$(nproc)" -n1 perl -I src/modules -I /usr/share/YaST2/modules -w -c
}

# initializa the defaults and parse the command line options
# initialize the defaults and parse the command line options

set_defaults

Expand Down
Loading