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 update #6850

Closed
macraj opened this issue Jan 11, 2024 · 4 comments
Closed

DietPi update #6850

macraj opened this issue Jan 11, 2024 · 4 comments
Labels
Bug 🐞 Solution available 🥂 Definite solution has been done
Milestone

Comments

@macraj
Copy link

macraj commented Jan 11, 2024

Details:

  • Date | Thu Jan 11 17:07:35 CET 2024
  • DietPi version | v9.-1.0 (MichaIng/dev)
  • Image creator |
  • Pre-image |
  • Hardware | RPi B+ (armv6l) (ID=1)
  • Kernel version | Linux DatsPi 6.1.21+ #1642 Mon Apr 3 17:19:14 BST 2023 armv6l GNU/Linux
  • Distro | bookworm (ID=7,RASPBIAN=1)
  • Command | rm dev.tar.gz DietPi-dev/dietpi/pre-patch_file DietPi-dev/dietpi/server_version-6
  • Exit code | 1
  • Software title | DietPi-Update

Steps to reproduce:

  1. sudo dietpi-update

Expected behaviour:

  • update

Actual behaviour:

  • ...

Extra details:

  • ...

Additional logs:

rm: cannot remove 'DietPi-dev/dietpi/pre-patch_file': No such file or directory
rm: cannot remove 'DietPi-dev/dietpi/server_version-6': No such file or directory
@Trimble-tech
Copy link

I had the same issue on a RPi4B; opening a sub-shell in the /tmp directory shows the pre-patch_file, server_version-6, and the dev.tar.gz not being present. Also, when running DietPi-BugReport the process was unable to finish after extended wait.

Below is the details output by the update error:

  • Date | Thu Jan 11 11:29:39 EST 2024
  • DietPi version | v9.-1.0 (MichaIng/dev)
  • Image creator |
  • Pre-image |
  • Hardware | RPi 4 Model B (aarch64) (ID=4)
  • Kernel version | Linux Piper2 6.1.21-v8+ #1642 SMP PREEMPT Mon Apr 3 17:24:16 BST 2023 aarch64 GNU/Linux
  • Distro | bookworm (ID=7,RASPBIAN=0)
  • Command | rm dev.tar.gz DietPi-dev/dietpi/pre-patch_file DietPi-dev/dietpi/server_version-6
  • Exit code | 1
  • Software title | DietPi-Update
    Based on the details I see here and in the original post, I believe the issue is due to the test images getting upgraded to Bookworm earlier; the updater is looking for files that don't exist due to it. In particular, they are the following in my case:
  • /tmp/DietPi-Update/DietPi-dev/dietpi/pre-patch_file
  • /tmp/DietPi-Update/DietPi-dev/dietpi/server_version-6
  • /tmp/Dietpi-Update/dev.tar.gz

I opened a sub-shell and touched all of these files (touch filename) to create them, then the update proceeded as normal. So far my system works and reports version 9.0.0 (dev) but I am unsure if this is a reasonable fix for the problem.

@Joulinar
Copy link
Collaborator

Guys this is a development branch. Not even in Beta status. But thx for reporting.

@MichaIng
Copy link
Owner

MichaIng commented Jan 11, 2024

Generally, if you are not aware that you were on a (naturally unstable) development branch:

G_CONFIG_INJECT 'DEV_GITBRANCH=' 'DEV_GITBRANCH=master' /boot/dietpi.txt

I was tinkering with removing the obsolete DietPi v6 files from the repo last night. I thought I had fixed it, worked on my VM finally, but seems it was too late and I was too tired.

DietPi version | v9.-1.0 (MichaIng/dev)

Hmm, that alone is strange already, it should be v9.0.-1 🤔.
EDIT: Ah nope all right if you were on v9.0.-1 before. Then v9.-1.0 would be an intermediate step to v9.0.0, to have the v9.0 patches applied again on the patch version increment.

EDIT: Ah, and now I know what the issue is: When the system was on v9.0 already but did this just as patch version update, the intended pre-patch is not applied.

@MichaIng MichaIng added the Solution available 🥂 Definite solution has been done label Jan 11, 2024
@MichaIng MichaIng added this to the v9.0 milestone Jan 11, 2024
@MichaIng
Copy link
Owner

MichaIng commented Jan 11, 2024

Fixed with: 7badaa5

In your cases, if you fixed it manually via touch or alike, no need to do anything else, as this is precisely what the pre-patch does as well. However, I suggest to switch back to master branch, unless you want to test something specific from the development branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 🐞 Solution available 🥂 Definite solution has been done
Projects
None yet
Development

No branches or pull requests

4 participants