Skip to content

Use ID_LIKE to figure out which bib file to use if a distro def doesn't exist.#1172

Open
PizzaLovingNerd wants to merge 1 commit intoosbuild:mainfrom
PizzaLovingNerd:main
Open

Use ID_LIKE to figure out which bib file to use if a distro def doesn't exist.#1172
PizzaLovingNerd wants to merge 1 commit intoosbuild:mainfrom
PizzaLovingNerd:main

Conversation

@PizzaLovingNerd
Copy link
Copy Markdown
Contributor

Also remove stillOS symlink as this commit removes the need for it.

I wanted to have a visible warning if this occurred, although I could not figure out how to display that in the CLI other than just adding it as a warning to the logs.

@PizzaLovingNerd PizzaLovingNerd requested a review from a team as a code owner December 30, 2025 04:56
@PizzaLovingNerd PizzaLovingNerd requested review from bcl, croissanne and supakeen and removed request for a team December 30, 2025 04:56
@supakeen supakeen self-assigned this Dec 31, 2025
Comment thread bib/cmd/bootc-image-builder/legacy_iso.go Outdated
Copy link
Copy Markdown
Member

@supakeen supakeen left a comment

Choose a reason for hiding this comment

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

We have similar code elsewhere (which really means we should proceed with the merge) but for now @PizzaLovingNerd if you can address @bcl's comment this can go in :)

@PizzaLovingNerd
Copy link
Copy Markdown
Contributor Author

Just added BCL's suggestion, it will now log what it is trying if finds an ID_Like options. Sorry for the delay in the fix.

Copy link
Copy Markdown

@bcl bcl left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@supakeen
Copy link
Copy Markdown
Member

supakeen commented Jan 9, 2026

Looks good, tests are failing due to: osbuild/images#2123 so let's have that merged first and then I'll retrigger them and this can go in.

@PizzaLovingNerd
Copy link
Copy Markdown
Contributor Author

I've just pulled in new commits to ensure everything is fresh. Since osbuild/images#2123 was merged, can you please re-trigger the tests? Thanks!

@supakeen
Copy link
Copy Markdown
Member

I've just pulled in new commits to ensure everything is fresh. Since osbuild/images#2123 was merged, can you please re-trigger the tests? Thanks!

Done.

@github-actions
Copy link
Copy Markdown

This PR is stale because it had no activity for the past 30 days. Remove the "Stale" label or add a comment, otherwise this PR will be closed in 7 days.

@github-actions github-actions Bot added the Stale Issue or PR with no activity for extended period of time label Mar 28, 2026
@PizzaLovingNerd
Copy link
Copy Markdown
Contributor Author

Hello, any updates on this? I cannot build ISOs with AlmaLinux's CI template atm and have to do it locally with my own fork. AlmaLinux bootc CI still uses the legacy_iso.

@bcl bcl removed the Stale Issue or PR with no activity for extended period of time label Mar 31, 2026
@bcl bcl self-requested a review March 31, 2026 23:39
Copy link
Copy Markdown

@bcl bcl left a comment

Choose a reason for hiding this comment

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

You've introduced a merge commit, we don't use those. Please rebase and squash things so that the commit history is clean.

Reapply the fork-specific stillOS legacy ISO handling on top of the latest upstream history without merge commits.

Made-with: Cursor
@PizzaLovingNerd
Copy link
Copy Markdown
Contributor Author

My apologies, I needed to use my fork with the functionality to produce an ISO, so I updated it from upstream. Should be one commit now.

Copy link
Copy Markdown

@bcl bcl left a comment

Choose a reason for hiding this comment

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

Thanks!

@supakeen supakeen added this pull request to the merge queue Apr 2, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 2, 2026
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.

3 participants