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

looks cool ✨ #7

Open
gingerlime opened this issue Mar 19, 2021 · 9 comments
Open

looks cool ✨ #7

gingerlime opened this issue Mar 19, 2021 · 9 comments

Comments

@gingerlime
Copy link

gingerlime commented Mar 19, 2021

This looks cool. I'm maintaining the thumbor docker image at https://github.com/minimalcompact/thumbor

Would you be interested creating a PR to integrate thumbor-wand-engine into the docker image? happy to help in the process. I would say this should be strictly optional, but could increase adoption if it's available out of the box (or container) ;)

@scorphus
Copy link
Owner

That would be great, @gingerlime. How should we proceed? What you think is the best option?

@gingerlime
Copy link
Author

@scorphus cool. Would you be comfortable opening a PR to add wand-engine into the Docker image? are you comfortable with docker to test it? I can definitely help with adding it to the baseline, but most likely need your help testing that things are working :)

@scorphus
Copy link
Owner

scorphus commented Mar 29, 2021

Would you be comfortable opening a PR to add wand-engine into the Docker image?

Definitely.

are you comfortable with docker to test it?

Comfortable enough, I'd say.

I can definitely help with adding it to the baseline

Awesome! Is there any change made in the past (one or more commits in git history) that may be helpful in some ways?

@gingerlime
Copy link
Author

Hey @scorphus sorry for the delay. The main thumbor Dockerfile should be the best starting point I assume? plus the requirements.txt file. In addition, the thumbor configuration lives in a template file that gets parsed to set up the actual config...

Let me know if you have any other questions or need help with anything else :)

@scorphus
Copy link
Owner

Hey, @gingerlime. Sorry for the delay too.

I gave it a try and it seemed to me that the container runs Python 2. The engine was born after the retirement of Python 2 and I wouldn't like to add support to retired software. Things should move forward 🙂

I then managed to install Thumbor@master ― with pip3 install git+https://github.com/thumbor/thumbor.git ― after installing python3-pip along with a few other APT packages. Some steps of the installation of Thumbor took very long but eventually I got almost everything working ― I got a strange issue with the watermark filter.

Would you be willing to add support to Python 3 and Thumbor@master as some sort of alpha image release?

@gingerlime
Copy link
Author

@scorphus 👋

Yeah, thumbor seems to move a bit slow to python 3 if I'm not mistaken? haven't seen any announcement or much changing, but I'm not following very closely. Definitely keen to get things over to python 3. Take a look at MinimalCompact/thumbor#76 where I started some work in this direction, but it's been kinda quiet on that one for a while ... This should be a good baseline though I think. Please feel free to comment there or ask on Thumbor to see where things are going.

@scorphus
Copy link
Owner

Hey, @gingerlime 👋

I'm almost certain you're already aware of thumbor 7 being released recently. I just felt like giving a heads up 🙂

@gingerlime
Copy link
Author

I'm well aware of it :)

@scorphus
Copy link
Owner

Hey, @gingerlime! What do we do about this issue? Should we transfer it to where changes have to be made to?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants