Skip to content

Bump up pip requirement to latest. - #577

Merged
jkarni merged 1 commit into
masterfrom
jkarni/feature/pip-upgrade
Aug 6, 2013
Merged

Bump up pip requirement to latest.#577
jkarni merged 1 commit into
masterfrom
jkarni/feature/pip-upgrade

Conversation

@jkarni

@jkarni jkarni commented Aug 5, 2013

Copy link
Copy Markdown

Solves an issue with using git > 1.8.1 for github based pip requirements.

    Solves an issue with using git > 1.8.1 for github based pip requirements.
@jkarni

jkarni commented Aug 5, 2013

Copy link
Copy Markdown
Author

@jzoldak @rocha

Tested on a mac.

@singingwolfboy

Copy link
Copy Markdown
Contributor

Can anyone who tests this pull request test #566 as well? Same principle: simply updating libraries in our requirements files.

@jzoldak

jzoldak commented Aug 5, 2013

Copy link
Copy Markdown
Contributor

Looks like the version was not upgraded when executed on the jenkins server. Needs more investigation before merging. It seems that if an old unpacked build file exists, pip will just use that one. Here's from the feature branch tests job:

00:03:56.546 Downloading/unpacking pip>=1.4 (from -r requirements/edx/base.txt (line 38))
00:03:56.546   Running setup.py egg_info for package pip
00:03:56.546     
00:03:56.546     warning: no files found matching '*.html' under directory 'docs'
00:03:56.546     warning: no previously-included files matching '*.txt' found under directory 'docs/_build'
00:03:56.546     no previously-included directories found matching 'docs/_build/_sources'
00:03:56.546   Requested pip>=1.4 (from -r requirements/edx/base.txt (line 38)), but installing version 1.3.1

However, when I wiped out the Workspace on the feature branch manual job then ran this branch, pip was updated correctly:

00:01:11.125 Downloading/unpacking pip>=1.4 (from -r requirements/edx/base.txt (line 38))
00:01:11.125   Using download cache from /mnt/pip-cache/https%3A%2F%2Fpypi.python.org%2Fpackages%2Fsource%2Fp%2Fpip%2Fpip-1.4.tar.gz
00:01:11.125   Running setup.py egg_info for package pip
00:01:11.125     
00:01:11.125     warning: no files found matching '*.html' under directory 'docs'
00:01:11.125     warning: no previously-included files matching '*.rst' found under directory 'docs/_build'
00:01:11.125     no previously-included directories found matching 'docs/_build/_sources'

Verified on Jenkins (ubuntu).
Tested on mac with pip install -I pip==1.3 then rake lms which upgraded it to 1.4.

👍

@rocha

rocha commented Aug 5, 2013

Copy link
Copy Markdown
Contributor

For some reason my dev environment was already running pip 1.4.

👍

jkarni pushed a commit that referenced this pull request Aug 6, 2013
Bump up pip requirement to latest.
@jkarni
jkarni merged commit 7ba6d19 into master Aug 6, 2013
@jkarni
jkarni deleted the jkarni/feature/pip-upgrade branch August 6, 2013 18:17
chrisrossi pushed a commit to jazkarta/edx-platform that referenced this pull request Mar 31, 2014
…cleanup

Feature/victor/fix url name cleanup
e-kolpakov referenced this pull request in open-craft/openedx-platform Dec 23, 2015
Group project v2 - Admin Dashboard Details View
mariajgrimaldi pushed a commit to eduNEXT/edx-platform that referenced this pull request Nov 23, 2021
Sujeet1379 pushed a commit to chandrudev/edx-platform that referenced this pull request Nov 17, 2022
Allowlist certs were hidden on the progress view because the grade
wasn't passing. This gives "downloadable" certs preference over the
grade.
bradenmacdonald pushed a commit that referenced this pull request Aug 24, 2023
* feat: add taxonomies for org api

* chore: run CI

* feat: Add retrieve object_tags REST API (#577)

* chore: update requirements

---------

Co-authored-by: Yusuf Musleh <yusuf@opencraft.com>
Abdul-Muqadim-Arbisoft referenced this pull request in edly-io/edx-platform May 8, 2024
Caddy was not properly starting because its associated configmap was not
starting.

Close #577.
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.

4 participants