Skip to content
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.

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

Update eext tool #94

Merged
merged 8 commits into from
Jan 5, 2024
Merged

Update eext tool #94

merged 8 commits into from
Jan 5, 2024

Conversation

aajith-arista
Copy link
Collaborator

  1. Point BuildRequires to alma 9.2 by default
  2. Make eext base image off alma 9.3 minimal
  3. Remove local-deps from manifest and instead make it a command line param to mock/build
  4. Add manifest spec changes for eextgen
  5. Remove unwanted file
  6. Fix the Dockerfile to be buildable again

Instead of specifying local-deps in manifest, make it a commandline
param. This way, when a generator is implemented, we can make it
generate the command with --use-local-deps argument, when dependencies
are specified in the generator spec.
Copy link
Contributor

@manith-arista manith-arista left a comment

Choose a reason for hiding this comment

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

Changes to support eextgen parameters in eext.yaml looks good.

barney.yaml Show resolved Hide resolved
@aajith-arista aajith-arista merged commit c588859 into main Jan 5, 2024
2 checks passed
@aajith-arista aajith-arista deleted the aajith-update-eext branch January 5, 2024 11:20
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