-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
[ci] remove unnecessary package installations in CI #6488
Changes from all commits
bc560b6
db04223
ba5f8f2
95b446f
78df986
9d7a250
ce70add
2f0f6f4
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,9 +20,7 @@ if [[ $OS_NAME == "macos" ]]; then | |
# Check https://github.com/actions/runner-images/tree/main/images/macos for available | ||
# versions of Xcode | ||
sudo xcode-select -s /Applications/Xcode_14.3.1.app/Contents/Developer || exit 1 | ||
if [[ $TASK != "mpi" ]]; then | ||
brew install gcc | ||
fi | ||
brew install gcc | ||
fi | ||
if [[ $TASK == "mpi" ]]; then | ||
brew install open-mpi | ||
|
@@ -41,35 +39,30 @@ else # Linux | |
software-properties-common | ||
|
||
sudo apt-get install --no-install-recommends -y \ | ||
apt-utils \ | ||
build-essential \ | ||
ca-certificates \ | ||
cmake \ | ||
curl \ | ||
git \ | ||
iputils-ping \ | ||
jq \ | ||
libcurl4 \ | ||
libicu-dev \ | ||
libssl-dev \ | ||
libunwind8 \ | ||
locales \ | ||
locales-all \ | ||
netcat \ | ||
unzip \ | ||
zip || exit 1 | ||
locales-all || exit 1 | ||
if [[ $COMPILER == "clang" ]]; then | ||
sudo apt-get install --no-install-recommends -y \ | ||
clang \ | ||
libomp-dev | ||
elif [[ $COMPILER == "clang-17" ]]; then | ||
sudo apt-get install wget | ||
sudo apt-get install --no-install-recommends -y \ | ||
wget | ||
wget -qO- https://apt.llvm.org/llvm-snapshot.gpg.key | sudo tee /etc/apt/trusted.gpg.d/apt.llvm.org.asc | ||
sudo apt-add-repository deb http://apt.llvm.org/jammy/ llvm-toolchain-jammy-17 main | ||
sudo apt-add-repository deb-src http://apt.llvm.org/jammy/ llvm-toolchain-jammy-17 main | ||
sudo apt-get update | ||
sudo apt-get install -y clang-17 | ||
sudo apt-get install --no-install-recommends -y libomp-17-dev | ||
sudo apt-get install -y \ | ||
clang-17 \ | ||
libomp-17-dev | ||
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. With a
https://dev.azure.com/lightgbm-ci/lightgbm-ci/_build/results?buildId=16401&view=results I'm unsure why. For now, just letting |
||
fi | ||
|
||
export LANG="en_US.UTF-8" | ||
|
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.
I believe that
!= "mpi"
was just a shorter way to represent some other subset of jobs and that this has been carried forward unnecessarily from older versions of these build scripts.Proposing removing this condition to simplify the scripts a bit... if the job has
COMPILER=gcc
set, we need to installgcc
.