Skip to content

prefer local sources over remote ones#585

Closed
dimpase wants to merge 1 commit into
mpmath:masterfrom
dimpase:patch-1
Closed

prefer local sources over remote ones#585
dimpase wants to merge 1 commit into
mpmath:masterfrom
dimpase:patch-1

Conversation

@dimpase

@dimpase dimpase commented Mar 26, 2021

Copy link
Copy Markdown
Collaborator

While trying to upgrade SageMath's mpmath to 1.2.0, I had to patch this, see
https://trac.sagemath.org/ticket/31564#comment:9
(also, it simply does not work with new versions of setuptools etc)

[mpmath-1.2.0]     LookupError: setuptools-scm was unable to detect version for '/tmp/pip-req-build-i64_7gu6'.
[mpmath-1.2.0] 
[mpmath-1.2.0]     Make sure you're either building from a fully intact git repository or PyPI tarballs. Most other sources (such as GitHub's tarballs, a git checkout without the .git folder) don't contain the necessary metadata and will not work.
  • probably due to missing meta-data.

While trying to upgrade SageMath's mpmath to 1.2.0, I had to patch this, see
https://trac.sagemath.org/ticket/31564#comment:9
(also, it simply does not work with new versions of setuptools etc)
```
[mpmath-1.2.0]     LookupError: setuptools-scm was unable to detect version for '/tmp/pip-req-build-i64_7gu6'.
[mpmath-1.2.0] 
[mpmath-1.2.0]     Make sure you're either building from a fully intact git repository or PyPI tarballs. Most other sources (such as GitHub's tarballs, a git checkout without the .git folder) don't contain the necessary metadata and will not work.
```
- probably due to missing meta-data.
@skirpichev

skirpichev commented Mar 26, 2021

Copy link
Copy Markdown
Collaborator

Maybe it could be possible to use version = attr: mpmath.__version__ in the setup.cfg? // #586

@dimpase

dimpase commented Mar 26, 2021

Copy link
Copy Markdown
Collaborator Author

this actually is working in 1.2.1, as opposed to 1.2.0. I'll close it then, sorry for noise.

@dimpase dimpase closed this Mar 26, 2021
@skirpichev

Copy link
Copy Markdown
Collaborator

You reported a real problem. It's hardly a noise. (Unfortunately, that problem was predicted)

t98yu pushed a commit to HPQC-LABS/mpmath that referenced this pull request Apr 26, 2021
The setuptools_scm usage was broken by 867d878 (likely,
this is a source of problem in mpmath#585)

Closes mpmath#580 (an alternative to mpmath#583 and mpmath#581)
Solves mpmath#585 as well.
t98yu pushed a commit to t98yu/mpmath that referenced this pull request Apr 27, 2021
The setuptools_scm usage was broken by 867d878 (likely,
this is a source of problem in mpmath#585)

Closes mpmath#580 (an alternative to mpmath#583 and mpmath#581)
Solves mpmath#585 as well.
t98yu pushed a commit to t98yu/mpmath that referenced this pull request Apr 27, 2021
The setuptools_scm usage was broken by 867d878 (likely,
this is a source of problem in mpmath#585)

Closes mpmath#580 (an alternative to mpmath#583 and mpmath#581)
Solves mpmath#585 as well.
t98yu pushed a commit to t98yu/mpmath that referenced this pull request Apr 28, 2021
The setuptools_scm usage was broken by 867d878 (likely,
this is a source of problem in mpmath#585)

Closes mpmath#580 (an alternative to mpmath#583 and mpmath#581)
Solves mpmath#585 as well.
t98yu pushed a commit to t98yu/mpmath that referenced this pull request Apr 28, 2021
The setuptools_scm usage was broken by 867d878 (likely,
this is a source of problem in mpmath#585)

Closes mpmath#580 (an alternative to mpmath#583 and mpmath#581)
Solves mpmath#585 as well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants