-
-
Notifications
You must be signed in to change notification settings - Fork 216
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
Use fallback_version if SCM isn't found #549
Comments
please post the full output of the build process when fallback is supposed to be used when all else fails, so when git fails it should be used |
Sorry I wasn't clear, I didn't want to report any error as using It was more a suggestion to simplify things from a usage point of view, as for me after reading the docs I was thinking that |
let me be clear, if setuptools_scm fails with fallback set, it is a bug and needs to be fixed fallback is something you use more as a author |
OK then, here's the trace for the error I get when
Version used is 6.0.1, installed from |
thanks, the controll flow is indeed incorrect, i'll need to add a test that validates this so i dont introduce it by accident |
There is a fallback_version configuration option for setuptools_scm which we don't use: https://github.com/pypa/setuptools_scm#configuration-parameters Although this setting seems to have issues according to pypa/setuptools-scm#549 We already have a workaround in setup.py for the functionality of the fallback_version option, but it is disabled for the case of sdist: https://github.com/apache/arrow/blob/master/python/setup.py#L529 Closes #10342 from kszucs/ARROW-12619 Authored-by: Krisztián Szűcs <[email protected]> Signed-off-by: Krisztián Szűcs <[email protected]>
There is a fallback_version configuration option for setuptools_scm which we don't use: https://github.com/pypa/setuptools_scm#configuration-parameters Although this setting seems to have issues according to pypa/setuptools-scm#549 We already have a workaround in setup.py for the functionality of the fallback_version option, but it is disabled for the case of sdist: https://github.com/apache/arrow/blob/master/python/setup.py#L529 Closes #10342 from kszucs/ARROW-12619 Authored-by: Krisztián Szűcs <[email protected]> Signed-off-by: Krisztián Szűcs <[email protected]>
There is a fallback_version configuration option for setuptools_scm which we don't use: https://github.com/pypa/setuptools_scm#configuration-parameters Although this setting seems to have issues according to pypa/setuptools-scm#549 We already have a workaround in setup.py for the functionality of the fallback_version option, but it is disabled for the case of sdist: https://github.com/apache/arrow/blob/master/python/setup.py#L529 Closes apache#10342 from kszucs/ARROW-12619 Authored-by: Krisztián Szűcs <[email protected]> Signed-off-by: Krisztián Szűcs <[email protected]>
I'm also trying to use My
and the debug logs give
This seems to be caused by the presence of a |
We are using SETUPTOOLS_SCM_PRETEND_VERSION because the fallback_version feature in setuptools_scm does not work when git commands return errors (eg. when building from a tarball or git repo where no tags are present). Add a link to the upstream issue adjacent to where SETUPTOOLS_SCM_PRETEND_VERSION is specified. Upstream will start providing fallback_version in the next release so we should be able to remove the environment variable once the upstream setuptools_scm issue has been resolved. Signed-off-by: David Aguilar <[email protected]>
I'm in some situation (testing my package in Docker) where git isn't available, and thought I could just use
fallback_version
argument, but it turns out I still get an exceptionOSError: 'git' was not found
.I found I can overcome this by defining the
SETUPTOOLS_SCM_PRETEND_VERSION
environment variable, but I thought it would be easier to usefallback_version
in this case as well?I'm not sure I understood the different cases where one should use
SETUPTOOLS_SCM_PRETEND_VERSION
orfallback_version
?The text was updated successfully, but these errors were encountered: