To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=48087


User vq changed the following:

                  What    |Old value                 |New value
================================================================================
                    Status|NEW                       |STARTED
--------------------------------------------------------------------------------




------- Additional comments from [EMAIL PROTECTED] Wed May 10 10:47:47 -0700 
2006 -------
The previous patch fixes the looping for note 3 from above and adds a few 
comments
and warnings/sanity checks.

Looking at the code and comparing it with the manpage, the manpage is wrong.

It says:
              %.c : %.y yaccsrc/%.y ; recipe...

       is a short form for the construct:

              %.c : %.y ; recipe...
              %.c : yaccsrc/%.y ; recipe...

But the ruleparsing handles it as if *both* prerequisites are needed to match
to choose this meta rule. This obviously fails later in the interference
algorithm, see examples above, but the target is definitely not split if more
than one prerequisite occur ...
unless the :| operator is used. (More about that later.)

Also the code handles:
     <%-target_1> .. <%-target_n> : <prereqs ; <recipe>
like
     <%-target_1> : <prereqs ; <recipe>
     ..
     <%-target_n> : <prereqs ; <recipe>

This wasn't documented at all, and only worked reliably when :| was used
(like it already is in OOo, see issue 65122). It now also works with
%-targets when : is used.
(Before %a %b : %c %d ; <recipe> was split into:
%a : %c %d ; <recipe>
%b : ; <recipe>
)

The :| operator now handles
<%-target_1> .. <%-target_n> : <prereq_1> ... <prereq_m> ; <recipe>
exactly as expected and creates n*m target rules.

Committed the previous patch to CWS dmake45.

---------------------------------------------------------------------
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