Issue 31082 - Fix some rough edges of cws tooling
Summary: Fix some rough edges of cws tooling
Status: CLOSED FIXED
Alias: None
Product: Build Tools
Classification: Code
Component: solenv (show other issues)
Version: current
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 2.0
Assignee: jens-heiner.rechtien
QA Contact: issues@tools
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-07-05 04:50 UTC by quetschke
Modified: 2006-03-14 21:03 UTC (History)
3 users (show)

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


Attachments
Patch for cws tooling (6.06 KB, patch)
2004-07-05 04:52 UTC, quetschke
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this issue.
Description quetschke 2004-07-05 04:50:40 UTC
The following patch polishes some of the rough edges.

It also allows to use COMMON_ENV_TOOLS, i.e. use the cws tools from a dir
outside of $SOLENV/bin with:

(insert your paths appropriately)
source winenv.set.sh
export PATH="$PATH:/cygdrive/e/w1/cwstoolsbin/bin"
export COMMON_ENV_TOOLS=/cygdrive/e/w1/cwstoolsbin/bin
Comment 1 quetschke 2004-07-05 04:52:00 UTC
Created attachment 16265 [details]
Patch for cws tooling
Comment 2 Martin Hollmichel 2004-07-05 10:45:06 UTC
dispatch to Heiner
Comment 3 jens-heiner.rechtien 2004-08-12 15:52:07 UTC
Committed to CWS hr5
Comment 4 jens-heiner.rechtien 2004-08-12 16:13:33 UTC
Close.
Comment 5 quetschke 2004-08-21 23:28:49 UTC
Hi Heiner,

you deviated a bit from fix to:
----
BEGIN {
    if ( !defined($ENV{SOLARENV}) ) {
        die "No environment found (environment variable SOLARENV is undefined)";
    }
    push(@lib_dirs, "$ENV{SOLARENV}/bin/modules");
    push(@lib_dirs, "$ENV{COMMON_ENV_TOOLS}/modules") if
defined($ENV{COMMON_ENV_TOOLS});
}
----
The difference is the order of SOLARENV and COMMON_ENV_TOOLS. I put
COMMON_ENV_TOOLS in front of SOLENV on purpose. This allows me to override
the checked out cws tools in SOLENV and use the ones chosen by COMMON_ENV_TOOLS.

Was this done on purpose or would you accept a patch to change to the order?

Volker
Comment 6 quetschke 2004-08-21 23:30:03 UTC
I meant:

> you deviated a bit from fix to:
-------  my  ------------^ 
Comment 7 jens-heiner.rechtien 2004-08-23 09:52:34 UTC
Hi Volker,

placing COMMON_ENV_TOOLS before SOLAR_ENV brakes the Hamburg build environment.
It's also logically not quite correct, because a workspace specific directory
like SOLAR_ENV should override a more general directory COMMON_ENV_TOOLS which
is meant serving all workspaces. This way we can make enhancements to the tools
without breaking all the old workspaces. The main problem is that we (that's
Hamburg RE) have several live workspaces online which all need to be active at
the same time - talk about the pain of always being able to create patches to 5
year old releases ... :-) 
Comment 8 quetschke 2004-08-24 17:17:01 UTC
Hi Heiner,

you're right. This way one can work in a cws to fix the tools, but I stepped
into this problem with cws 0815 where your patches for the cwstools are not
applied yet and they overrode my HEAD checkout in COMMON_ENV_TOOLS.

But time will solve this problem, newer milestones should work OK ;)
Comment 9 thorsten.ziehm 2005-08-10 14:19:10 UTC
This issue is integrated into a build for OOo2.0, but the 'target milestone'
isn't set. To have a better overview about all fixed and integrated tasks in
OOo2.0, I set the field 'target milestone' to OOo2.0.