Issue 28827 - Makefile.mk: Force javac to generate $(JAVACLASSFILES)
Summary: Makefile.mk: Force javac to generate $(JAVACLASSFILES)
Status: ACCEPTED
Alias: None
Product: Build Tools
Classification: Code
Component: solenv (show other issues)
Version: 680m24
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-05-06 16:22 UTC by Stephan Bergmann
Modified: 2017-05-20 10:47 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description Stephan Bergmann 2004-05-06 16:22:57 UTC
In solenv/inc/javaunittest.mk 1.3.26.1, I would like to ensure that javac is run
(to generate $(JAVACLASSFILES)) whenever the javamaker has been run (and has
touched its $(MISC)$/$(TARGET).javamaker.flag file).  But the dependency

  $(JAVACLASSFILES): $(MISC)$/$(TARGET).javamaker.flag

does not work reliably---javac is only run if dmake is run two times.
Comment 1 Stephan Bergmann 2005-08-18 12:44:05 UTC
For an example, see comphelper/qa/propertysetmixin/makefile.mk rev. 1.2 lines
149--151 (note: will be moved to cppuhelper/qa/propertysetmixin on CWS
skeletonmaker).
Comment 2 hjs 2007-03-09 15:59:54 UTC
hmm, seems to work for me when doing a small change to the dependency line.
might be not intuitive...

RCS file: /cvs/udk/cppuhelper/qa/propertysetmixin/makefile.mk,v
retrieving revision 1.3
diff -u -r1.3 makefile.mk
--- makefile.mk 25 Jan 2007 12:07:39 -0000      1.3
+++ makefile.mk 9 Mar 2007 15:56:48 -0000
@@ -128,7 +128,7 @@

 # The following dependency (to execute javac whenever javamaker has run) does
 # not work reliably, see #i28827#:
-$(JAVAFILES) $(JAVACLASSFILES): $(MISC)$/$(TARGET)$/javamaker.flag
+$(JAVATARGET) $(JAVACLASSFILES): $(MISC)$/$(TARGET)$/javamaker.flag

 $(MISC)$/$(TARGET)$/$(TARGET).uno.jar: $(JAVACLASSFILES) \
         $(MISC)$/$(TARGET)$/javamaker.flag manifest
Comment 3 Stephan Bergmann 2007-03-09 16:10:07 UTC
Will search for places that need to be adapted.
Comment 4 Marcus 2017-05-20 10:47:49 UTC
Reset assigne to the default "issues@openoffice.apache.org".