Fix for future versions of setuptools

[ghost]

Starting python 3.10, the use of - instead of _ will get a warn (see https://bugs.gentoo.org/796281 for reference)

Signed-off-by: Marco Scardovi marco@scardovi.com

[google-cla]

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.

---

What to do if you already signed the CLA

Individual signers

• It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

• Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
• The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
• The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ️ Googlers: Go here for more info.

[AppVeyorBot]

✅ Build brotli 1.0.0#1625 completed (commit 52a92b214c by @scardracs)

[ghost]

@dlemstra i see you are the latest one that committed inside this repo. Could you tell me who is the right person to ask for merge? Or could you merge it?

[dlemstra]

You can check the pull request that is referenced in my commit and see who made the merge.

[ghost]

@dlemstra thanks :) @eustas can you take a look on my PR?

[candrews]

@scardracs it appears you need to sign the CLA

@eustas or anyone else, can this please be merged?

[ghost]

@candrews sorry for late answer, it seems no one take a look into it

[dlemstra]

You will need to sign the CLA @scardracs.

[ghost]

@googlebot I signed it!

[candrews]

Can this PR now be merged?

[eustas]

Hello.

Sorry for super-late reply.

One question - what will happen with PY2 builds?

[eustas]

Please rebase as well.

[ghost]

Please rebase as well.

I haven't tested against py2 as it should be EOL.

[eustas]

@anthrotype WDYT?

[ghost]

@eustas rebased

[eustas]

@scardracs thanks. Still waiting for opinion from @anthrotype.
As I can see we don't build for PY2 in https://github.com/google/brotli-wheels
So, most likely this change is fine.

[ghost]

@scardracs thanks. Still waiting for opinion from @anthrotype. As I can see we don't build for PY2 in https://github.com/google/brotli-wheels So, most likely this change is fine.

I have no problem on waiting :)

[ghost]

News?

[anthrotype]

LGTM, sorry for the delay in responding