Skip to content

Use vendor directory in /opt#1542

Draft
d10n wants to merge 1 commit intolemonade-sdk:mainfrom
d10n:linux-packaging-std
Draft

Use vendor directory in /opt#1542
d10n wants to merge 1 commit intolemonade-sdk:mainfrom
d10n:linux-packaging-std

Conversation

@d10n
Copy link
Copy Markdown

@d10n d10n commented Apr 5, 2026

According to Linux packaging conventions, each item in /opt should be a self-contained folder. This allows packages to co-exist without stepping on each other.

Items to be considered for a different PR:
TODO just use the regular FHS directories instead of /opt
TODO use systemd-sysusers and systemd-tmpfiles instead of postinst-rpm

According to Linux packaging conventions, each item in /opt should be a
self-contained folder. This allows packages to co-exist without stepping
on each other.

TODO just use the regular FHS directories instead of /opt
TODO use systemd-sysusers and systemd-tmpfiles instead of postinst-rpm
Comment thread src/cpp/postinst-rpm
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.

Perhaps there should be some migration for the home directory in this new structure

@superm1
Copy link
Copy Markdown
Member

superm1 commented Apr 5, 2026

TODO just use the regular FHS directories instead of /opt
TODO use systemd-sysusers and systemd-tmpfiles instead of postinst-rpm

Another alternative is to have spec file in the tree that matches the Fedora review, which will use FHS.
https://bugzilla.redhat.com/show_bug.cgi?id=2453414

This is what we do for Debian/Ubuntu - the packaging is nearly identical.

@d10n
Copy link
Copy Markdown
Author

d10n commented Apr 5, 2026

@superm1 to make sure I understand, you suggest removing cpack and adopting that Fedora COPR spec file into this repository, right?

@superm1
Copy link
Copy Markdown
Member

superm1 commented Apr 5, 2026

Yes!

@superm1
Copy link
Copy Markdown
Member

superm1 commented Apr 6, 2026

I would propose just put it in contrib/ and use the rpm ci job to build it instead of cpack.

@d10n
Copy link
Copy Markdown
Author

d10n commented Apr 7, 2026

Planning to make the suggested change later today.

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