Immich 1.136 Photo and Video Backup Brings Breaking Changes (linuxiac.com)
from KarnaSubarna@lemmy.ml to linux@lemmy.ml on 27 Jul 13:50
https://lemmy.ml/post/33743364

#linux

threaded - newest

BakedCatboy@lemmy.ml on 27 Jul 14:05 next collapse

The new beta timeline is sooo smooth! I finally don’t hate scrolling back to find a specific old photo. The scrolling performance feels completely native to me now.

utopiah@lemmy.ml on 27 Jul 15:29 next collapse

As the title might appear a bit alarmist, saving a click “For most users, there’s nothing to worry about. However, if you’ve manually set a custom relative path for “IMMICH_MEDIA_LOCATION” in your “.env” file, you’ll need to convert it to an absolute path. For example, “IMMICH_MEDIA_LOCATION=./my-library” must become “IMMICH_MEDIA_LOCATION=/usr/src/app/my-library“.”

cravl@slrpnk.net on 27 Jul 18:25 collapse

I wouldn’t think this would cause any data loss either, it just wouldn’t find your media or it would throw an error. Very alarmist indeed.

baronvonj@lemmy.world on 27 Jul 17:16 next collapse

I’ve been meaning to give this a try on my Synology.

But breaking changes in a point release? Not cool.

InnerScientist@lemmy.world on 27 Jul 17:44 next collapse

Tbf this is actually version v1.136 .0 and

Disclaimer

  • ⚠️ The project is under very active development.
  • ⚠️ Expect bugs and breaking changes.
  • ⚠️ Do not use the app as the only way to store your photos and videos.
  • ⚠️ Always follow 3-2-1 backup plan for your precious photos and videos!
KarnaSubarna@lemmy.ml on 27 Jul 19:07 next collapse

Personally I’m waiting for the day it comes out of “under active development” state so that I can migrate from NextCloud to it.

baronvonj@lemmy.world on 27 Jul 20:28 collapse

A breaking change should have been 2.0, not a new 1.<minor> release.

It should still be 0.<minor> if they’ve not reached the stability for keeping backwards compatibly in all 1.x releases.

InnerScientist@lemmy.world on 27 Jul 20:46 collapse

To quote them:

We are still in a fast development cycle, so the versioning is to keep track of the progress/iteration of the project. When a stable release is reached (2?), then any breaking change would require more proper major version changes

baronvonj@lemmy.world on 27 Jul 21:11 collapse

Yes, I understand they have declared that. Their declaration does not, however, negate the common semantic versioning standards, found at semver.org. These common standards are significant for admins running shared systems where they automatic upgrade processes based on common semantic versioning rules. The software will stabilize and they will adopt a more stringent policy. But they should still be releasing 0.x versions since they’ve not yet reached it.

irotsoma@lemmy.blahaj.zone on 27 Jul 18:07 next collapse

It’s a full release, not a point/patch release, the title just doesn’t show the second .0. They use semantic versioning so it’s major.minor.patch.

It’s also a very minor change and only affects a single configuration property and only people who used relative paths in that property.

baronvonj@lemmy.world on 27 Jul 20:25 collapse

Breaking changes should warrant a 2.0 version, not a 1.minor version.

Edit: I am basing my comments on semver.org guidelines

hedgehog@ttrpg.network on 27 Jul 20:45 next collapse

Immich isn’t a library (the main use case for semver is dependencies that will be pulled into other projects) and as far as I know they don’t state that they use semver.

baronvonj@lemmy.world on 27 Jul 21:13 collapse

But it is a service that clients connect to via an API.

irotsoma@lemmy.blahaj.zone on 28 Jul 00:10 collapse

It’s not that kind of breaking change. It’s a change that won’t affect most people. Only those who chose to use a custom location for their media location and chose to set that to a relative path instead of an absolute one which caused the application to have trouble resolving the paths. The change eliminates a bug by preventing people from doing something that was not intended to be supported. So it’s not a “breaking” change necessarily in the sense that they are changing documented functionality. They are eliminating a way that people can misconfigure the application which may in some cases cause the application to break if someone successfully configured the application in this unintended way.

Vincent@feddit.nl on 27 Jul 18:23 collapse

From the release notes:

one of the last breaking changes we want to make before reaching the stable release milestone

So you’ll probably want to wait until they do a stable release.

baronvonj@lemmy.world on 27 Jul 20:34 collapse

Yes indeed. 🙂

Vincent@feddit.nl on 27 Jul 18:22 next collapse

Actual release notes: github.com/immich-app/immich/releases/…/v1.136.0

avidamoeba@lemmy.ca on 28 Jul 00:53 collapse

The Android app finally does IO on a background thread. 🫠