Opened 5 years ago

Closed 9 months ago

#23893 closed defect (invalid)

make openssl a dependency for pip packages

Reported by: mderickx Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: packages: optional Keywords:
Cc: dunfield Merged in:
Authors: Reviewers: Nathan Dunfield
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #29291 Stopgaps:

Status badges


Right now one has:

sage -i beautifulsoup
make[1]: Entering directory '/users/derickx/sage_patch2/build/make'
sage-logger -p 'sage --pip install beautifulsoup' /users/derickx/sage_patch2/logs/pkgs/beautifulsoup.log
[beautifulsoup] pip is configured with locations that require TLS/SSL, however the ssl module in Python is not available.
[beautifulsoup] Collecting beautifulsoup
[beautifulsoup]   Could not fetch URL There was a problem confirming the ssl certificate: Can't connect to HTTPS URL because the SSL module is not available. - skipping
[beautifulsoup]   Could not find a version that satisfies the requirement beautifulsoup (from versions: )
[beautifulsoup] No matching distribution found for beautifulsoup
Makefile:1114: recipe for target 'beautifulsoup' failed
make[1]: *** [beautifulsoup] Error 1
make[1]: Leaving directory '/users/derickx/sage_patch2/build/make'

real    0m1.186s
user    0m0.788s
sys     0m0.040s
Error building Sage.

The following package(s) may have failed to build (not necessarily
during this run of 'make beautifulsoup'):

The build directory may contain configuration files and other potentially
helpful information. WARNING: if you now run 'make' again, the build
directory will, by default, be deleted. Set the environment variable
SAGE_KEEP_BUILT_SPKGS to 'yes' to prevent this.

Makefile:16: recipe for target 'beautifulsoup' failed
make: *** [beautifulsoup] Error 1

However this does not need to fail since it succeeds after:

sage -i openssl
sage -f python2

Change History (6)

comment:1 Changed 3 years ago by embray

  • Milestone changed from sage-8.1 to sage-wishlist

I think what we may want to do about this is actually just make building Sage without SSL fail unless explicitly requested. See e.g. #27705

comment:2 Changed 2 years ago by dunfield

  • Cc dunfield added

comment:3 Changed 20 months ago by slelievre


  • #30560: Make openssl an optional dependency of python3
  • #29555: Upgrade: OpenSSL 3.0, make it a standard package

comment:4 Changed 9 months ago by mkoeppe

  • Dependencies set to #29291
  • Milestone changed from sage-wishlist to sage-duplicate/invalid/wontfix
  • Status changed from new to needs_review

Outdated; with #29291 we just no longer accept python3 that does not have openssl.

comment:5 Changed 9 months ago by dunfield

  • Reviewers set to Nathan Dunfield
  • Status changed from needs_review to positive_review

Agree this outdated, setting positive review.

comment:6 Changed 9 months ago by mkoeppe

  • Resolution set to invalid
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.