loolwsd/Common.hpp     |    1 +
 loolwsd/LOOLWSD.cpp    |    4 +---
 loolwsd/LOOLWSD.hpp    |    1 -
 loolwsd/loolwsd.xml.in |    1 -
 4 files changed, 2 insertions(+), 5 deletions(-)

New commits:
commit b7e0766fea74f851ccda9b6b172357029eed05db
Author: Ashod Nakashian <ashod.nakash...@collabora.co.uk>
Date:   Mon Jun 27 08:27:08 2016 -0400

    loolwsd: removed lo_jail_subpath
    
    Really unused setting that caused deployment complications.
    
    Change-Id: If837fca4c904e82b23babd4a1f0f3f15b5ef57f4
    Reviewed-on: https://gerrit.libreoffice.org/26705
    Reviewed-by: Ashod Nakashian <ashnak...@gmail.com>
    Tested-by: Ashod Nakashian <ashnak...@gmail.com>

diff --git a/loolwsd/Common.hpp b/loolwsd/Common.hpp
index f0be984..f76b823 100644
--- a/loolwsd/Common.hpp
+++ b/loolwsd/Common.hpp
@@ -36,6 +36,7 @@ constexpr auto FIFO_PATH = "pipe";
 constexpr auto JAILED_DOCUMENT_ROOT = "/user/docs/";
 constexpr auto CHILD_URI = "/loolws/child?";
 constexpr auto NEW_CHILD_URI = "/loolws/newchild?";
+constexpr auto LO_JAIL_SUBPATH = "lo";
 
 // The client port number, both loolwsd and the kits have this.
 extern int ClientPortNumber;
diff --git a/loolwsd/LOOLWSD.cpp b/loolwsd/LOOLWSD.cpp
index 903dad3..cef575f 100644
--- a/loolwsd/LOOLWSD.cpp
+++ b/loolwsd/LOOLWSD.cpp
@@ -1174,7 +1174,6 @@ std::string LOOLWSD::Cache = LOOLWSD_CACHEDIR;
 std::string LOOLWSD::SysTemplate;
 std::string LOOLWSD::LoTemplate;
 std::string LOOLWSD::ChildRoot;
-std::string LOOLWSD::LoSubPath = "lo";
 std::string LOOLWSD::ServerName;
 std::string LOOLWSD::FileServerRoot;
 std::string LOOLWSD::LOKitVersion;
@@ -1273,7 +1272,6 @@ void LOOLWSD::initialize(Application& self)
     SysTemplate = getPathFromConfig("sys_template_path");
     LoTemplate = getPathFromConfig("lo_template_path");
     ChildRoot = getPathFromConfig("child_root_path");
-    LoSubPath = getPathFromConfig("lo_jail_subpath");
     ServerName = config().getString("server_name");
     FileServerRoot = getPathFromConfig("file_server_root_path");
     NumPreSpawnedChildren = getUIntConfigValue(conf, "num_prespawn_children", 
1);
@@ -1427,7 +1425,7 @@ Process::PID LOOLWSD::createForKit()
 {
     Process::Args args;
 
-    args.push_back("--losubpath=" + LOOLWSD::LoSubPath);
+    args.push_back("--losubpath=" + std::string(LO_JAIL_SUBPATH));
     args.push_back("--systemplate=" + SysTemplate);
     args.push_back("--lotemplate=" + LoTemplate);
     args.push_back("--childroot=" + ChildRoot);
diff --git a/loolwsd/LOOLWSD.hpp b/loolwsd/LOOLWSD.hpp
index d82e026..76e942e 100644
--- a/loolwsd/LOOLWSD.hpp
+++ b/loolwsd/LOOLWSD.hpp
@@ -40,7 +40,6 @@ public:
     static std::string SysTemplate;
     static std::string LoTemplate;
     static std::string ChildRoot;
-    static std::string LoSubPath;
     static std::string ServerName;
     static std::string FileServerRoot;
     static std::string LOKitVersion;
diff --git a/loolwsd/loolwsd.xml.in b/loolwsd/loolwsd.xml.in
index 3399ef8..c55cf30 100644
--- a/loolwsd/loolwsd.xml.in
+++ b/loolwsd/loolwsd.xml.in
@@ -7,7 +7,6 @@
     <sys_template_path desc="Path to a template tree with shared libraries etc 
to be used as source for chroot jails for child processes." type="path" 
relative="true" default="systemplate"></sys_template_path>
     <lo_template_path desc="Path to a LibreOffice installation tree to be 
copied (linked) into the jails for child processes. Should be on the same file 
system as systemplate." type="path" relative="false" 
default="/opt/collaboraoffice5.0"></lo_template_path>
     <child_root_path desc="Path to the directory under which the chroot jails 
for the child processes will be created. Should be on the same file system as 
systemplate and lotemplate. Must be an empty directory." type="path" 
relative="true" default="jails"></child_root_path>
-    <lo_jail_subpath desc="Relative path where the LibreOffice installation 
will be copied inside a jail. Set relative to false to prevent expanding the 
path." type="path" relative="false" default="lo">lo</lo_jail_subpath>
 
     <server_name desc="Hostname:port of the server running loolwsd. If empty, 
it's derived from the request." type="string" default=""></server_name>
     <file_server_root_path desc="Path to the directory that should be 
considered root for the file server. This should be the directory containing 
loleaflet." type="path" relative="true" 
default="../loleaflet/../"></file_server_root_path>
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to