Question: why is pycparser listed in requirements.txt ?

Demindiro / agreper

Minimal, no-JS web forum software

https://forum.agreper.com/

GNU Affero General Public License v3.0

654 stars 27 forks source link

Closed Lucas-C closed 1 year ago

Lucas-C commented 1 year ago

Hi!

Thank you for building & releasing this software as FLOSS 😊

I was checking this projects dependencies and I spotted pycparser==2.21 among them: https://github.com/Demindiro/agreper/blob/master/requirements.txt#L12

It does not seem used in the source code though. Is it really required?

Demindiro commented 1 year ago

pycparser is an indirect dependency of argon2-cffi. Agreper does not directly depend on pycparser.

I generated requirements.txt with pip3 freeze, which also included a bunch of indirect dependencies. I'll remove those.

Lucas-C commented 1 year ago

Thank you for your answer