Skip to content

Fedora packaging#444

Merged
JeromeCCP9 merged 1 commit intowannier-developers:developfrom
LecrisUT:fedora
Feb 3, 2026
Merged

Fedora packaging#444
JeromeCCP9 merged 1 commit intowannier-developers:developfrom
LecrisUT:fedora

Conversation

@LecrisUT
Copy link
Contributor

@LecrisUT LecrisUT commented Apr 18, 2023

If you would like to support Fedora packaging, I am happy to help with that. Here is a skeleton of that framework that automatically builds all PR and commits, and automatically submit new releases to fedora. For this to work there are a few things needed:

  • A proper release github CI, I have a few templates I can add here if you would want
  • Installing packit app

Depends-on: #487

@LecrisUT LecrisUT mentioned this pull request Sep 25, 2023
@LecrisUT LecrisUT changed the base branch from cmake to develop February 22, 2024 07:23
@LecrisUT LecrisUT force-pushed the fedora branch 5 times, most recently from 66c664c to 0a888b6 Compare February 22, 2024 13:51
@LecrisUT LecrisUT marked this pull request as ready for review February 22, 2024 13:55
@LecrisUT LecrisUT force-pushed the fedora branch 3 times, most recently from e0820df to f87057e Compare February 23, 2024 10:38
@LecrisUT LecrisUT mentioned this pull request Feb 23, 2024
@LecrisUT LecrisUT mentioned this pull request Jul 8, 2025
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Copy link
Collaborator

@JeromeCCP9 JeromeCCP9 left a comment

Choose a reason for hiding this comment

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

This looks very helpful indeed. I suspect some header file names have changed; I'll double check these and update separately if needed.

I'll merge this and then return to update the license version to LGPLv2 as that change happens.

@LecrisUT
Copy link
Contributor Author

LecrisUT commented Feb 3, 2026

Note that there is an onboarding step needed to actually get the CI working for this.

@JeromeCCP9
Copy link
Collaborator

Hi Cristian! Yes, I guess we'll have to work on the CI configuration--but I guess we can merge this PR right now anyway, correct?

@LecrisUT
Copy link
Contributor Author

LecrisUT commented Feb 3, 2026

Yes, I guess we'll have to work on the CI configuration

Not really, all of the setup is in .packit.yaml, just needs to do the app instalation (and I think a Fedora account is still needed).

Hope you do not mean the intel tests, those need more explicit tuning and investigation.

but I guess we can merge this PR right now anyway, correct?

Yes, but it would have no affect until the app is installed.

@JeromeCCP9
Copy link
Collaborator

Yes; I guess I'll open an issue to make note of the to-be-done configuration for fedora.

I did some poking in the intel case, yes, I'll go find that shortly & hopefully resolve it.

@JeromeCCP9 JeromeCCP9 merged commit 77b35f7 into wannier-developers:develop Feb 3, 2026
9 of 11 checks passed
@LecrisUT
Copy link
Contributor Author

LecrisUT commented Feb 3, 2026

Oh yeah, I forgot I had the new jobs running i my fork if you want to see how it looks LecrisUT#1

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