Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DietPi-Config | Adjust IPv related settings #1386

Merged
merged 4 commits into from
Jan 12, 2018
Merged

Conversation

MichaIng
Copy link
Owner

@MichaIng MichaIng added this to the v6.0 milestone Jan 11, 2018
@MichaIng MichaIng self-assigned this Jan 11, 2018
@MichaIng MichaIng requested review from Fourdee and k-plan January 11, 2018 13:31
+ DietPi-Prep | Remove doubled IPv4 settings
Copy link
Contributor

@k-plan k-plan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MichaIng

  • looks good for me, but Dan and your "coding stuff" do overstrained me. Can't really check it for syntax errors. Sorry, I only will notice, if something in the end doesn't work as expected.

+ DietPi-Globals | G_DIETPI-NOTIFY: Final hardenings of processing notifications
@MichaIng
Copy link
Owner Author

MichaIng commented Jan 12, 2018

Ah, I accidentally added the last commit to this PR branch. Should have become a separate PR:

DietPi-Globals | G_DIETPI-NOTIFY: Final hardenings of processing notifications

And I double researched and checked: ainput_string, defined as local variable, will be definitely removed when G_DIETPI-NOTIFY function ends, so no manual unset necessary here (as well as aprocess_string) 😉.

@MichaIng MichaIng mentioned this pull request Jan 12, 2018
23 tasks
+ DietPi-Globals | G_DIETPI-NOTIFY: Further minor improvements
Copy link
Collaborator

@Fourdee Fourdee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Fourdee
Copy link
Collaborator

Fourdee commented Jan 12, 2018

@MichaIng

Looks good, many thanks 👍

Merged for testing.

@Fourdee Fourdee merged commit 85589b4 into testing Jan 12, 2018
@Fourdee
Copy link
Collaborator

Fourdee commented Jan 12, 2018

@MichaIng
Just noticed this throughout Sparky SBC Stretch using PREP

[  OK  ] Step 7: Prep system for DietPi ENV:
[ INFO ] ./dietpi-globals: line 201: printf: --: invalid option
printf: usage: printf [-v var] format [arguments]

###

[  OK  ] CPU gov applied: performance


[ INFO ] ./dietpi-globals: line 201: printf: --: invalid option
printf: usage: printf [-v var] format [arguments]

###

[  OK  ] Step 8: Finalise system for first run of DietPi:
[ INFO ] ./dietpi-globals: line 201: printf: --: invalid option
printf: usage: printf [-v var] format [arguments]

@MichaIng MichaIng deleted the adjust_ipv_selection branch January 12, 2018 13:30
@MichaIng
Copy link
Owner Author

MichaIng commented Jan 12, 2018

@Fourdee
Jep, can replicate:

 printf '-----------------------------------'
-bash: printf: --: invalid option
printf: usage: printf [-v var] format [arguments]
root@VM-Stretch:~# echo -ne '-----------------------------------'
-----------------------------------

printf seem to not like printing a hyphen line. Actually a hyphen as first character seems to be interpreted as option, even with single quotes:

printf '-a---------------------------------'
-bash: printf: -a: invalid option
printf: usage: printf [-v var] format [arguments]

And people say, that printf is more predictable/reliable than echo 😆. I will revert (everything, just in case) to echo -ne, never change a running system...
€: Done: https://github.com/Fourdee/DietPi/pull/1389


But besides:
Do you wan't to keep the step headers printed as info and ok? Maybe G_DIETPI-NOTIFY 3 with e.g. capital letters and an extra line (to make it dominant compared to other dietpi scripts starting)?

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

Successfully merging this pull request may close these issues.

3 participants