Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release/1.8.4 #3802

Merged
merged 2 commits into from Oct 19, 2018

Conversation

Projects
None yet
3 participants
@lasote
Copy link
Contributor

commented Oct 19, 2018

Changelog: Fix: Changed tqdm dependency to a temporarily forked removing the "man" directory write permissions issue installing the pip package.

Changelog: Fix: Removed ndg-httpsclient and pyasn dependencies from OSX requirements file because they shouldn't be necessary.

Closes #3793
Closes #3785
Closes #3728

lasote added some commits Oct 19, 2018

Manifest diff prints (#3801)
* Manifest diff prints

* Fixed and improved test

* Update conans/client/cmd/uploader.py

Co-Authored-By: lasote <lasote@gmail.com>

* Update conans/client/cmd/uploader.py

Co-Authored-By: lasote <lasote@gmail.com>

@ghost ghost assigned lasote Oct 19, 2018

@ghost ghost added the stage: review label Oct 19, 2018

@lasote lasote added this to the 1.8.4 milestone Oct 19, 2018

@lasote lasote merged commit eace810 into master Oct 19, 2018

2 of 3 checks passed

continuous-integration/jenkins/branch This commit is being built
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details
license/cla Contributor License Agreement is signed.
Details

@ghost ghost removed the stage: review label Oct 19, 2018

@tivek

This comment has been minimized.

Copy link
Contributor

commented on conans/requirements.txt in f290f80 Oct 19, 2018

What is the reason to fork tqdm 4.27.0?

This comment has been minimized.

Copy link
Contributor

replied Oct 19, 2018

Because it is very problematic to install, as it tries to install to the "man" folder, which is non-existing and fails. The issue is reported upstream to tqdm, but will take a while to be released, it seems. Meanwhile, it seems that the least problematic way to install Conan is with this fork. Is there any problem for you (ArchLinux)?

This comment has been minimized.

Copy link
Contributor

replied Oct 19, 2018

I see. I'm relying on the python-tqdm package from Arch repos. They seem to have worked around the man issue back in tqdm 4.19.1, so until now I was unaware there even was a problem..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.