LP1850156: Remove Browsers From *-developer Targets

Because the developer targets are currently necessary for a from-git
installation, both Firefox and Chrome are removed and the docs updated
to note that you should install one or both if you want to run the tests.

Signed-off-by: Jason Boyer <JBoyer@equinoxOLI.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>

diff --git a/Open-ILS/src/extras/install/Makefile.debian-buster b/Open-ILS/src/extras/install/Makefile.debian-buster
index 623e3a6..ad20c98 100644 (file)
--- a/Open-ILS/src/extras/install/Makefile.debian-buster
+++ b/Open-ILS/src/extras/install/Makefile.debian-buster
@@ -130,9 +130,7 @@ PGSQL_SERVER_DEBS_10 = \
 DEVELOPER_DEBS = \
        automake \
        autoconf \
-       libtool \
-       chromium \
-       firefox-esr
+       libtool
 
 PACKAGER_DEBS = \
        asciidoc \

diff --git a/Open-ILS/src/extras/install/Makefile.debian-jessie b/Open-ILS/src/extras/install/Makefile.debian-jessie
index 545a1a5..703ece0 100644 (file)
--- a/Open-ILS/src/extras/install/Makefile.debian-jessie
+++ b/Open-ILS/src/extras/install/Makefile.debian-jessie
@@ -131,9 +131,7 @@ PGSQL_SERVER_DEBS_10 = \
 DEVELOPER_DEBS = \
        automake \
        autoconf \
-       libtool \
-       chromium \
-       firefox-esr
+       libtool
 
 PACKAGER_DEBS = \
        asciidoc \

diff --git a/Open-ILS/src/extras/install/Makefile.debian-stretch b/Open-ILS/src/extras/install/Makefile.debian-stretch
index 93221e3..fb04c20 100644 (file)
--- a/Open-ILS/src/extras/install/Makefile.debian-stretch
+++ b/Open-ILS/src/extras/install/Makefile.debian-stretch
@@ -130,9 +130,7 @@ PGSQL_SERVER_DEBS_10 = \
 DEVELOPER_DEBS = \
        automake \
        autoconf \
-       libtool \
-       chromium \
-       firefox-esr
+       libtool
 
 PACKAGER_DEBS = \
        asciidoc \

diff --git a/Open-ILS/src/extras/install/Makefile.ubuntu-bionic b/Open-ILS/src/extras/install/Makefile.ubuntu-bionic
index a734286..760d20f 100644 (file)
--- a/Open-ILS/src/extras/install/Makefile.ubuntu-bionic
+++ b/Open-ILS/src/extras/install/Makefile.ubuntu-bionic
@@ -133,9 +133,7 @@ PGSQL_SERVER_DEBS_10 = \
 DEVELOPER_DEBS = \
        automake \
        autoconf \
-       libtool \
-       chromium-browser \
-       firefox
+       libtool
 
 PACKAGER_DEBS = \
        asciidoc \

diff --git a/Open-ILS/src/extras/install/Makefile.ubuntu-focal b/Open-ILS/src/extras/install/Makefile.ubuntu-focal
index b002042..1db9acf 100644 (file)
--- a/Open-ILS/src/extras/install/Makefile.ubuntu-focal
+++ b/Open-ILS/src/extras/install/Makefile.ubuntu-focal
@@ -134,9 +134,7 @@ PGSQL_SERVER_DEBS_10 = \
 DEVELOPER_DEBS = \
        automake \
        autoconf \
-       libtool \
-       chromium-browser \
-       firefox
+       libtool
 
 PACKAGER_DEBS = \
        asciidoc \

diff --git a/docs/modules/installation/pages/server_installation.adoc b/docs/modules/installation/pages/server_installation.adoc
index 694dc06..fcb4019 100644 (file)
--- a/docs/modules/installation/pages/server_installation.adoc
+++ b/docs/modules/installation/pages/server_installation.adoc
@@ -82,13 +82,14 @@ make -f Open-ILS/src/extras/Makefile.install <osname>
 3. OPTIONAL: Developer additions
 +
 To perform certain developer tasks from a Git source code checkout, 
-including the testing of the Angular web client components,
-additional packages may be required.  As the *root* Linux account:
+additional packages are required.  As the *root* Linux account:
 +
  * To install packages needed for retrieving and managing web dependencies,
    use the <osname>-developer Makefile.install target.  Currently, 
    this is only needed for building and installing the web
    staff client.
+ * To optionally run Angular and AngularJS tests you will need to manually
+   install Firefox and your choice of Chromium or Chrome.
 +
 [source, bash]
 ------------------------------------------------------------------------------
@@ -141,11 +142,11 @@ npm install   # fetch JS dependencies
 +
 [source,sh]
 ------------------------------------------------------------------------------
-# build, concat+minify
 npm run build-prod
 ------------------------------------------------------------------------------
 +
-4. OPTIONAL: Test web client code if the <osname>-developer packages were installed.
+4. OPTIONAL: Test web client code if the <osname>-developer packages and
+   the necessary browsers are installed.
    CHROME_BIN should be set to the path to chrome or chromimum, e.g.,
    `/usr/bin/chromium`:
 +
@@ -177,11 +178,11 @@ npm install   # fetch JS dependencies
 +
 [source,sh]
 ------------------------------------------------------------------------------
-# build and run tests
 ng build --prod
 ------------------------------------------------------------------------------
 +
-4. OPTIONAL: Test eg2 web client code if the <osname>-developer packages were installed:
+4. OPTIONAL: Test eg2 web client code if the <osname>-developer packages and
+   the necessary browsers are installed:
    CHROME_BIN should be set to the path to chrome or chromimum, e.g.,
    `/usr/bin/chromium`:
 +