Issue 3626

Summary: Error in module dependency list in shell and tools
Product: Build Tools Reporter: Unknown <non-migrated>
Component: codeAssignee: Unknown <non-migrated>
Status: CLOSED FIXED QA Contact: issues@tools <issues>
Severity: Trivial    
Priority: P3 CC: issues
Version: current   
Target Milestone: ---   
Hardware: All   
OS: All   
Issue Type: DEFECT Latest Confirmation in: ---
Developer Difficulty: ---
Attachments:
Description Flags
Patch for shell/prj/build.lst
none
tools/prj/build.lst none

Description Unknown 2002-03-22 05:37:08 UTC 
The current module dependency list in the shell module is:
+sh	shell	: offapi rdbmaker NULL

However, the shell module is dependent on files generated by offuh, so the 
dependency list should look like:
+sh	shell	: offuh rdbmaker

offapi is a dependency of offuh, so there is no need to keep it in the list.
Comment 1 Unknown 2002-03-22 05:38:30 UTC 
Created attachment 1266 [details]
Patch for shell/prj/build.lst
Comment 2 Unknown 2002-03-22 05:39:20 UTC 
adding keyword
Comment 3 Unknown 2002-03-22 05:52:11 UTC 
This also needs to be applied to tools
Comment 4 Unknown 2002-03-22 05:52:54 UTC 
Created attachment 1267 [details]
tools/prj/build.lst
Comment 5 Unknown 2002-03-26 11:46:02 UTC 
I agree with you on shell/prj/build.lst and change the dependency 
from 'offapi' to 'offuh'. Indeed, building 'shell' without prior 
build of 'offuh' won't work. 

I to not agree, however, on tools/prj/build.lst. The file needed 
by tools stems from offapi, and I can build tools without offuh. 
So even though having offuh in the dependency there would lead to 
a correct build, 'offapi' as a dependency is more logical. 

Did you had another build experience in tools?
Comment 6 rt 2002-03-27 17:24:47 UTC 
OTOH, head revision of tools/prj/build.lst already contains a
dependency on offuh. Comment by Stephan Bergmann says: "Make tools
depend on offuh, because urlobj.hxx includes
com/sun/star/uno/Reference.hxx, which in turn includes generated
headers." Looks as if XInterface.hdl is needed to be delivered from
offuh.
Comment 7 Unknown 2002-03-28 09:24:08 UTC 
ok, hadn't seen the head revision. I added the dependency 
also on the stable branch and close this issue
Comment 8 michael.bemmer 2003-03-16 20:10:54 UTC 
As discussed with Armin, he thought these bug were closed, he sees all of them
as verified.
Comment 9 michael.bemmer 2003-03-16 20:14:34 UTC 
As discussed with Armin, he thought these bug were closed, he sees all of them
as verified.