Issue 87533 - to-be-DEV300_m5: /chrel.sed
Summary: to-be-DEV300_m5: /chrel.sed
Status: CLOSED FIXED
Alias: None
Product: Build Tools
Classification: Code
Component: code (show other issues)
Version: DEV300m4
Hardware: All Windows, all
: P1 (highest) Trivial (vote)
Target Milestone: OOo 3.0
Assignee: hjs
QA Contact: issues@tools
URL:
Keywords:
: 87675 (view as issue list)
Depends on:
Blocks:
 
Reported: 2008-03-27 23:15 UTC by pavel
Modified: 2008-05-06 18:39 UTC (History)
5 users (show)

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


Attachments
Patch for solenv (12.44 KB, patch)
2008-03-28 18:22 UTC, quetschke
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this issue.
Description pavel 2008-03-27 23:15:54 UTC
Build as I always do and in to-be-m5:

=============
Building module ucbhelper
=============
/cygdrive/f/home/pavel/BuildDir/ooo_DEV300_m5_src/ucbhelper/inc
-------------
/cygdrive/f/home/pavel/BuildDir/ooo_DEV300_m5_src/ucbhelper/source/client
-------------
/cygdrive/f/home/pavel/BuildDir/ooo_DEV300_m5_src/ucbhelper/source/provider
-------------
/cygdrive/f/home/pavel/BuildDir/ooo_DEV300_m5_src/ucbhelper/util
sed -f /chrel.sed ../wntmsci10.pro/slb/ucbhelper.lin >> 
../wntmsci10.pro/misc/ucbhelper4MSC_link.lst
sed: couldn't open file /chrel.sed: No such file or directory
dmake:  Error code 4, while making '../wntmsci10.pro/misc/ucbhelper4MSC_link.lst'
dmake:  '../wntmsci10.pro/misc/ucbhelper4MSC_link.lst' removed.

ERROR: Error 65280 occurred while making 
/cygdrive/f/home/pavel/BuildDir/ooo_DEV300_m5_src/ucbhelper/util
pavel@drogo:~/BuildDir/ooo_DEV300_m5_src/ucbhelper>
Comment 1 hjs 2008-03-28 15:16:28 UTC
hitting one of the dark corners of the env. that suddenly is in public use :(

the scrip itself is no magic, consisting of just 

s/\(\.\.\\\)\{2,4\}/..\\/g

which obviously won't work with cygwin. it also assumes that linking is alway
done one level below the module root...

so hacking a cygwin version (two backslashes to slash):

s/\(\.\.\/\)\{2,4\}/..\//g

placing it in solenv/bin and changing its usage ([_]tg_app.mk and [_]tg_shl.mk) from

$(COMMON_ENV_TOOLS)$/chrel.sed

to

$(SOLARENV)$/bin$/chrel.sed

could be a hacky workaround...
Comment 2 quetschke 2008-03-28 18:22:22 UTC
Created attachment 52351 [details]
Patch for solenv
Comment 3 quetschke 2008-03-28 18:23:53 UTC
I put your suggestions into a patch, see before, and it indeed makes ucbhelper
build.

Thanks. Masterfix?
Comment 4 hjs 2008-03-28 18:53:49 UTC
this "fix" wasn't meant to be committed as it is quite likely to break the 4nt
build here. also i'd like PRJ variable to appear in the sed script.
any suggestions how to modify chrel.sed welcome
Comment 5 hjs 2008-03-31 17:10:53 UTC
for now, having both sed (native and cygwin - shouldn't affect each other) lines
in chrel.sed seems to be they way to go to get this fix in fast. no smart
detection of the directory level but being able to link again...
i'll a bit of building with that and propose it as a masterfix if noone objects.
Comment 6 pavel 2008-04-01 08:28:11 UTC
*** Issue 87675 has been marked as a duplicate of this issue. ***
Comment 7 hjs 2008-04-02 18:34:27 UTC
proposed fix done as masterfix for m6
Comment 8 hjs 2008-05-06 18:39:40 UTC
.