You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I mentioned this in #21, but I think this might actually deserve independent discussion, hence this issue.
Rpi-gen currently supports systems that use direct device names (e.g. /dev/mmcblk0p1) to reference partitions in cmdline.txt and fstab. This supports consists of two parts:
The --edit-fstab edit that tells rpi-clone about the new device name and makes it replace reference to the old device name with reference to the new device name (and some code to do this automatically when cloning back to an SD card for some reason that is unclear to me).
The --convert-fstab-to-partuuid option, that replaces direct references to the device partitions with PARTUUID references, which are more stable and easier to manage.
However, raspbian started using PARTUUIDs in 2017, so I expect these options are likely not used and not really needed anymore. Also other possibly supported systems such as Ubuntu and Armbian use UUIDs (filesystem UUIDS instead of PARTUUIDs, which need some small changes to support, but these are at least not raw device names).
So would suggest considering to remove these features, which would significantly simplify the some parts of the code which helps with readability and make future changes easier. Would that be something you would accept a PR for?
Two reasons people might still use these options are if they are running a real old (>6 years) system, or when they are adding extra partitions manually to fstab. In the former case, people are likely already running rpi-clone and can just stick to their current version. In the latter case, people can just manually add/convert to PARTUUIDs instead of raw device names (so this is a matter of documentation, and maybe a runtime check/warning to point people at the documentation).
Note that in #21 I only talked about --convert-fstab-to-partuuid, but looking through the code more, I realized that the same holds for --edit-fstab.
The text was updated successfully, but these errors were encountered:
I mentioned this in #21, but I think this might actually deserve independent discussion, hence this issue.
Rpi-gen currently supports systems that use direct device names (e.g.
/dev/mmcblk0p1
) to reference partitions incmdline.txt
andfstab
. This supports consists of two parts:--edit-fstab
edit that tells rpi-clone about the new device name and makes it replace reference to the old device name with reference to the new device name (and some code to do this automatically when cloning back to an SD card for some reason that is unclear to me).--convert-fstab-to-partuuid
option, that replaces direct references to the device partitions withPARTUUID
references, which are more stable and easier to manage.However, raspbian started using PARTUUIDs in 2017, so I expect these options are likely not used and not really needed anymore. Also other possibly supported systems such as Ubuntu and Armbian use UUIDs (filesystem UUIDS instead of PARTUUIDs, which need some small changes to support, but these are at least not raw device names).
So would suggest considering to remove these features, which would significantly simplify the some parts of the code which helps with readability and make future changes easier. Would that be something you would accept a PR for?
Two reasons people might still use these options are if they are running a real old (>6 years) system, or when they are adding extra partitions manually to
fstab
. In the former case, people are likely already running rpi-clone and can just stick to their current version. In the latter case, people can just manually add/convert to PARTUUIDs instead of raw device names (so this is a matter of documentation, and maybe a runtime check/warning to point people at the documentation).Note that in #21 I only talked about
--convert-fstab-to-partuuid
, but looking through the code more, I realized that the same holds for--edit-fstab
.The text was updated successfully, but these errors were encountered: