search

FormerLurker / Octolapse

Stabilized timelapses for Octoprint
GNU Affero General Public License v3.0
639 stars 99 forks source link

Fix typo in required pillow version #901

Closed r-stiller closed 1 year ago

r-stiller commented 1 year ago

As mentioned in #888 and #900 installing octolapse 0.4.2 fails. This is due to a missing comma in the in required pillow version string of setup.py.

jneilliii commented 1 year ago

@FormerLurker you should probably merge this one pretty quick. Already starting to get posts in the forum about failed installs.

FormerLurker commented 1 year ago

I have had this fixed in the devel branch for a couple weeks. this PR also attempted to address this. Some additional changes were required since new versions of Pillow changed the imports. I was waiting for additional confirmation that the fix worked without causing issues in older versions, but I guess i should just push it and see what happens.

Thanks for your contribution, and sorry if I sort of wasted your time by not getting this fix out there earlier. I plan to add a PR template that specifies the appropriate branch (devel) to target to keep this from happening in the future.