To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=71742
                 Issue #|71742
                 Summary|rm -f'ing a target and then let cp check whether the s
                        |ource is newer than the target doesn't make sense
               Component|api
                 Version|680m193
                Platform|All
                     URL|
              OS/Version|All
                  Status|NEW
       Status whiteboard|
                Keywords|
              Resolution|
              Issue type|DEFECT
                Priority|P3
            Subcomponent|code
             Assigned to|jsc
             Reported by|cloph





------- Additional comments from [EMAIL PROTECTED] Sun Nov 19 09:09:39 -0800 
2006 -------
sdk_oo/pack/copying/makefile.mk contains recipes like:

$(DESTDIRDOCU)$/DevelopersGuide$/DevelopersGuide.pdf : DevelopersGuide.pdf
        +-$(MKDIRHIER) $(@:d)
        +-rm -f $@ >& $(NULLDEV)
        $(MY_COPY) DevelopersGuide.pdf $(DESTDIRDOCU)$/DevelopersGuide

With MY_COPY being defined in sdk_oo/util/makefile.pmk as 

MY_COPY=$(GNUCOPY) -u

-u option in gnu cp is:
"copy only when the SOURCE file is  newer  than  the  destination file or when
the destination file is missing"

Given that the destination file just was forcely removed, this check is 
superfluous.

Additionally, this seems (so far) to be the only place wher gnucp is actually
required. Everything else works as well with bsd cp (read: the standard cp on 
mac)

If nobody objects, I'll make this regular $(CP) calls in cloph04. 
If you want to really use the existence/newer than check, then don't use "gnucp
-u", but the "$^" placeholder in the recipe. (don't do the makesystem's job
yourself)

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to