Skip to content

V0.4.1 rework#2

Draft
towo2099 wants to merge 5 commits into
v0.4.0from
v0.4.1_rework
Draft

V0.4.1 rework#2
towo2099 wants to merge 5 commits into
v0.4.0from
v0.4.1_rework

Conversation

@towo2099

Copy link
Copy Markdown
Member

Ich habe das Paket mal komplett überarbeitet.
Damit braucht man im Calamares das ganze Scripts-Gedöns nicht mehr machen, sondern dieses Paket konfiguuriert snapper so, wie von uns gewünscht.

Ich habe allerdings ALLOW_GROUPS="" statt ALLOW_GROUPS="users" gesetzt, weil letzteres eigentlich keine so dolle Idee ist

Was meint ihr dazu?

@towo2099 towo2099 requested a review from ak-li June 23, 2026 13:30
@ak-li

ak-li commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

@towo2099
Welches "Scripts-Gedöns " meinst du? Ich finde außer calamares-settings-siduction kein aktuelles Repo mit Bezug zu Calamares und dort gibt es nur ein Verzeichnis 'scripts' mit zwei Skripten und natürlich die calamares Module.

Die Änderungen mit Bezug zu systemd erscheinen sinnvoll, das kann ich aber nicht genau beurteilen.

ALLOW_GROUPS="users" hat den Sinn, dass der Benutzer snapper Aktionen ohne root-Rechte ausführen kann. Er kann das System (auch als root) mittels snapper Aktionen nicht zerstören. Das verhindert Btrfs.

Der gesamte Rest baut viele Fehler (wieder) in das Paket ein.

Was hat tuxedo da drin zu suchen?
Das Paket wurde von mir in langwieriger und mühevoller Arbeit entwickelt und ich verbitte mir die Übernahme durch tuxedo für dieses Paket. Von mir aus kann tuxedo das alles für sich verwenden, Aber in siduction-btrfs hat tuxedo nichts zu suchen.
Wenn du Verbesserungsvorschläge hast, die auf Arbeit von tuxedo basieren, können wir gerne einen Vermerk im copyright eintragen.

@towo2099

Copy link
Copy Markdown
Member Author

Was hat tuxedo da drin zu suchen?

Tuxedo hat in den Files gar nichts zu suchen, das waren Überbleibsel aus meinen Umbau-Arbeiten für Tuxedo, ist gefixt.

Welches "Scripts-Gedöns " meinst du?

modules/shellprozess:

script:
    - "-umount ${ROOT}.snapshots"
    - "-rm -r ${ROOT}.snapshots"
    - "-snapper --no-dbus create-config -f btrfs ${ROOT}"
    - "-snapper --no-dbus -c root set-config 'TIMELINE_CREATE=no'"
    - "-snapper --no-dbus -c root set-config 'ALLOW_GROUPS=users'"
    - "-snapper --no-dbus -c root set-config 'SYNC_ACL=yes'"
    - "-snapper --no-dbus -c home create-config -f btrfs ${ROOT}home"
    - "-snapper --no-dbus -c home set-config 'TIMELINE_CREATE=no'"
    - "-snapper --no-dbus -c home set-config 'ALLOW_GROUPS=users'"
    - "-snapper --no-dbus -c home set-config 'SYNC_ACL=yes'"
    - "-systemctl disable snapper-boot.timer"
    - "-systemctl disable snapper-timeline.timer"
    - "-umount ${ROOT}.snapshots"
    - "-btrfs sub del ${ROOT}.snapshots/"
    - "-mkdir ${ROOT}.snapshots"
    - "-sed -i 's|GRUB_BTRFS_DISABLE=|#GRUB_BTRFS_DISABLE=|g' ${ROOT}etc/default/grub-btrfs/config"

Das ist im Prinzip sogar falsch, weil modules/mount hat

    - mountPoint: /.snapshots
      subvolume: /@snapshots

gesetzt, was in meinen Augen falsch ist, das snapper create-config sowohl das subvolume, als auch das verzeichnis anlegt.
Durch dieses Setting landed .snapshots dann auch in der fstab und das hat da, glaube ich, nichts zu suchen.

Der gesamte Rest baut viele Fehler (wieder) in das Paket ein.

Lässt Du mich wissen, welche?

@towo2099

Copy link
Copy Markdown
Member Author

Das Paket wurde von mir in langwieriger und mühevoller Arbeit entwickelt und ich verbitte mir die Übernahme durch tuxedo für dieses Paket

Tuxedo will da nichts übernehmen. Ich habe das Paket geforkt und für TUXEDO OS umgebaut. Wenn Du als Autor mir sagts, wir dürfen das micht benutzen, dann müssen wir das akzeptieren und uns etwas anderes einfallen lassen.

Wenn wir es unter TUXEDO OS nur als siduction-btrfs einsetzen dürfen, dann kann ich das auch abklären.

@ak-li

ak-li commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Durch dieses Setting landed .snapshots dann auch in der fstab und das hat da, glaube ich, nichts zu suchen.

Das schaue ich mir genauer an. Das .snapshots Subvolumen einfach mounten zu können hat schon was für sich wenn mann in den Snapshots auf die Suche geht. Das Setup für die Konfig 'root' ist ohne natürlich einfacher.

Lässt Du mich wissen, welche?

Heute nicht mehr. Werden morgen ein paar Sachen aufzeigen.

@ak-li

ak-li commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Betr. Tuxedo
Natürlich darf Tuxedo das Paket benutzen. Ist und bleibt eine freie Lizenz und ist ja auch Sinn der Sache.
Mein Anliegen war, dass "Tuxedo OS" nichts in siduction-btrfs zu suchen hat (aber das hast du ja oben schon klargestellt), es sei denn wir bekommen einige Zeilen von Tuxedo dazu. Zum Beispiel den Krempel mit systemd.

@towo2099

Copy link
Copy Markdown
Member Author

Das .snapshots Subvolumen einfach mounten zu können hat schon was für sich wenn mann in den Snapshots auf die Suche geht.

Also ich kann da ohne fstab Eintrag z.B. per Midnightcommander drin rumsuchen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants