Issue 8181 - config_office/set_soenv.1 ignores the current path when configuring
Summary: config_office/set_soenv.1 ignores the current path when configuring
Status: CLOSED FIXED
Alias: None
Product: Build Tools
Classification: Code
Component: code (show other issues)
Version: OOo 1.0.1
Hardware: PC Linux, all
: P2 Trivial (vote)
Target Milestone: ---
Assignee: foskey
QA Contact: issues@installation
URL:
Keywords:
: 12246 (view as issue list)
Depends on:
Blocks:
 
Reported: 2002-10-09 16:13 UTC by tusharteredesai
Modified: 2013-08-07 15:35 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
append current path to path set in LinuxIntelEnv.Set (462 bytes, patch)
2002-10-09 16:15 UTC, tusharteredesai 		no flags Details | Diff
Honor the user's path before using the standard system wide locations. (615 bytes, patch)
2003-03-02 17:35 UTC, tusharteredesai 		no flags Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description tusharteredesai 2002-10-09 16:13:58 UTC 
When running configure, the set_soenv.1 script ignores the current path when
creating LinuxIntelEnv.Set. This creates a problem when the required binaries
are in non-standard locations. The following patch fixes that by appending the
current path to the path variable that is written to LinuxIntelEnv.Set.

diff -Nur oo_1.0_src.orig/config_office/set_soenv.1
oo_1.0_src/config_office/set_soenv.1
--- oo_1.0_src.orig/config_office/set_soenv.1   2002-04-17 15:41:31.000000000 -0500
+++ oo_1.0_src/config_office/set_soenv.1    2002-07-14 13:43:30.000000000 -0500
@@ -1057,6 +1059,7 @@
                         $ps.$USR.$BIN.$ds."X11".
            $ps.$USR.$ds."X11".$BIN.
                         $ps.$USR.$LIB;
+   $PATH             .= $oldPATH;
 }
 elsif ($platform eq "$Winnt")
 {
Comment 1 tusharteredesai 2002-10-09 16:15:23 UTC 
Created attachment 3100 [details]
append current path to path set in LinuxIntelEnv.Set
Comment 2 Olaf Felka 2002-10-14 15:58:20 UTC 
Hi Sander,
please have a look.
Comment 3 tusharteredesai 2003-01-20 01:33:42 UTC 
Any comments on this one? There has been no feedback (positive or
negative) on this one ever since it was filed.
Comment 4 tusharteredesai 2003-03-02 17:34:01 UTC 
I have attached a new (and better) patch that utilizes the path set by
the user properly. The patch is against OpenOffice 1.0.2 released
source. The patch moves the values of the current PATH to the begining
of the PATH variable thereby ensuring that the binaries in the user's
path are picked up before the standard locations such as /usr/bin. I
came across the problem because OpenOffice fails to compile with
bison-1.875 and even though I installed bison-1.75 in /opt/bison-1.75
and added /opt/bison-1.75/bin to the front of my PATH variable, the
build kept using /usr/bin/bison instead of /opt/bison-1.75/bin/bison.
Comment 5 tusharteredesai 2003-03-02 17:35:14 UTC 
Created attachment 4956 [details]
Honor the user's path before using the standard system wide locations.
Comment 6 chris 2003-03-03 16:24:13 UTC 
Ken, please have a look.
Comment 7 foskey 2003-03-13 12:29:50 UTC 
I have tested the patch and it causes no problem in LInux.  Will
commit with next development tree.
Comment 8 foskey 2003-03-18 03:59:52 UTC 
*** Issue 11307 has been marked as a duplicate of this issue. ***
Comment 9 foskey 2003-03-18 11:04:27 UTC 
*** Issue 12246 has been marked as a duplicate of this issue. ***
Comment 10 foskey 2003-03-28 12:08:46 UTC 
Applied to cws_srx644_ooo20030412
Comment 11 foskey 2003-06-18 14:50:21 UTC 
Please verify and close this issue.
Comment 12 foskey 2003-07-31 02:37:03 UTC 
Closing