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.

Sign up for GitHub

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

Jump to bottom

don't hardcode grub #100

Closed
lnussel opened this issue Mar 23, 2023 · 2 comments
Closed

don't hardcode grub #100

lnussel opened this issue Mar 23, 2023 · 2 comments

Comments

@lnussel
Copy link
Member

lnussel commented Mar 23, 2023

lib/Transaction.cpp wants to iterate over /boot/grub2 and fails if it doesn't exist. Would be good to make all places that somehow assume grub fail gracefully :)
@lnussel
Copy link
Member Author

lnussel commented May 3, 2023

any update on this?

@laenion
Copy link
Collaborator

laenion commented May 9, 2023

Fixed in 49f308e / transactional-update 4.2.0.

@laenion laenion closed this as completed May 9, 2023
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this issue May 10, 2023
@bmwiedemann
Update transactional-update to version 4.2.1 / rev 99 via SR 1085717
be35c24 
https://build.opensuse.org/request/show/1085717
by user fos + dimstar_suse
- Version 4.2.1
  - Implement "apply" command to switch into new snapshot directly
    [jsc#PED-3912]
  - Use new snapper functionality to set default snapshot - this
    makes it possible to execute hooks as requested in
    [poo#127160], [gh#openSUSE/transactional-update#85] and
    [gh#openSUSE/transactional-update#105].
  - Don't hardcode GRUB2 [gh#openSUSE/transactional-update#100] /
    [poo#127154]
  - Fix cleanup handler
  - Fix unmounting temporary mounts
  - Prevent loosing track of snapshots in certain rollback
    scenarios - these would not be marked for deletion otherwise
  - Document "notify" reboot method
- Minimal required snapper version is 0.8.10 now, for the new
  snapper functionality there is a backwards compatibility layer.
- Conflict with health-checker < 1.8 - the "
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@lnussel @laenion