Docker: add missing reporting libs

[SchrodingersGat]

Ref: #5007

[wolflu05]

Was this intentional, or an oversight? We'll need to add these back in.

Missed the poppler-utils one, but the rest was intentional as they do not exist on alpine, see, hopefully it works even without them. Unfortunately, I already testet label generation and it worked.

But your changes do not work, I suggest adding only poppler-utils to this line

InvenTree/Dockerfile

Line 61 in 61840bc

py3-pip py3-pillow py3-cffi py3-brotli pango \

[SchrodingersGat]

But your changes do not work

Do not work how? The docker image builds OK for me?

[wolflu05]

Do not work how? The docker image builds OK for me?

Yes, it's basically the same as without your changes. Look closely at the last line where you added the \ the fc-cache command just ignores the extra args you provided. You need to add them to a apk add command. I suggest adding it to L60, because there is the other report stuff.

[SchrodingersGat]

Yes, it's basically the same as without your changes. Look closely at the last line where you added the \ the fc-cache command just ignores the extra args you provided. You need to add them to a apk add command. I suggest adding it to L60, because there is the other report stuff.

Ah, yes ok that is obvious now :)

[wolflu05]

Keep an eye on the size, not sure if we really need the added packages, I looked at the official docs and that's why I didn't add them.

Image is now at 588MB on my machine.

[SchrodingersGat]

Keep an eye on the size, not sure if we really need the added packages, I looked at the official docs and that's why I didn't add them.

Yep, looks like unit tests pass correctly without them, so have removed again :)

[SchrodingersGat]

@wolflu05 very nice work :) The size difference is impressive