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

Fix deprecation of Pillow ImageDraw textsize() #171

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

johnsom
Copy link

@johnsom johnsom commented Feb 16, 2023

Pillow has marked ImageDraw.textsize() as deprecated[1] for removal in version 10. This patch updates blockdiag to use the new recommended method for getting the text height and width.

[1] https://pillow.readthedocs.io/en/stable/deprecations.html#font-size-and-offset-methods

Pillow has marked ImageDraw.textsize() as deprecated[1] for removal in version 10. This patch updates blockdiag to use the new recommended method for getting the text height and width.

[1] https://pillow.readthedocs.io/en/stable/deprecations.html#font-size-and-offset-methods
@Dridi
Copy link
Contributor

Dridi commented Jul 9, 2023

@johnsom there are other deprecated APIs gone with pillow 10, like FreeTypeFont.getsize().

Would you be able to take a stab at patching blockdiag et al for pillow 10? I'm maintaining the package on Fedora and between this, #159 and my lack of time I'm considering orphaning *diag on Fedora. There has unfortunately been no activity since the 3.0.0 release.

@johnsom
Copy link
Author

johnsom commented Jul 13, 2023

It's not clear to me that this project is maintained now as this PR has not progressed and the last commit was 2021.
Due to this, I do not plan to work on any other incompatibilities.

In my use case, I will be removing blockdiag for our project.

@Dridi
Copy link
Contributor

Dridi commented Jul 14, 2023

Fair enough, I will orphan *diag on Fedora.

openstack-mirroring pushed a commit to openstack/openstack that referenced this pull request Jul 31, 2023
* Update designate from branch 'master'
  to 4e7e471760d3df40b3282bf73477fda2d08fb384
  - Merge "Replace blockdiag with graphviz"
  - Replace blockdiag with graphviz
    
    The python blockdiag module is unmaintained and will likely be removed from distros.[1]
    This patch replaces blockdiag with graphviz diagrams.
    
    [1] blockdiag/blockdiag#171
    
    Change-Id: I3a902c60b59071b466e1bf1ec6bda205e58b378c
openstack-mirroring pushed a commit to openstack/designate that referenced this pull request Jul 31, 2023
The python blockdiag module is unmaintained and will likely be removed from distros.[1]
This patch replaces blockdiag with graphviz diagrams.

[1] blockdiag/blockdiag#171

Change-Id: I3a902c60b59071b466e1bf1ec6bda205e58b378c
@tjni tjni mentioned this pull request Aug 5, 2023
openstack-mirroring pushed a commit to openstack/designate that referenced this pull request Aug 9, 2023
The python blockdiag module is unmaintained and will likely be removed from distros.[1]
This patch replaces blockdiag with graphviz diagrams.

[1] blockdiag/blockdiag#171

Change-Id: I3a902c60b59071b466e1bf1ec6bda205e58b378c
(cherry picked from commit beb79ac)
openstack-mirroring pushed a commit to openstack/openstack that referenced this pull request Aug 17, 2023
* Update octavia from branch 'master'
  to 2e563016ce77617e5416f27ff5800b8821561c25
  - Merge "Remove blockdiag from Octavia"
  - Remove blockdiag from Octavia
    
    The python blockdiag module is unmaintained and will likely be removed from distros.[1]
    This patch removes mention of blockdiag from Octavia as it is unused.
    
    [1] blockdiag/blockdiag#171
    
    Change-Id: I88371364b88bec5f1fd42d6ade8b316be3130f3f
openstack-mirroring pushed a commit to openstack/octavia that referenced this pull request Aug 17, 2023
The python blockdiag module is unmaintained and will likely be removed from distros.[1]
This patch removes mention of blockdiag from Octavia as it is unused.

[1] blockdiag/blockdiag#171

Change-Id: I88371364b88bec5f1fd42d6ade8b316be3130f3f
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

Successfully merging this pull request may close these issues.

None yet

2 participants