Skip to content

common: adjust build system to versions like 1.4.99-1-g4cff24b - #45

Merged
janekmi merged 2 commits into
masterfrom
janekmi/nigtly-fix
Aug 7, 2026
Merged

common: adjust build system to versions like 1.4.99-1-g4cff24b#45
janekmi merged 2 commits into
masterfrom
janekmi/nigtly-fix

Conversation

@janekmi

@janekmi janekmi commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Preview: https://github.com/daos-stack/pmdk/actions/runs/31095435388

Note: None of this affects how DAOS builds PMDK.

  • RPM version does not accept '-'. Replace them with '~'.
  • pmdk.spec has to force SRCVERSION so generated *.pc files specify the same version string.
  • fix test so it accepts '~'.

This change is Reviewable

@janekmi
janekmi force-pushed the janekmi/nigtly-fix branch 3 times, most recently from db5d0d8 to 9041dc1 Compare August 6, 2026 11:26
@janekmi
janekmi marked this pull request as ready for review August 6, 2026 11:27
@janekmi janekmi changed the title common: replace all - with ~ for RPM version common: adjust build system to versions like 1.4.99-1-g4cff24b Aug 6, 2026
Note: None of this affects how DAOS builds PMDK.

- RPM version does not accept '-'. Replace them with '~'.
- pmdk.spec has to force SRCVERSION so generated *.pc files specify the
  same version string.
- fix test so it accepts '~'.

Signed-off-by: Jan Michalski <jan-marian.michalski@hpe.com>
@janekmi
janekmi force-pushed the janekmi/nigtly-fix branch from 9041dc1 to 89c427f Compare August 6, 2026 11:28
@janekmi
janekmi requested review from grom72 and osalyk August 6, 2026 11:31

@osalyk osalyk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

:lgtm:

@osalyk reviewed 3 files and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on grom72).

Signed-off-by: Jan Michalski <jan-marian.michalski@hpe.com>

@osalyk osalyk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@osalyk reviewed 1 file and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on grom72).

@janekmi
janekmi merged commit 08618ed into master Aug 7, 2026
9 checks passed
@janekmi
janekmi deleted the janekmi/nigtly-fix branch August 7, 2026 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants