buildscript: don't error out when running as root #299

jkimmel · 2024-01-19T22:26:57+01:00

jkimmel commented

2024-01-19 22:26:57 +01:00

Ein bisschen Kontext zum PR:

Der Pull Request wurde als Demonstration verfasst, um den Workflow mit git etwas zu veranschaulichen.

Der Prozess und manche Kommentare waren unterhaltsam gestaltet

Das Ergebnis ist aber durchaus ernst gemeint :)

Signed-off-by: Johannes Kimmel fff@bareminimum.eu

> Ein bisschen Kontext zum PR: > - Der Pull Request wurde als Demonstration verfasst, um den Workflow mit git etwas zu veranschaulichen. > - Der Prozess und manche Kommentare waren unterhaltsam gestaltet > - Das Ergebnis ist aber durchaus ernst gemeint :) Signed-off-by: Johannes Kimmel <fff@bareminimum.eu>

jkimmel added 1 commit 2024-01-19 22:26:57 +01:00

ci/woodpecker/pr/woodpecker Pipeline is pending Details

e2cc7fc21d buildscript: don't error out when running as root

Signed-off-by: Johannes Kimmel <fff@bareminimum.eu>

rohammer commented

2024-01-19 22:30:18 +01:00

das ist doch Kacke!!! Das bricht später so und so!

👍 1

McUles commented

2024-01-19 22:31:04 +01:00

First-time contributor

das ist doch Kacke!!! Das bricht später so und so!

Immer hat einer was zu meckern -.-

> das ist doch Kacke!!! Das bricht später so und so! Immer hat einer was zu meckern -.-

👎 1

fbl commented

2024-01-19 22:32:42 +01:00

Gute Sache, diese Bevormundung sollte weg.

Ich fürchte aber, dass das Bauen von OpenWrt als root trotzdem nicht klappen wird. Als ich das das letzte mal probiert habe hat sich irgendeins von den hosttools beim kompilieren beschwert, dass es nicht als root gebaut werden möchte.

Das kriegt man aber leider nur bei einem debug-build raus; ohne debug sieht man dann nur dass das tool nicht kompiliert hat.
Daher würde ich diese Meldung gerne beibehalten. Wir können es aber gerne in eine Warnung (am besten mit Farbe 🤩) umwandeln, damit man nicht von unseren Tools behindert wird, sollte dieses Problem upstream mal gelöst werden.

Gute Sache, diese Bevormundung sollte weg. Ich fürchte aber, dass das Bauen von OpenWrt als root trotzdem nicht klappen wird. Als ich das das letzte mal probiert habe hat sich irgendeins von den hosttools beim kompilieren beschwert, dass es nicht als root gebaut werden möchte. Das kriegt man aber leider nur bei einem debug-build raus; ohne debug sieht man dann nur dass das tool nicht kompiliert hat. Daher würde ich diese Meldung gerne beibehalten. Wir können es aber gerne in eine Warnung (am besten mit Farbe 🤩) umwandeln, damit man nicht von unseren Tools behindert wird, sollte dieses Problem upstream mal gelöst werden.

👎 1

jkimmel added 2 commits 2024-01-19 22:40:59 +01:00

e70d4a538c Revert "buildscript: don't error out when running as root"

This reverts commit e2cc7fc21d.

ci/woodpecker/pr/woodpecker Pipeline is pending Details

25114a1812 buildscript: print warning instead of failing build

Signed-off-by: Johannes Kimmel <fff@bareminimum.eu>

jkimmel added 1 commit 2024-01-19 22:56:38 +01:00

ci/woodpecker/pr/woodpecker Pipeline is pending Details

802aedeb43 buildscript: add color to the root warning

Signed-off-by: Johannes Kimmel <fff@bareminimum.eu>

jkimmel commented

2024-01-19 22:58:11 +01:00

Jetzt mit Farbe:

Jetzt mit Farbe: ![root.png](/attachments/23c2de31-497d-48c0-a7e7-e6ce083132c5)

root.png

14 KiB

jkimmel force-pushed dont-check-root from 802aedeb43 to 2cbe37a14b

2024-01-19 23:05:52 +01:00

Compare

fbl commented

2024-01-19 23:08:01 +01:00

Super, das finde ich gut.
nitpick: echo -e ist nicht POSIX compliant. Alternativ könnte man printf verwenden, das müsste POSIX-kompatibel sein.

Reviewed-by: Fabian Bläse <fabian@blaese.de>

Super, das finde ich gut. nitpick: `echo -e` ist nicht POSIX compliant. Alternativ könnte man `printf` verwenden, das müsste POSIX-kompatibel sein. `Reviewed-by: Fabian Bläse <fabian@blaese.de>`

jkimmel force-pushed dont-check-root from 2cbe37a14b to 6860eee843

2024-01-22 02:54:44 +01:00

Compare

jkimmel commented

2024-01-22 02:55:55 +01:00

So.. jetzt aber hoffentlich in richtig und portabel.