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

changed instructions to switch branch #6940

Merged
merged 3 commits into from
Mar 10, 2024
Merged

Conversation

JappeHallunken
Copy link
Contributor

Let's use the G_DEV_BRANCH function, it will make life easier if you have no option to copy and paste the instruction.

Let's use the function, it will make life easier if you have no option to copy and paste the instruction
@MichaIng
Copy link
Owner

But then the 3rd step is not required anymore. It updates right away, without the chance to generate a backup or so.

@JappeHallunken
Copy link
Contributor Author

Ok so then just remove the 3rd step in the docs, since the function does dietpi-update -1 anyway?

G_DEV_BRANCH(){
		G_CHECK_ROOT_USER 1
		G_CONFIG_INJECT 'DEV_GITBRANCH=' "DEV_GITBRANCH=$1" /boot/dietpi.txt
		/boot/dietpi/dietpi-update -1
	}

Also the recommendation for making a back up is mentioned in the first step.

@MichaIng
Copy link
Owner

MichaIng commented Mar 9, 2024

Ah right, so only step 3 needs to be removed and we are good.

@MichaIng MichaIng added the META Everything that is not code related, e.g. GitHub, Wiki, website, community label Mar 9, 2024
@MichaIng MichaIng added this to the v9.2 milestone Mar 9, 2024
Removed step 3 in "beta on existing image"-block, because `G_DEV_BRANCH()` does a `/boot/dietpi/dietpi-update -1` anyway.
@MichaIng MichaIng merged commit 8b0e131 into MichaIng:dev Mar 10, 2024
1 check passed
@JappeHallunken JappeHallunken deleted the patch-1 branch March 12, 2024 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
META Everything that is not code related, e.g. GitHub, Wiki, website, community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants