xrtdeps : fix and update Fedora package list#8633
Open
Byvirven wants to merge 2 commits intoXilinx:masterfrom
Open
xrtdeps : fix and update Fedora package list#8633Byvirven wants to merge 2 commits intoXilinx:masterfrom
Byvirven wants to merge 2 commits intoXilinx:masterfrom
Conversation
Collaborator
|
Can one of the admins verify this patch? |
Collaborator
|
Byvirven - is not a collaborator |
2 similar comments
Collaborator
|
Byvirven - is not a collaborator |
Collaborator
|
Byvirven - is not a collaborator |
Signed-off-by: Marco Dos Santos Oliveira <marco.dossantosoliveira@sunrise.ch>
- Added logic to detect Fedora version from /etc/os-release. - Introduced conditional inclusion of openssl-static, protobuf-static, and python2-sphinx based on Fedora version: - openssl-static for Fedora ≤ 35 - protobuf-static for Fedora ≤ 38 - python2-sphinx for Fedora < 32, otherwise python3-sphinx for Fedora ≥ 32 - Cleaned up redundant entries in the package list (FD_LIST). - Improved error handling for unavailable packages with --skip-broken option in dnf. - Ensured compatibility with Fedora's evolving package ecosystem by addressing deprecated packages. Signed-off-by: Marco Dos Santos Oliveira <marco.dossantosoliveira@sunrise.ch>
f682ae4 to
dba89d9
Compare
Collaborator
|
Byvirven - is not a collaborator |
stsoe
requested changes
Dec 2, 2024
Collaborator
stsoe
left a comment
There was a problem hiding this comment.
Why is aiebu submodule changed?
Author
but maybe I mess somewhere with submodules linking |
Collaborator
I don't see [0f3a6a] in https://github.com/Xilinx/aiebu/commits/main/ so I am not sure exactly what is being pulled in here. I do see aie-rt#13 but this PR is to-be merged and the connection to aiebu appears to be missing? |
mackncheesiest
added a commit
to mackncheesiest/XRT
that referenced
this pull request
Feb 21, 2026
As noted also in Xilinx#8633, the dependency script has issues for Fedora systems. This updates that script for Fedora 43, taking care to try and not introduce any issues for earlier versions. Structured as a separate commit + PR because it takes a slightly different approach to the task.
mackncheesiest
added a commit
to mackncheesiest/XRT
that referenced
this pull request
Feb 21, 2026
As noted also in Xilinx#8633, the dependency script has issues for Fedora systems. This updates that script for Fedora 43, taking care to try and not introduce any issues for earlier versions. Structured as a separate commit + PR because it takes a slightly different approach to the task.
mackncheesiest
added a commit
to mackncheesiest/XRT
that referenced
this pull request
Feb 22, 2026
As noted also in Xilinx#8633, the dependency script has issues for Fedora systems. This updates that script for Fedora 43, taking care to try and not introduce any issues for earlier versions. Structured as a separate commit + PR because it takes a slightly different approach to the task. Signed-off by: Joshua Mack <mackncheesiest@gmail.com>
mackncheesiest
added a commit
to mackncheesiest/XRT
that referenced
this pull request
Feb 22, 2026
As noted also in Xilinx#8633, the dependency script has issues for Fedora systems. This updates that script for Fedora 43, taking care to try and not introduce any issues for earlier versions. Structured as a separate commit + PR because it takes a slightly different approach to the task. Signed-off-by: Joshua Mack <mackncheesiest@gmail.com>
stsoe
pushed a commit
that referenced
this pull request
Feb 23, 2026
As noted also in #8633, the dependency script has issues for Fedora systems. This updates that script for Fedora 43, taking care to try and not introduce any issues for earlier versions. Structured as a separate commit + PR because it takes a slightly different approach to the task. Signed-off-by: Joshua Mack <mackncheesiest@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem solved by the commit
This commit updates the xrtdeps.sh script to fix the Fedora package installation list, ensuring compatibility with newer Fedora versions. Specifically, it addresses the absence of some packages and adapts the list dynamically based on the Fedora version.
Bug / issue (if any) fixed, which PR introduced the bug, how it was discovered
No specific PR introduced the issue. The problem was identified while attempting to execute the xrtdeps.sh script on Fedora 41, where several packages were missing or incompatible with the current package list. Probably not the first time.
How problem was solved, alternative solutions (if any) and why they were rejected
The problem was solved by:
Adding logic to detect the Fedora version dynamically from /etc/os-release.
Introducing conditional logic to include specific packages only for supported Fedora versions:
Alternative solutions, such as manually installing missing packages or ignoring unavailable ones, were rejected as they would not ensure compatibility across Fedora versions and could cause issues in automated environments.
Risks (if any) associated the changes in the commit
The risks are minimal, but possible issues include:
What has been tested and how, request additional testing if necessary
The updated script was tested on Fedora 41 to ensure:
Additional testing could include running the script on other Fedora versions (e.g., 35, 38) to confirm conditional logic behaves as expected.
Documentation impact (if any)
The documentation for the xrtdeps.sh script, if it exists, may need an update to reflect: