Opened 3 years ago

Closed 3 years ago

#26613 closed defect (fixed)

sagenb fails to build due to missing dependencies

Reported by: Konrad127123 Owned by:
Priority: major Milestone: sage-8.5
Component: packages: standard Keywords: sagenb, packaging, pyparsing, dependencies
Cc: jdemeyer, fbissey, kcrisman Merged in:
Authors: Konrad K. Dabrowski Reviewers: Jeroen Demeyer, François Bissey
Report Upstream: N/A Work issues:
Branch: aa8b3fb (Commits, GitHub, GitLab) Commit: aa8b3fbbbcbd2c69e357d7807cb33aed481d453f
Dependencies: Stopgaps:

Status badges

Description (last modified by Konrad127123)

sagenb currently fails to build in a clean build environment because pyparsing is missing. The error message comes from packaging, which should depend on pyparsing. Adding this dependency solves the issue, as can be checked with make distclean && make sagenb.

Error message:

[sagenb-1.1.0] Traceback (most recent call last):
[sagenb-1.1.0]   File "/home/sage/sage-8.4/local/bin/sphinx-build", line 6, in <mod
ule>
[sagenb-1.1.0]     from pkg_resources import load_entry_point
[sagenb-1.1.0]   File "/home/sage/sage-8.4/local/lib/python2.7/site-packages/pkg_re
sources/__init__.py", line 3098, in <module>
[sagenb-1.1.0]     @_call_aside
[sagenb-1.1.0]   File "/home/sage/sage-8.4/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3082, in _call_aside
[sagenb-1.1.0]     f(*args, **kwargs)
[sagenb-1.1.0]   File "/home/sage/sage-8.4/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 3111, in _initialize_master_working_set
[sagenb-1.1.0]     working_set = WorkingSet._build_master()
[sagenb-1.1.0]   File "/home/sage/sage-8.4/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 573, in _build_master
[sagenb-1.1.0]     ws.require(__requires__)
[sagenb-1.1.0]   File "/home/sage/sage-8.4/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 891, in require
[sagenb-1.1.0]     needed = self.resolve(parse_requirements(requirements))
[sagenb-1.1.0]   File "/home/sage/sage-8.4/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 777, in resolve
[sagenb-1.1.0]     raise DistributionNotFound(req, requirers)
[sagenb-1.1.0] pkg_resources.DistributionNotFound: The 'pyparsing>=2.0.2' distribution was not found and is required by packaging
[sagenb-1.1.0] make[2]: *** [Makefile:20: html] Error 1
[sagenb-1.1.0] make[2]: Leaving directory '/home/sage/sage-8.4/local/var/tmp/sage/build/sagenb-1.1.0/src/doc'
[sagenb-1.1.0] 

Change History (10)

comment:1 Changed 3 years ago by Konrad127123

  • Branch set to u/Konrad127123/sagenb_fails_to_build_due_to_missing_dependencies

comment:2 Changed 3 years ago by Konrad127123

  • Authors set to Konrad K. Dabrowski
  • Commit set to c389a43f96e093888bd894ad69bf316f0ea72424
  • Description modified (diff)
  • Keywords sagenb packaging pyparsing dependencies added
  • Status changed from new to needs_review

New commits:

c389a43packaging should depend on pyparsing

comment:3 Changed 3 years ago by tscrim

  • Cc jdemeyer fbissey kcrisman added

comment:4 Changed 3 years ago by fbissey

Legit. Gentoo ebuild for packaging has runtime dependency (but not build time) for pyparsing but also six. As a precaution I think this ticket should also add six to the dependency list of packaging.

comment:5 Changed 3 years ago by git

  • Commit changed from c389a43f96e093888bd894ad69bf316f0ea72424 to 328415b4eb68fb8a59a46b424d8ab090d67b0314

Branch pushed to git repo; I updated commit sha1. New commits:

328415bpyparsing should depend on six

comment:6 Changed 3 years ago by jdemeyer

  • Status changed from needs_review to needs_work

pyparsing does not depend on six. François meant that packaging should depend on six.

comment:7 Changed 3 years ago by git

  • Commit changed from 328415b4eb68fb8a59a46b424d8ab090d67b0314 to aa8b3fbbbcbd2c69e357d7807cb33aed481d453f

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

aa8b3fbpackaging should depend on six

comment:8 Changed 3 years ago by jdemeyer

  • Reviewers set to Jeroen Demeyer
  • Status changed from needs_work to positive_review

comment:9 Changed 3 years ago by jdemeyer

  • Reviewers changed from Jeroen Demeyer to Jeroen Demeyer, François Bissey

comment:10 Changed 3 years ago by vbraun

  • Branch changed from u/Konrad127123/sagenb_fails_to_build_due_to_missing_dependencies to aa8b3fbbbcbd2c69e357d7807cb33aed481d453f
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.