Skip to content

fix(build): The number of build jobs should depend on RAM size #1939

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
Jan 23, 2025
Merged

fix(build): The number of build jobs should depend on RAM size #1939

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ef2105d
The number of build jobs depends on RAM
lslezak Jan 23, 2025
ad44f40
Use the standard %_smp_tasksize_proc macro
lslezak Jan 23, 2025
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
17 changes: 13 additions & 4 deletions rust/package/agama.spec
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#
# spec file for package agama
#
# Copyright (c) 2023-2024 SUSE LLC
# Copyright (c) 2023-2025 SUSE LLC
#
# All modifications and additions to the file contributed by third parties
# remain the property of their copyright owners, unless otherwise agreed
Expand All @@ -12,9 +12,15 @@
# license that conforms to the Open Source Definition (Version 1.9)
# published by the Open Source Initiative.

# Please submit bugfixes or comments via http://bugs.opensuse.org/
# Please submit bugfixes or comments via https://bugs.opensuse.org/
#

# Require at least 1.3GB RAM per each parallel job (the size is in MB)
%global _smp_tasksize_proc 1300

# Redefine the _smp_mflags macro so it takes the amount of RAM into account
%define _smp_mflags "-j%{getncpus:proc}"

Name: agama
# This will be set by osc services, that will run after this.
Version: 0
Expand All @@ -23,7 +29,7 @@ Summary: Agama Installer
# If you know the license, put it's SPDX string here.
# Alternately, you can use cargo lock2rpmprovides to help generate this.
License: GPL-2.0-or-later
Url: https://github.com/opensuse/agama
Url: https://github.com/agama-project/agama
Source0: agama.tar
Source1: vendor.tar.zst

Expand Down Expand Up @@ -70,7 +76,7 @@ Version: 0
Release: 0
Summary: Agama command-line interface
License: GPL-2.0-only
Url: https://github.com/opensuse/agama
Url: https://github.com/agama-project/agama

%description -n agama-cli
Command line program to interact with the Agama installer.
Expand Down Expand Up @@ -129,6 +135,9 @@ package contains a systemd service to run scripts when booting the installed sys
# find vendor -type f -name \*.rs -exec chmod -x '{}' \;

%build
# just for debugging failures with low memory
cat /proc/meminfo | head -n 3

%{cargo_build}
cargo run --package xtask -- manpages
gzip out/man/*
Expand Down