particle-os/debian-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

osbuild-mpp: conditional losetup

Browse source 
`mpp-define-images` can create an image file, using `losetup` to deal
with non-standard sector sizes requires root. Not all users run
`osbuild-mpp` as root.

While I am not a fan of "suddenly sudo" based on the input manifest this
does alleviate builds breaking for manifests with default sector sizes
when non-root.
This commit is contained in:
Simon de Vlieger 2023-11-30 11:40:20 +01:00
parent 0a2e0bb3d2
commit b225d1cf04
1 changed files with 19 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

28
tools/osbuild-mpp
View file

@ -968,6 +968,8 @@ class PartitionTable:
class Image:
DEFAULT_SECTOR_SIZE = 512
def __init__(self, size, layout):
self.size = size
self.layout = layout
@ -976,19 +978,27 @@ class Image:
def from_dict(cls, js):
size = js["size"]
data = js["table"]
sector_size = js.get('sector_size', 512)
sector_size = js.get('sector_size', Image.DEFAULT_SECTOR_SIZE)
with tempfile.TemporaryDirectory() as tmp:
image = os.path.join(tmp, "disk.img")
subprocess.run(["truncate", "--size", size, image], check=True)
cp = subprocess.run(["losetup", "--find", "--show", f"--sector-size={sector_size}", image],
stdout=subprocess.PIPE, check=True)
loopimage = cp.stdout.rstrip()
try:
table = PartitionTable.from_dict(data)
table.write_to(loopimage)
finally:
subprocess.run(["losetup", "-d", loopimage])
table = PartitionTable.from_dict(data)
# Running losetup requires to be root but losetup is only necessary if the sector size
# is different from the default.
if sector_size == Image.DEFAULT_SECTOR_SIZE:
table.write_to(image)
else:
cp = subprocess.run(["losetup", "--find", "--show", f"--sector-size={sector_size}", image],
stdout=subprocess.PIPE, check=True)
loopimage = cp.stdout.rstrip()
try:
table.write_to(loopimage)
finally:
subprocess.run(["losetup", "-d", loopimage])
return cls(size, table)