tools/dep-tree: fix broken tool #126

rohammer · 2021-02-17T15:54:08+01:00

rohammer commented

2021-02-17 15:54:08 +01:00

With commit 1946aaca87 the variantselection has changed.
This adapts dep-tree to the new situation.

Additional:

The variant can now passed by as an option.
A help text is added.

Signed-off-by: Robert Langhammer rlanghammer@web.de

With commit 1946aaca87 the variantselection has changed. This adapts dep-tree to the new situation. Additional: * The variant can now passed by as an option. * A help text is added. Signed-off-by: Robert Langhammer <rlanghammer@web.de>

adschm reviewed 2021-02-17 18:22:44 +01:00

tools/dep-tree Outdated

						
				@ -5,1 +14,3 @@

				# variant=$(cat selected_variant)

				eval "$(grep "^FFF_VARIANTS=" buildscript)"

				variant=$(cat selected_variant)

				[ $# = 0 ] || variant="$@"

adschm commented

2021-02-17 18:22:44 +01:00

Hatten wir nicht jüngst die Diskussion, dass "$@" in "$1" "$2" etc. expandiert wird?

rohammer commented

2021-02-17 19:12:45 +01:00

Ja, hatten wir. Darum musste ich hier auch nicht lange ueberlegen.

fbl approved these changes 2021-03-02 01:15:13 +01:00

fbl left a comment

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

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

tools/dep-tree Outdated

						
				@ -3,3 +13,2 @@

				builddir=./build

				[ -f selected_bsp ] && . selected_bsp

				# variant=$(cat selected_variant)

				eval "$(grep "^FFF_VARIANTS=" buildscript)"

fbl commented

2021-03-02 01:13:15 +01:00

Hmm. eval ist irgendwie immer gruselig..

Hmm. `eval` ist irgendwie immer gruselig..

rohammer commented

2021-03-02 11:49:50 +01:00

Oh, ich wollte niemanden erschrecken!
Man koennte Adrians Vorschlag aufgreifen und die Varianten hard coden. Es wird ja nur im usage verwendet. Und so haeufig wird keine neue Variante dazu kommen.

Oh, ich wollte niemanden erschrecken! Man koennte Adrians Vorschlag aufgreifen und die Varianten hard coden. Es wird ja nur im usage verwendet. Und so haeufig wird keine neue Variante dazu kommen.