Change to pep517.meta.load

[wwuck]

Should this plugin change to using pep517.meta.load? I've tried to change from setup.py to setup.cfg for project metadata and this plugin doesn't detect install_requires metadata in setup.cfg.

ref: pypa/setuptools#1951 (comment)

[arkq]

There is one problem with using pep517.meta.load. Firstly, it's not a public API of pep517 module. Secondly, it seems that it requires pip to be available. I think that for now I can add built-in support for requirements from setup.cfg file. Do you have any edge cases for install_requires section in setup.cfg? Because from the documentation, it seems that the syntax is rather simple: https://setuptools.readthedocs.io/en/latest/userguide/declarative_config.html

[arkq]

@wwuck could you please check whether PR #40 will work for you?

[wwuck]

@arkq I can confirm that PR #40 works with setup.cfg. Thanks!

My other thought behind using pep517.meta.load was that it would also support PEP621 (either currently or in the future, haven't checked this).
Now that I've looked further, I see that pep517.meta.load will possibly be deprecated/removed soon (pypa/pyproject-hooks#100), but there might be an alternative appearing out of that discussion pypa/build#181. It looks like that might be the way to go when it finally gets implemented?

It would be nice to have support for projects using PEP621 pyproject.toml configuration (eg. flit) as well and I'm guessing using something common like that proposed feature of pypa/build would get PEP621 support included automatically?

[arkq]

It would be nice to have support for projects using PEP621 pyproject.toml configuration (eg. flit) as well and I'm guessing using something common like that proposed feature of pypa/build would get PEP621 support included automatically?

Yes, that would be nice to have a separate module for querying requirements. If there will be such used by other tools, I will jump the bandwagon as well.

[wwuck]

@arkq Does this do the job?

pypa/build#301

looking at pypa/build#181 (comment) suggests a proper helper function will appear soon, but until then the sample at end of issue 301 should work? That change is already part of the latest release of pypa/build