potential command to add to the install scripts (develop branch) #473

Closed
7a opened this Issue Jun 24, 2015 · 6 comments

Comments

Projects
None yet
4 participants
@7a
Member

7a commented Jun 24, 2015

Should we run the following command towards the beginning of the Kali installer?

pip install --upgrade pip

Reasoning:
There are a bunch of warnings like the following when you install OWTF in the latest Kali, and maybe things work more smoothly after this?

You are using pip version 6.0.8, however version 7.0.3 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

@tunnelshade

This comment has been minimized.

Show comment
Hide comment
@tunnelshade

tunnelshade Jun 24, 2015

Member

Most of the time pip won't interfere with what it is installing as of now. We are not having an issue with pip 6.0.8 . Upgrading pip at the start can be implemented.

Member

tunnelshade commented Jun 24, 2015

Most of the time pip won't interfere with what it is installing as of now. We are not having an issue with pip 6.0.8 . Upgrading pip at the start can be implemented.

@viyatb

This comment has been minimized.

Show comment
Hide comment
@viyatb

viyatb Jun 25, 2015

Member

IMO keeping in mind the issue #474, we should atleast add these:

pip install --upgrade pip
rm -rf /usr/lib/python2.7/dist-packages/setuptools.egg-info
Member

viyatb commented Jun 25, 2015

IMO keeping in mind the issue #474, we should atleast add these:

pip install --upgrade pip
rm -rf /usr/lib/python2.7/dist-packages/setuptools.egg-info
@7a

This comment has been minimized.

Show comment
Hide comment
@7a

7a Jun 26, 2015

Member

Sounds good to me, but please let's 1st:

  1. Backup before we delete things (so that the user has the option to
    rollback, if needed)
  2. prompt the user before running things like "rm -rf directory" on the
    user filesystem.

Maybe something like:

Delete /usr/lib/python2.7/dist-packages/setuptools.egg-info? (y/n)
(recommended, solves some issues)

Then, if "yes" is chosen:

  1. We backup the system directory we are about to remove (IMPORTANT: add
    timestamp to backup, to ensure a previous backup does not break the install
    script, i.e. make the directory different each time)
    NOTE: Here it would be nice if we showed a message like "directory backed
    up to path/to/new/location"

  2. We can now safely delete because: a) we prompted the user (who gave
    consent) + b) we backed up the directory

Because of all of the above, the "mv" command might be a better way than
"rm -rf", i.e.
mv /usr/lib/python2.7/dist-packages/setuptools.egg-info
/usr/lib/python2.7/dist-packages/setuptools.egg-infoBACKUP20150626

2015-06-25 14:05 GMT+02:00 Viyat Bhalodia notifications@github.com:

IMO keeping in mind the issue #474
#474, we should atleast add these:

pip install --upgrade pip
rm -rf /usr/lib/python2.7/dist-packages/setuptools.egg-info


Reply to this email directly or view it on GitHub
#473 (comment).

Abraham Aranguren
http://7-a.org

Member

7a commented Jun 26, 2015

Sounds good to me, but please let's 1st:

  1. Backup before we delete things (so that the user has the option to
    rollback, if needed)
  2. prompt the user before running things like "rm -rf directory" on the
    user filesystem.

Maybe something like:

Delete /usr/lib/python2.7/dist-packages/setuptools.egg-info? (y/n)
(recommended, solves some issues)

Then, if "yes" is chosen:

  1. We backup the system directory we are about to remove (IMPORTANT: add
    timestamp to backup, to ensure a previous backup does not break the install
    script, i.e. make the directory different each time)
    NOTE: Here it would be nice if we showed a message like "directory backed
    up to path/to/new/location"

  2. We can now safely delete because: a) we prompted the user (who gave
    consent) + b) we backed up the directory

Because of all of the above, the "mv" command might be a better way than
"rm -rf", i.e.
mv /usr/lib/python2.7/dist-packages/setuptools.egg-info
/usr/lib/python2.7/dist-packages/setuptools.egg-infoBACKUP20150626

2015-06-25 14:05 GMT+02:00 Viyat Bhalodia notifications@github.com:

IMO keeping in mind the issue #474
#474, we should atleast add these:

pip install --upgrade pip
rm -rf /usr/lib/python2.7/dist-packages/setuptools.egg-info


Reply to this email directly or view it on GitHub
#473 (comment).

Abraham Aranguren
http://7-a.org

@viyatb

This comment has been minimized.

Show comment
Hide comment
@viyatb

viyatb Jun 27, 2015

Member

Ok, I'll change that bit :)

Member

viyatb commented Jun 27, 2015

Ok, I'll change that bit :)

@DePierre

This comment has been minimized.

Show comment
Hide comment
@DePierre

DePierre Jul 26, 2015

Contributor

@delta24 Is this issue closed thanks to your commit?

Contributor

DePierre commented Jul 26, 2015

@delta24 Is this issue closed thanks to your commit?

@viyatb

This comment has been minimized.

Show comment
Hide comment
@viyatb

viyatb Jul 26, 2015

Member

Yes

Member

viyatb commented Jul 26, 2015

Yes

@viyatb viyatb closed this Oct 21, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment