Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Copy uninstall scripts to /opt/google #303

Closed
tst-lsavoie opened this issue Jul 6, 2017 · 3 comments
Closed

Copy uninstall scripts to /opt/google #303

tst-lsavoie opened this issue Jul 6, 2017 · 3 comments
Labels
beginner
Milestone
5.3.1

Comments

@tst-lsavoie
Copy link
Collaborator

The installers should copy the uninstall scripts to somewhere under /opt/google so users can uninstall GEE without having the source repo available. This fix needs to be applied to both the fusion and server installers. Documentation files 3499964 and 4492617 (describing the uninstall process) will also need to be updated.
@tst-lsavoie tst-lsavoie mentioned this issue Jul 6, 2017
Merged
@tst-ccamp tst-ccamp added this to the 5.2.1 milestone Jul 12, 2017
@tst-lsavoie tst-lsavoie changed the title Install uninstall scripts Copy uninstall scripts to /opt/google Jul 13, 2017
@tst-lsavoie tst-lsavoie mentioned this issue Jul 13, 2017
Closed
@tst-lsavoie tst-lsavoie mentioned this issue Sep 8, 2017
Open
@tst-lsavoie tst-lsavoie removed this from the 5.2.1 milestone Jan 10, 2018
Amndeep7 added a commit to Amndeep7/earthenterprise that referenced this issue Mar 21, 2019
@Amndeep7
Added copying uninstall scripts to the install staging directory (goo…
17c3c68 
…gle#303)
@Amndeep7
Copy link
Contributor

VirtualBox_CentOS 7 GEE Build Environment_21_03_2019_00_29_58

Seeing this issue with the changes in the above commit. @tst-lsavoie has determined that it's a bug with khEnvironment.py:

The problem is on line 553: bld = t.get_builder() or my_alias_builder
If t.get_builder() doesn't return a valid builder it throws the exception. We really need to try t.has_builder() - if' that's true we should call t.get_builder(), otherwise we should use my_alias_builder.

I'll try to resolve that as part of this issue/pr.

Amndeep7 added a commit to Amndeep7/earthenterprise that referenced this issue Mar 21, 2019
@Amndeep7
Fixed bug causing SCons to have an InternalError (google#303)
175c652
Amndeep7 added a commit to Amndeep7/earthenterprise that referenced this issue Mar 21, 2019
@Amndeep7
Changed the temp staging area subdirectory for storing the uninstall …
62a4eb9 
…scripts along with also copying over their dependency (google#303)
Amndeep7 added a commit to Amndeep7/earthenterprise that referenced this issue Mar 21, 2019
@Amndeep7
Added the uninstall scripts being copied from the staging area to the…
a9813bc 
… install directory (google#303)
@Amndeep7
Copy link
Contributor

Created the PR (#1232). Need to sign the CLA which I'll do right now.

@Amndeep7 Amndeep7 mentioned this issue Mar 21, 2019
Merged
@Amndeep7
Copy link
Contributor

Oh whoops, forgot to add the documentation. Will add the commit for that shortly.

Amndeep7 added a commit to Amndeep7/earthenterprise that referenced this issue Mar 21, 2019
@Amndeep7
Modified documentation to also reference the ability to uninstall fro…
b42affe 
…m the installation directory (google#303)
Amndeep7 added a commit to Amndeep7/earthenterprise that referenced this issue Mar 21, 2019
@Amndeep7
Added version.txt to the set of files that need to be copied over on …
034c845 
…install so as to allow the uninstall scripts to state the version of the product (google#303)
@tst-nfarah-zz tst-nfarah-zz added this to the 5.3.0 milestone Mar 22, 2019
Amndeep7 added a commit to Amndeep7/earthenterprise that referenced this issue Mar 25, 2019
@Amndeep7
Updated release documentation and copyright notices (google#303)
9015ef9
tst-ccamp pushed a commit that referenced this issue Apr 4, 2019
@Amndeep7 @tst-ccamp
Copy uninstall scripts (#1232)
a16e2d2 
* Added copying uninstall scripts to the install staging directory (#303)
* Fixed bug causing SCons to have an InternalError (#303)
* Changed the temp staging area subdirectory for storing the uninstall scripts along with also copying over their dependency (#303)
* Added the uninstall scripts being copied from the staging area to the install directory (#303)
* Added version.txt to the set of files that need to be copied over on install so as to allow the uninstall scripts to state the version of the product (#303)
tst-ccamp added a commit to tst-ccamp/earthenterprise that referenced this issue Apr 4, 2019
@tst-ccamp
Issue google#303: Made documentation updates in 5.3.1 directory
7613fbd
@tst-ccamp tst-ccamp mentioned this issue Apr 4, 2019
Merged
tst-ccamp added a commit that referenced this issue Apr 24, 2019
@tst-ccamp
Issue #303: Made uninstallation scripts documentation updates in 5.3.…
46a3168 
…1 directory (#1262)

* Issue #303: Made documentation updates in 5.3.1 directory
* Fixed copyright year for release notes
@tst-nfarah-zz tst-nfarah-zz modified the milestones: 5.3.0, 5.3.1 Apr 24, 2019
husf-dsheremata added a commit to husf-dsheremata/earthenterprise that referenced this issue Jun 4, 2019
@husf-dsheremata
Merge remote-tracking branch 'upstream/master'
4c510c5 
* upstream/master:
  Issue google#303: Made uninstallation scripts documentation updates in 5.3.1 directory (google#1262)
  I1258 b update release notes (google#1267)
  1279 Fix getversion.py for Windows and update OpenGEE copyright year. (google#1280)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
beginner
Projects
None yet
Milestone
5.3.1
Development

No branches or pull requests
4 participants
@Amndeep7 @tst-ccamp @tst-lsavoie @tst-nfarah-zz