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.

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

More failsafe 200_partition_layout.sh #2802

Closed
wants to merge 2 commits into from

Conversation

jsmeix
Copy link
Member

@jsmeix jsmeix commented May 10, 2022

In layout/save/GNU/Linux/200_partition_layout.sh
ensure the partition name/type entry in disklayout.conf
is always set (and percent encoded if needed)
at least it is set to the fallback value 'rear-noname'
regardless of the 'disk_label' value

In layout/save/GNU/Linux/200_partition_layout.sh
ensure the partition name/type entry in disklayout.conf
is always set (and percent encoded if needed)
at least it is set to the fallback value 'rear-noname'
regardless of the 'disk_label' value, cf.
#2801 (comment)
@jsmeix jsmeix added the enhancement Adaptions and new features label May 10, 2022
@jsmeix jsmeix added this to the ReaR v2.7 milestone May 10, 2022
@jsmeix jsmeix self-assigned this May 10, 2022
@jsmeix jsmeix changed the title Update 200_partition_layout.sh More failsafe 200_partition_layout.sh May 10, 2022
In layout/save/GNU/Linux/200_partition_layout.sh
ensure $disk_label is one of the supported partition tables
cf. #2801 (comment)
@jsmeix
Copy link
Member Author

jsmeix commented May 10, 2022

My attempt here makes things even worse!

That old code is so terribly obfuscated
that it is basically impossible to fix things
with reasonable effort.

@jsmeix jsmeix closed this May 10, 2022
@jsmeix jsmeix removed the request for review from a team May 10, 2022 08:36
@jsmeix jsmeix deleted the jsmeix-partition_layout-more-failsafe branch May 10, 2022 08:52
@jsmeix
Copy link
Member Author

jsmeix commented May 10, 2022

Now its sucessor #2803
tries to avoid issues like
#2801
by ensuring a supported partition table is found
and if not it errors out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant