Documentation should mention variable expansion works only in [[source]] entries

[OrangeDog]

At least on Windows, in a file string, environment variables are not being expanded.

_python-ldap = {platform_system = "== 'Windows'", file = "%USERPROFILE%/Downloads/python_ldap-3.1.0-cp36-cp36m-win_amd64.whl"}

WARNING: Invalid requirement, parse error at "'%USERPRO'"
ABORTING INSTALL... You will have to reinstall any packages that failed to install.

The same thing happens for ${USERPROFILE} and $USERPROFILE.
Python 3.6.5
Pipenv 2018.05.18

[uranusjr]

I think variable expansion is only supported in [[source]] now. I don’t recall whether it is planned to be supported for all of Pipfile (probably not).

[OrangeDog]

Sad face.
The documentation needs updating at least.

Is it possible to do what I'm trying to do here by defining a new source instead?

[uranusjr]

It is doable if you jump through a few hoops…well the quick answer is no, that’s probably not what you want. In reality though, what you’re looking for is quite strange. If you’re expecting a variable setup, why not expect the wheel file inside the project, and use a relative path (e.g. ./deps/python_ldap-3.1.0-cp36-cp36m-win_amd64.whl) instead? If you expect the file to be at a specific place, why not just expand the environment variable? The use case is really too unorthodox to be supported by a generic tool like Pipenv.

[OrangeDog]

Yeah, it is a bit odd, and I'm using a relative path now.
I guess if I weren't so lazy I should set up an actual repo to serve this wheel for Windows users.
I wish python-ldap or would just do it themselves.

[uranusjr]

Well maybe you can help them on this? I am not involved in python-ldap in any way, but normally wheels are not provided because either 1. they don’t want you to use wheels, or 2. they have insufficient resource to build them. If it’s the former, you probably shouldn’t use wheels anyway, and if it’s the latter, you can help out!

A wheel server is also much more easier than you may think. Just push it to a GIt repo, and use a raw URL to install. It will just work.

Closing because I believe we’ve covered everything here, and there’s nothing to fix (in Pipenv).

[OrangeDog]

there’s nothing to fix

The documentation should make it clear that they only work in [[source]]. The whole section may even be redundant to the "Injecting credentials into Pipfiles via environment variables" section.

[techalchemy]

I do wonder about this... I feel like we had expansion everywhere and it broke a lot of things. The test cases all use ${} syntax for both windows and Linux fwiw

[uranusjr]

The implementation uses expandvars, which (according to the documentation) accepts %...% form on Windows. But yeah ${...} is probably better if you want to do this on multiple platforms.

Reopening (and changing the title) to track the doc amendment.