Skip to content

Conversation

@verdurin
Copy link
Member

(created using eb --new-pr)

#}),
#('appdirs', '1.4.0', {
# 'source_urls': ['https://pypi.python.org/packages/source/a/appdirs/'],
#}),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks related to recent changes in setuptools, where dependencies required by setuptools are not longer 'vendored' by setuptools and thus need to be installed first, but they require setuptools themselves... >_<

In Spack, this was fixed by patching the setuptools dependencies so they only rely on distutils rather than setuptools, cfr. spack/spack#3198

Instead, I would rather use the most recent version of setuptools that doesn't require installing the dependencies first without having setuptools, i.e. setuptools version < 34.0, since setuptools is planning to take care of this problem themselves, see pypa/setuptools#980

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@boegel are we going to get back to this soon? I'm happy to help, given the local requests we have...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@verdurin To what specifically, a more recent version of setuptools, or having Python 2.7.13 and Tkinter as a separate module?

Python 2.7.13 without Tkinter is already there, cfr. #4260
I haven't had a need for Tkinter yet, so I haven't spent time on the separate module for it...

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's the separate Tkinter module that we need.

@boegel boegel modified the milestones: 3.1.1, 3.2.0 Mar 8, 2017
@boegel boegel modified the milestones: 3.2.0, 3.3.0 May 2, 2017
dependencies = [
('Python', '2.7.13', '-bare'),
('Tcl', '8.6.5'),
('Tk', '8.6.5', '-libX11-1.6.3'),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pull in the whole X11 bundle here?

@boegel boegel merged commit c859967 into easybuilders:develop May 23, 2017
@boegel boegel removed the 2018b label Jun 15, 2018
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.

3 participants