GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||||||||||||||||||||||||
GeezerVison is showing github.com — gh-115119: Default to --with-system-libmpdec=yes by erlend-aasland · Pull Request #118539 · python/cpython Conversation
|
$ ./configure --with-system-libmpdec=no | grep -E "(decimal|mpdec)" checking for --with-system-libmpdec... no checking for --with-decimal-contextvar... yes checking for decimal libmpdec machine... universal checking for stdlib extension module _decimal... yes $ ./configure | grep -E "(decimal|mpdec)" checking for --with-system-libmpdec... yes checking for libmpdec... yes checking for --with-decimal-contextvar... yes checking for stdlib extension module _decimal... yes $ grep -E "_DECIMAL_(CFLAGS|LDFLAGS)=" Makefile MODULE__DECIMAL_CFLAGS=-I/opt/homebrew/Cellar/mpdecimal/4.0.0/include MODULE__DECIMAL_LDFLAGS=-L/opt/homebrew/Cellar/mpdecimal/4.0.0/lib -lmpdec -lm |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@zware: I'm not sure if we should fall back to the internal copy if libmpdec is missing. For now, it is marked as missing if it is missing :) We could change it so |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
Things to consider:
For now, we can consider to use the bundled version for the Ubuntu CI. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Right, an explicit Footnotes
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Yep
Yes, we need to warn for that case.
Hm, I think this case should fail, similar to how we handle |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
I don't want a bare |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Ok, how about this:
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
I have a couple of questions and qualms, but overall I'm okay with this as is. We can tweak it through the beta phase if needed.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
…l of the mpdecimal sources Co-authored-by: Zachary Ware <[email protected]>
…rsion check in configure
| libgdbm-dev \ | ||
| libgdbm-compat-dev \ | ||
| liblzma-dev \ | ||
| libmpdec-dev \ |
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
@hugovk: FYI, this will fail for Ubuntu 24.04, as libmpdec-dev is unavailable there.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
I've been trying to figure out why that is, and have come up short (I get lost quickly in the Debian/Ubuntu development process). All I've found is a note from @doko42 in the Ubuntu python3.11 3.11.2-4 changelog, saying "Build with internal mpdecimal library, so that mpdecimal can be removed for bookworm." (and indeed, Debian bookworm does not have a libmpdec package either).
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
| libgdbm-dev \ | ||
| libgdbm-compat-dev \ | ||
| liblzma-dev \ | ||
| libmpdec-dev \ |
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
I've been trying to figure out why that is, and have come up short (I get lost quickly in the Debian/Ubuntu development process). All I've found is a note from @doko42 in the Ubuntu python3.11 3.11.2-4 changelog, saying "Build with internal mpdecimal library, so that mpdecimal can be removed for bookworm." (and indeed, Debian bookworm does not have a libmpdec package either).
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Thanks for helping out, Zach! |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Co-authored-by: Zachary Ware <[email protected]>
|
fwiw this is going to be painful for ubuntu and debian which has removed the |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
That seems to have been mentioned at #118539 (comment) already, but I'm not sure what the problem is. We didn't have Why can't Debian/Ubuntu just set --with-system-libmpdec=no or --without-system-libmpdec (whichever it is) if they can't package libmpdec quickly? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@thesamesam, @asottile: we're adding a fallback to the bundled version if an external libmpdec cannot be found. FTR, I find the recent removal of libmpdec from Ubuntu/Debian very strange. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.