[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 Marian Csontos changed: What|Removed |Added Status|POST|CLOSED Resolution|--- |RAWHIDE Flags|needinfo?(ngomp...@gmail.co | |m) | Last Closed||2018-07-17 14:09:42 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/LJMMGV45UOJX4ZZGCL4KV2MVTKOEOIBS/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #43 from Marian Csontos --- Thanks! Everything is done, pushed to master, f27 and f28, built. I did a smoke test - F28 works fine, Rawhide is broken ATM. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/7BBCTJHLJWT4E4ECOYQYXSICKUKCK6ID/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #42 from Neal Gompa --- Looks good except for one bit: > Supplements: (grub2 and python3-boom) Change this to "Supplements: (grub2 and boom-boot)". You can fix this on import. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/P445MT5JGIHCRNBLX4WE6BD672QDHDJ3/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 Marian Csontos changed: What|Removed |Added Flags||needinfo?(ngomp...@gmail.co ||m) --- Comment #41 from Marian Csontos --- So the final version with separate boom-boot and boom-boot-conf. Neal, could you look at it and ack, so I can push to the repo. Downside is now we have 4 RPMs instead of 2, and repo metadata are growing... :-( Spec URL: https://raw.githubusercontent.com/csonto/fedpkg-boom-boot/split/boom-boot.spec SRPM URL: https://mcsontos.fedorapeople.org/boom-boot/boom-boot-0.9-3.fc29.src.rpm Scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=28333649 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/2XHLZ7S57D24BMHOZI467NUGP4JNBM2J/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #40 from Neal Gompa --- (In reply to Jason Tibbitts from comment #38) > Our rules are pretty clear, though. You don't name standalone applications > with a language-specific prefix. If you have something that's both an > application and a module then you are encouraged to split the module and the > application into separate (sub)packages. > > https://fedoraproject.org/wiki/Packaging: > Guidelines#Libraries_and_Applications Unfortunately, a lot of people have been doing the opposite and claiming this is supported by the guidelines. Hell, even I doubted myself and I don't do it because it doesn't even make sense... -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/34N3OIEDMWN2UGP6J6NCYXPD7URGZLXR/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #39 from Mohan Boddu --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/boom-boot -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/K5C7GXC7Y2WCHVHTECYACH5FEXCSJAVK/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #38 from Jason Tibbitts --- Our rules are pretty clear, though. You don't name standalone applications with a language-specific prefix. If you have something that's both an application and a module then you are encouraged to split the module and the application into separate (sub)packages. https://fedoraproject.org/wiki/Packaging:Guidelines#Libraries_and_Applications -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/NJSQIQDKSQECI3K3NYSW5NPZSWEZ45TN/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #37 from Marian Csontos --- Now that's actually a valid concern and something which is bugging me too. After reviewing all those tools with modules, it is quite common to have a package name not start with python. I will give it some more thought over weekend while fedpkg repo is ready. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/RY4OD4X74H2UILX7N7X6P4WOCICIUDKF/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413
--- Comment #36 from Neal Gompa ---
(In reply to Marian Csontos from comment #35)
> We must be looking at different yum.spec files, as the one I have opened has
> everything in the package %{name}.lang, including %config, launcher, and
> python modules.
>
> I think dnf is the closest to what you have proposed and fitting boom, but
> IMHO that split is partially driven by having support for both python2 and
> python3, and in our case does not make that much sense.
>
> Also main package in dnf is only the launcher (and man page and
> bash-completion), this requiring either python2 or python3 module, and both
> those modules require the common conf subpackage - so conf is required by
> python module, and not the other way.
>
> May be antipattern was not the best choice. But in this case that pattern
> does not fit - the configuration is required by python module, so should be
> either included, or python module should require the configuration
> subpackage, just like dnf does.
>
> Big thanks for all your time!
I just think that it's "surprising" for people and they're not likely to find
the boom boot manager if it's called "python3-boom". Our rules say python
modules that other people are supposed to be able to use should be
`python3-`, but they are specifically silent on what you should do with
the rest of it.
Whatever...
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives:
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/3MV3SF6SNCNSRPQ5GN2OTKFMT5WVNDJ3/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413
--- Comment #35 from Marian Csontos ---
We must be looking at different yum.spec files, as the one I have opened has
everything in the package %{name}.lang, including %config, launcher, and python
modules.
I think dnf is the closest to what you have proposed and fitting boom, but IMHO
that split is partially driven by having support for both python2 and python3,
and in our case does not make that much sense.
Also main package in dnf is only the launcher (and man page and
bash-completion), this requiring either python2 or python3 module, and both
those modules require the common conf subpackage - so conf is required by
python module, and not the other way.
May be antipattern was not the best choice. But in this case that pattern does
not fit - the configuration is required by python module, so should be either
included, or python module should require the configuration subpackage, just
like dnf does.
Big thanks for all your time!
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives:
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/VO4A344DCFNE5FNXQNSSWVYUCDG3IOE6/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 Neal Gompa changed: What|Removed |Added Status|NEW |POST Flags|fedora-review? |fedora-review+ --- Comment #34 from Neal Gompa --- kiwi and yum are examples of this pattern that you call an antipattern, too. But it's not worth fighting this, since I think it doesn't matter what I say anyway.. PACKAGE APPROVED. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/3Y53RELW6FKO2CBB4IQUHLVI6PTIGPQ7/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #33 from Marian Csontos --- The only one I found is dnf, where the split could be driven by the fact both python 2 and 3 are supported. Either almost everyone is doing it wrong, or the split was never meant to be mandatory. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/OYPQ7DWE7ZSNMWTLKJZZSBYOX6JKYXDG/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #32 from Marian Csontos --- ...and quickly looking at dozens of those without python in name, I still have to find one which would do such a split, so it looks like what you are suggesting is an antipattern. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/LAGWSDTA2KN427GM6TH45UVNZI2JFWX7/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #31 from Marian Csontos --- (Actually, I searched only packages with python in name, and with config files. I admit there may be a package using python and having the launcher in a separate subpackage...) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/7R4NUNRCNLGBQE4CT6MNLDZ3PTPFW6Z2/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #30 from Marian Csontos --- I have made a search, and have not found single package which would made such an artificial splitting of launcher and configuration into a separate subpackage. OTOH there is a bunch of packages shipping launcher and configuration: python-arc python-bucky python-carbon python-fedmsg-genacls python-flower python-gencpp python-genlisp python-genpy python-ldaptor python-pycadf python-sippy python-tahrir python-virtualenvwrapper python-wstool I think the most prominent of them are python-virtualenvwrapper and fedora own python-fedmsg-genacls. (Actually python-pycadf does split config files into a -common subpackage, required by both python2 and python3 modules) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/2MRUECF5OGWAID63TMPSDGPZBGRQYOTK/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #29 from Marian Csontos --- It can be used as python module. The issue is the configuration is required by both the module and the wrapper. The split just does not make sense. And it is rather common for python apps to include the launcher, it is even covered in packaging guidelines - https://fedoraproject.org/wiki/Packaging:Python#Executables_in_.2Fusr.2Fbin -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/WEH6FOFCTHZEG7HMU3O6ZCS3U4YW2WYW/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 Neal Gompa changed: What|Removed |Added Flags|needinfo?(ngomp...@gmail.co | |m) | --- Comment #28 from Neal Gompa --- Please don't call it python3-boom unless it actually can be used as a Python module. If it is supposed to be able to be used as a Python module, then only the module sources should be in python3-boom, and the rest in boom-boot, which would depend on python3-boom. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/TLGALYJIKRBLVRU24L7KM7XKNCHJFEYV/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 Marian Csontos changed: What|Removed |Added Flags||needinfo?(ngomp...@gmail.co ||m) --- Comment #27 from Marian Csontos --- Neal, could you please check the changes? Is there anything else what needs addressing? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/C6H3EP3EGPSXZVGNF3TE2RD2LMNDKLZI/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #26 from Marian Csontos --- Spec URL: https://raw.githubusercontent.com/csonto/fedpkg-boom-boot/split/boom-boot.spec SRPM URL: https://mcsontos.fedorapeople.org/boom-boot/boom-boot-0.9-2.fc29.src.rpm Scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=27953249 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/XJGXJYRDAAS6VRKPVSLNDNLCN7UY5LZP/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #25 from Marian Csontos --- ...except now I remember another reason why I made single package - I am not sure how much sense it makes to stuff anything into the main boom-boot package. It is the python library which make use of the configuration, profiles and creates BLS entries - and my original solution had only two packages. I think that split is rather artificial - it would be only the "script" and man page - and these are, when provided, commonly packaged with python libraries. I can spin off the grub2 part, but that would be only the grub.d and default/boom file - but it does not make much sense to have them on system without grub2. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/6ROTE6JKKEXF53NQLIB73WZNN2BOUMEF/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #24 from Marian Csontos --- Splitting it into more subpackages and using rich dependencies was actually one of the earlier ideas[1], but then I decided to go with plain 1 RPM, as I thought the simpler spec would be easier to review and maintain... [1]: https://github.com/csonto/boom/blob/master/boom.spec Do you see any other issues with the package? I will update spec with subpackages to see how that looks like -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/PK2HCRJGMC6UHFVROPSXJ2NCVIUNMMY3/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #23 from Neal Gompa --- Yeah, I've got a couple of concerns: * Why are we stuffing everything into a "python3-boom" subpackage? Why can't most of the "bootloader content" be in the main "boom-boot" package? * Since boom can be used with grub2 or sd-boot, shouldn't the grub2 files be in their own subpackage that requires grub2 to be installed? Then you can use a rich Supplements statement to install it automatically as needed. For example: "Supplements: (boom-boot and grub2)" on "grub2-boom" (or "grub2-boom-boot" or "boom-boot-grub2") will ensure it's auto-installed in the event grub2 is on the system. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/JNK46CVHNRZBJJAWYCIHB6J3SHQ3JSPL/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #22 from Marian Csontos --- Hi Neal, have you found some time to look at this? NOTE: There is a new boom version 0.9 - this is the same as the previously built version except the version and one small patch. Spec URL: https://raw.githubusercontent.com/csonto/fedpkg-boom-boot/master/boom-boot.spec SRPM URL: https://mcsontos.fedorapeople.org/boom-boot/boom-boot-0.9-1.fc29.src.rpm Scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=27894771 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/KPSCPEMW55RUB5B5NBWDZO5I2C3ZCIFY/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #21 from Marian Csontos --- Hi Neal, I updated the spec file, made a scratch build, and uploaded rpm and srpm to below locations: Spec URL: https://raw.githubusercontent.com/csonto/fedpkg-boom-boot/master/boom-boot.spec SRPM URL: https://mcsontos.fedorapeople.org/boom-boot/boom-boot-0.8.5-6.2.fc29.src.rpm Scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=27490751 - wouldbe fedpkg git repo is here: https://github.com/csonto/fedpkg-boom-boot - the fresh srpm and rpm results: https://mcsontos.fedorapeople.org/boom-boot complete rpmlint output: python3-boom.noarch: W: spelling-error %description -l en_US systemd -> systems, system, system d python3-boom.noarch: W: spelling-error %description -l en_US bls -> lbs, bl, ls python3-boom.noarch: W: non-etc-or-var-file-marked-as-conffile /boot/boom/boom.conf python3-boom.noarch: E: non-standard-dir-perm /boot/boom/profiles 700 python3-boom.noarch: E: non-standard-dir-perm /boot/loader/entries 700 1 packages and 0 specfiles checked; 2 errors, 3 warnings. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/I2WPHNH25STBHWI6L72ZJEDLNH2BK4GI/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #20 from Neal Gompa --- I'm okay with the changes proposed so far, so please get up an updated spec and SRPM so I can review it. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/S53JUCRPR5FKSIAXSKFGAW53NYQDYRZL/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #19 from Neal Gompa --- For what it's worth, I'm okay with boom.conf being shipped in /boot for now. We already do this in GRUB 2, and everyone here knows that both GRUB 2 and Boom will eventually need to be fixed. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/U3C4PL6N2LRSUJNIRZZDAIJWWP5FWMWZ/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #18 from Bryn M. Reeves --- Right: the profiles can be added by the user (although we do need the user to tell us the utsname pattern in that case, since it cannot be guessed from os-release data). For boom.conf itself, if it causes such a headache, we can just make it optional: the only options a regular user would currently want to set there is enabling the legacy grub1 support (this is principally aimed at users migrating from distributions like RHEL6). The only other options are to control the location of /boot and /boot/boom themselves: these are principally intended for debugging and testing the software and are used in conjunction with -c/--config. Given that Fedora is well past grub1 at this point we aren't really gaining anything by shipping a default config file (this does not need any change to boom itself). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/S22KDHLGD5UVI2UGUB6S4KQQG7KJF2KW/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #17 from Marian Csontos --- Let's get this to users. I have removed profiles from /boot/boom/profiles/ - these will be created by CLI on demand. These are data, and after all should not be packaged. But /boot/boom/boom.conf is supposed to be shared among distributions and kept outside of snapshots. If there were a better destination for the machine wide config file, as there is not, we would be happy to use that. Precedent is grub2 doing the same. Also where grub2 fails, is where it is not doing the same - as each OS is rewriting the file on its own. boom config file should be created once and changed by user, not by some generator. Is this a show stopper? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org/message/JTGX67JOTFFX37UZKEEDPWKUW3YTDDQ7/
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 Bryn M. Reeves changed: What|Removed |Added Flags|needinfo?(b...@redhat.com) | --- Comment #16 from Bryn M. Reeves --- > The reason I do it is because everything from EFI files (shim and grub2), > GRUB > module files, to kernel images, to stuff like this is being installed there, > and the rpmdb isn't happy when those files are "mysteriously missing" or > "wrong". This isn't something that will happen soon, so it's really out of scope for this review request, but in the future we would like to see a situation where /boot is treated as a cache, and the corresponding packages install their files to another location under the root file system. There's actually some progress toward this in other distros, as well as other boot management tools, but it's not something that the majority of distributions are going to be able to switch over to in short order. When /boot is viewed in this way the management of files there becomes much easier and with less potential for conflict between multiple instances that are sharing the same volume. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #15 from Neal Gompa --- (In reply to Marian Csontos from comment #9) > I think snpshots of /boot has its own set of problems. One of snapper's > flaws when using BTRFS is exactly that - it keeps information about > snapshots inside the filesystem which is "snapshotted". > I wouldn't be snapshotting /boot at the same time as / if it weren't for the fact that we deliberately install crap there. The reason I do it is because everything from EFI files (shim and grub2), GRUB module files, to kernel images, to stuff like this is being installed there, and the rpmdb isn't happy when those files are "mysteriously missing" or "wrong". > I wonder, if you have multiple snapshots of /boot, where do you keep > information about those snapshots and grub.cfg to pick the > snapshot+kernel+initrd+cmdline to boot? > You're right that this is a problem. I currently have the information on ESP, but I don't like it there. It'd be cleaner if stuff wasn't getting installed into /boot directly and everything in there was something that was recoverable from the main filesystem. Then I could leave /boot alone. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 Marian Csontos changed: What|Removed |Added Flags||needinfo?(b...@redhat.com) --- Comment #14 from Marian Csontos --- Bryn, after all, boom packaging could use different locations for the files in /boot: */boot/boom/boom.conf* => /etc/boom.conf || /etc/boom/boom.conf: This specifies boot_root, which is where the BLS entries will be written, and it may be different from /boot in the OS instance - e.g. user may want to create BLS entries into different path than /boot - e.g. IMO this could solve the BTRFS usecase where /boot is part of the /, and it would use different partition/path for BLS entries. */boot/boom/profiles/*.profile* => /dev/null: Profiles go to %(boom_root)s/profiles/, which is configurable, so IMO these should be created on demand only. Also these will be outdated most of the time (and already are). Let's keep them as examples only in doc dir. What do you think? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #13 from Bryn M. Reeves --- > If Boom cannot handle this, then it needs to do something to make that work. I don't think that it's reasonable to expect boom to make things work that are explicitly outside the scope of the standard (BLS) that we are following. We can try to find ways to accommodate it to the best extent possible, and will look at doing so, but fundamentally we will always be limited by the applicability of the standard that we are adopting. > My understanding is that boom doesn't do any _real_ boot management > functions, > and instead just generates configuration for loaders like GRUB2, rEFInd, or > sd- > boot to use instead. Define "real boot management" ;-) The purpose of this is as documented: to allow *any* booted instance of the OS (including snapshots, which may have been taken prior to the creation of other managed instances) to manage *any other*. This can only be done if there is a single, shared system-wide source for this configuration (which is exactly what the BLS sets out to create). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #12 from Alasdair Kergon --- 1) document more clearly? 2) have the tool recognise this situation and provide helpful messages? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #11 from Bryn M. Reeves --- It's also worth noting that the boot-as-directory-of-root scenario is somewhat incompatible with the aims of BLS: to provide a single, system-wide boot volume. If it's a directory of one particular OS instance it can never fulfil this role. It's possible that we may simply need to rule this configuration out as supportable and to make that clear in our documentation and guidance. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #10 from Bryn M. Reeves --- > My own setup has /boot as part of the OS disk that is snapshotted with the > rest > of the OS We're aware of this limitation and are trying to find long-term solutions, however, even before this there's a bigger problem with this configuration: the grub2 BLS support is fundamentally broken in this scenario (since it is unable to recognise that it must search within the boot/ directory contained within the root device). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #9 from Marian Csontos --- I think snpshots of /boot has its own set of problems. One of snapper's flaws when using BTRFS is exactly that - it keeps information about snapshots inside the filesystem which is "snapshotted". I wonder, if you have multiple snapshots of /boot, where do you keep information about those snapshots and grub.cfg to pick the snapshot+kernel+initrd+cmdline to boot? And if you boot into a snapshot, and want to make an update to that config, where to save that information? Either you still keep data inside "master" snapshot of /boot, or you need to keep it up to date somehow, don't you? Or do you move the problem to EFI boot partition or elsewhere? And than one can not "snapshot" that, and files are saved/installed there too. And should the FS get dirty, how does one fsck it? Of course /boot can get dirty itself too, but it is rather low traffic, compared to /. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #8 from Neal Gompa --- Alasdair, This is fundamentally flawed. My own setup has /boot as part of the OS disk that is snapshotted with the rest of the OS. The reason being is precisely because we install files into /boot. If Boom cannot handle this, then it needs to do something to make that work. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #7 from Alasdair Kergon --- From the docs https://github.com/bmr-cymru/boom : > Boom configuration data is stored in the /boot file system to permit > the tool to be run from any booted instance of any installed operating system. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #6 from Alasdair Kergon --- Have you tried out using the package to swap between different environments or snapshots? Whichever environment you choose to boot into - with different instances of root volumes and data etc. - you need to be able to see the current version of all the boom data. So there isn't really anywhere else to put /boot/boom unless you're going to start imposing filesystem layouts on people which would defeat part of the reason for using boom. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #5 from Neal Gompa --- Marian, Is there a solid reason for these things being in /boot as content managed by the package? Can they not be installed in more normal locations? My understanding is that boom doesn't do any _real_ boot management functions, and instead just generates configuration for loaders like GRUB2, rEFInd, or sd-boot to use instead. Thus, I don't see why I should allow non-compliant stuff like this like we do for GRUB 2. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #4 from Marian Csontos --- Scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=26891979 SRPM: https://mcsontos.fedorapeople.org/boom-boot/ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #3 from Marian Csontos --- I have made the package slightly more non-compliant, adding these: python3-boom.noarch: W: non-etc-or-var-file-marked-as-conffile /boot/boom/boom.conf python3-boom.noarch: W: non-etc-or-var-file-marked-as-conffile /boot/boom/profiles/21e37c8002f33c177524192b15d91dc9612343a3-ubuntu16.04.profile python3-boom.noarch: W: non-etc-or-var-file-marked-as-conffile /boot/boom/profiles/3fc389bba581e5b20c6a46c7fc31b04be465e973-rhel7.2.profile python3-boom.noarch: W: non-etc-or-var-file-marked-as-conffile /boot/boom/profiles/9736c347ccb724368be04e51bb25687a361e535c-fedora25.profile python3-boom.noarch: W: non-etc-or-var-file-marked-as-conffile /boot/boom/profiles/98c3edb94b7b3c8c95cb7d93f75693d2b25f764d-rhel6.profile python3-boom.noarch: W: non-etc-or-var-file-marked-as-conffile /boot/boom/profiles/9cb53ddda889d6285fd9ab985a4c47025884999f-fedora24.profile python3-boom.noarch: W: non-etc-or-var-file-marked-as-conffile /boot/boom/profiles/c0b921ea84dbc5259477cbff7a15b000dd222671-rhel7.profile python3-boom.noarch: W: non-etc-or-var-file-marked-as-conffile /boot/boom/profiles/d4439b7d2f928c39f1160c0b0291407e5990b9e0-fedora26.profile python3-boom.noarch: W: non-etc-or-var-file-marked-as-conffile /boot/boom/profiles/efd6d41ee868310fec02d25925688e4840a7869a-rhel4.profile Files in /boot must be treated as config files. These are supposed to be edited by user, and must be in /boot to ensure they are shared among all snapshots and/or operating systems. The precedent for this is grub2, which has the same reasons for placing files in /boot and using %config for those. I have also added Conflict, and dependencies (Requires: and Suggests:) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 Neal Gompa changed: What|Removed |Added CC||ngomp...@gmail.com Assignee|nob...@fedoraproject.org|ngomp...@gmail.com Flags||fedora-review? --- Comment #2 from Neal Gompa --- Taking this review. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1576413] Review Request: boom-boot - boot manager
https://bugzilla.redhat.com/show_bug.cgi?id=1576413 --- Comment #1 from Marian Csontos --- Benefit to Fedora: This package handles entries in /boot for both multiple linux distributions and for snapshots of root filesystem. This is for those old fashioned folks who do not run containers, atomic and ostree... -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
