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

Beta v6.22.2 #2643

Merged
merged 28 commits into from
Mar 21, 2019
Merged

Beta v6.22.2 #2643

merged 28 commits into from
Mar 21, 2019

Conversation

Fourdee
Copy link
Collaborator

@Fourdee Fourdee commented Mar 19, 2019

Beta v6.22.2

(22/03/19)

As always, many smaller code performance and stability improvements, visual and spelling fixes have been done, too much to list all of them here. Check out all code changes of this release on GitHub: https://github.com/MichaIng/DietPi/pulls?q=is%3Aclosed+base%3Amaster

Changes / Improvements / Optimisations (since v6.22.1)

Known/Outstanding Issues

For all additional issues that may appear after release, please see the following link for active tickets: https://github.com/MichaIng/DietPi/issues

MichaIng and others added 12 commits March 17, 2019 03:32
+ Improve code to check for forwarding/proxy etc.
+ WIP dietpi-banner changes.

NB: need to add back in reobtain IP details if not found at time.
+ Tweaks
+ Minor tweak
+ - DietPi-Banner | You can now customise which options to display during system login. These include CPU temps, WAN IP, free disk space and more! Simply run 'dietpi-banner': #2627

+ G_CHECK_FREESPACE (info_autoscale). Switched to MB as thats what G_CHECK_FREESPACE  uses, and, throughout DietPi I believe.
+ dietpi-banner hostname option #2627 (comment)
+ Minor ordering
+ Domainname: #2627
beta v2
@Fourdee Fourdee added the Beta 🧪 Issues specific to the Beta branch testing label Mar 19, 2019
@Fourdee Fourdee added this to the v6.22 milestone Mar 19, 2019
@Fourdee
Copy link
Collaborator Author

Fourdee commented Mar 19, 2019

@MichaIng

dietpi-banner changes has passed local testing my end. Any chance we can merge this for v6.22 beta testing and release?

@Fourdee Fourdee requested a review from MichaIng March 19, 2019 15:02
+ Weather and uptime: #2627
@MichaIng
Copy link
Owner

@Fourdee
Jep since it's visual only I see no issue in adding this to v6.22 already.

There are two issues that I would like to resolve, but require your help due to lack of devices:

  • DietPi-Software | Kodi: Video playback freezes after few seconds on Odroid XU4 #2584
    If I got it right, you could verify freezing Kodi on your XU4 as well, regardless of using the default xorg.conf from the Odroid repo package? If we cannot resolve this currently, I think we need to forward to Meveric and add this to known issues for now.
  • Rock64 | 3.5mm audio jack not working #2522
    On Rock64 it looks like we need to enable overlays now for certain features like I2C, I2s, general 3.5mm jack output and such. Found the related steps from the armbian-config script (which we remove during DietPi-PREP), but needs testing if indeed the overlay files are at desired location, as on raw image (inspect via 7zip on Windows) this is not the case.

@MichaIng MichaIng changed the title Beta 2 Beta v6.22.2 Mar 19, 2019
+ myip.php | Minor alignment and wording
+ DietPi-Banner | Function reorder and minor coding
+ DietPi-Globals | G_CHECK_VALIDINT(): Failsafe: Assure $disable_error is either 1 or 0 and that "unset disable_error" always affects the non-local variable if this was set to 1
+ DietPi-Globals | G_CHECK_FREESPACE(): Fix MiB => GiB conversion (*1024) and minor coding
+ DietPi-Banner | Further code simplifications and consistency
@MichaIng
Copy link
Owner

MichaIng commented Mar 20, 2019

@Fourdee
Sorry for coding so much around your banner commit 😅. I was not happy with the inconstancy between aENABLED/DESCRIPTION and aOBTAIN arrays and first wanted to add all options to the aOBTAIN array with identical indices. Sadly $($var) does not support any kind of redirection or piping and even has issues with single quotes that I was not able to resolve. I guess this was also the reason why you left the affected obtain commands outside the array. As I still didn't like the non-matching array indices, I now removed aOBTAIN completely, which simplifies the script a bid.

Only reliable solution I found with array usage is to write the commands to a file and source/execute it back. But that's way too complicated IMO.

+ DietPi-Banner | Allow custom banner entry
+ DietPi-PreBoot | Moved Fire3 TTY2 workaround into PreBoot + tiny coding/wording
+ DietPi-PreBoot | Moved applying install stage on first boot into PreBoot, since no first run steps are done in DietPi-Boot anymore
+ DietPi-Boot | NanoPi Fire3 TTY2 workaround has been moved into DietPi-PreBoot; Moved applying install stage on first boot as well, since no first run steps are left anymore
+ DietPi-Patch | Skip single /dietpi/conf/ file removal on v6.18 => v6.19 since we remove the whole dir on v6.19 => v6.20
+ DietPi-Patch | v6.22 patch: Fix wrong mpd.conf path and wrong user data global variable call
+ DietPi-Patch | Minor coding
+ DietPi-Globals | Tiny coding/reorder + default RC version update
@Fourdee
Copy link
Collaborator Author

Fourdee commented Mar 21, 2019

@MichaIng

Sorry for coding so much around your banner commit

No worries, all good 👍

There are two issues that I would like to resolve, but require your help due to lack of devices:

The XU4, yep, we need to get in touch with Meveric. Will do that when I can.

Same for the Rock64, will run local tests and find the solution.

Regardless, the above can wait for v6.23. No need to hold up v6.22 for them?

@Fourdee
Copy link
Collaborator Author

Fourdee commented Mar 21, 2019

@MichaIng

Only reliable solution I found with array usage is to write the commands to a file and source/execute it back. But that's way too complicated IMO.

Yep, the plan with aOBTAIN was to allow us to re-obtain on the fly without needing to have multiple copies of the command in the script.
But yeah, obtain network was the only real one that we needed that for I believe.

@MichaIng MichaIng mentioned this pull request Mar 21, 2019
@Fourdee
Copy link
Collaborator Author

Fourdee commented Mar 21, 2019

@MichaIng

  • Custom banner entry | Hello World!

Very nice 👍 💘

+ DietPi-PreBoot | Syntax
+ DietPi-Patch | Syntax
+ DietPi-Banner | Make FQDN and NIS differentiation clearer
@MichaIng
Copy link
Owner

MichaIng commented Mar 21, 2019

@Fourdee

Yep, the plan with aOBTAIN was to allow us to re-obtain on the fly without needing to have multiple copies of the command in the script.

Currently there is only the main banner where we call these commands. When something in the future is obtained on different parts of the script, I would create a function instead. This would have been the other possibility btw:

Obtain_1(){ command 1; }
Obtain_2(){ command 2; }
...

And then call via $(Obtain_$i).


About domainname command:

Program name:
       {yp,nis,}domainname=hostname -y
       dnsdomainname=hostname -d
...
    -y, --yp, --nis        NIS/YP domain name

I changed the banner to now show the FQDN/hostname and NIS domainname, to make the difference clearer. If NIS/YP is not used, then FQDN will show everything relevant for most users. The short hostname is not too useful and already part of the console prompt string (PS1).

+ DietPi-Patch | Syntax
+ DietPi-PreBoot | Syntax
Copy link
Owner

@MichaIng MichaIng left a comment

Choose a reason for hiding this comment

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

Looks and works great for me. Will merge into Beta

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Beta 🧪 Issues specific to the Beta branch testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants