Skip to content

Fixes and improvements for git-import-srpm and qlogic-fastlinq-alt_8.42 submodules - #81

Merged
tescande merged 5 commits into
mainfrom
tee/fixes_and_qlogic-fastlinq-alt_8.42_submodules
Jul 30, 2026
Merged

tescande merged 5 commits into
mainfrom
tee/fixes_and_qlogic-fastlinq-alt_8.42_submodules

Conversation

@tescande

Copy link
Copy Markdown
Contributor

No description provided.

tescande added 4 commits July 29, 2026 17:55
This patch adds support for command line option -d use to import
drivers.

Signed-off-by: Thierry Escande <thierry.escande@vates.tech>
This patch adds support for command line option -D use to not delete
temporary files after a run.

Signed-off-by: Thierry Escande <thierry.escande@vates.tech>
Signed-off-by: Thierry Escande <thierry.escande@vates.tech>
In order to get the srpm build directory, the function get_build_dir()
was using 'rpmspec -q --qf' by using the format string
'%{name}-%{version}' or the format string passed to the %autosetup
macro, which can use non standard rpm macros (i.e macro declared locally
with '%define macro value').
Since 'rpmspec -q --qf' does not evaluate local macros, this patch
introduces a new function query_format_spec_file() that uses 'rpmspec
--shell' instead.

This also fixes the build dir when a recent rpmbuild (ver >=6) is used
that sets the build dir as '%{name}-%{version}-build/${build_dir}'.

Signed-off-by: Thierry Escande <thierry.escande@vates.tech>
@tescande
tescande requested a review from a team as a code owner July 29, 2026 16:00
Signed-off-by: Thierry Escande <thierry.escande@vates.tech>
@tescande
tescande force-pushed the tee/fixes_and_qlogic-fastlinq-alt_8.42_submodules branch from 5da453a to a6d0d95 Compare July 29, 2026 16:04

@casasnovas casasnovas left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm, thanks for fixing!

@tescande
tescande merged commit c35613c into main Jul 30, 2026
4 checks passed
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.

2 participants