Skip to content

feat(cli): allow building packages for unknown distros - #25

Open
mikonse wants to merge 1 commit into
mainfrom
milo/allow-non-debian-ubuntu-distros
Open

feat(cli): allow building packages for unknown distros#25
mikonse wants to merge 1 commit into
mainfrom
milo/allow-non-debian-ubuntu-distros

Conversation

@mikonse

@mikonse mikonse commented Aug 11, 2026

Copy link
Copy Markdown
Member

E.g. allows packaging for unknown debian / ubuntu derivates or other random distros like yocto
which just happen to use apt / dpkg.

@mikonse
mikonse requested a review from TheJJ August 11, 2026 18:34
E.g. allows packaging for unknown debian / ubuntu derivates or
other random distros like yocto
which just happen to use apt / dpkg.
@mikonse
mikonse force-pushed the milo/allow-non-debian-ubuntu-distros branch from 48c0ea5 to f58a7af Compare August 11, 2026 18:40

@TheJJ TheJJ left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this complicates the distro handling quite a lot, i think we can design it better by using a custom distro enum value.

use std::sync::LazyLock;

#[derive(Clone, Debug, Serialize, Deserialize, Eq, PartialEq, Hash)]
pub enum Distro {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

instead of getting rid of this, why not have a third "custom" entry?
then we can get rid of all the is_ functions and can still support any custom approach incl other package managers someday.

}
}

fn check_bare_os_release_for_builtin(

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

hm but what if we have dpkg/apt installed on gentoo? i think we can leave out this check, we should rather test if apt/dpkg/whatever the distro's pkg manager is, is available.

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