# [PATCH v2 1/4] engine.pl: Fix a recent breakage of the buildsystem generator

2 messages from 2010-01-22 to 2010-01-22. Participants: Ramsay Jones, Sebastian Schuberth.
Thread: https://gitlist.dev/t/22345

## Ramsay Jones, 2010-01-22 20:00

Subject: [PATCH v2 1/4] engine.pl: Fix a recent breakage of the buildsystem generator
Message-ID: <4B5A03EF.1030208@ramsay1.demon.co.uk>
URL: https://gitlist.dev/e/4B5A03EF.1030208%40ramsay1.demon.co.uk

```

Commit ade2ca0c (Do not try to remove directories when removing
old links, 27-10-2009) added an expression to a 'test' using an
'-o' or connective. This resulted in the buildsystem generator
mistaking a conditional 'rm' for a linker command. In order to
fix the breakage, we filter out all 'test' commands before then
attempting to identify the commands of interest.

Signed-off-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
---

Changes from v1, in response to comments from Pete Harlan and
Sebastian Schuberth:

    - remove second (useless) regex from condition in filter code
    - remove the commented out version of the original filter code
    - re-worded commit message to reflect the above changes

ATB,
Ramsay Jones

 contrib/buildsystems/engine.pl |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/contrib/buildsystems/engine.pl b/contrib/buildsystems/engine.pl
index d506717..8f890fb 100644
--- a/contrib/buildsystems/engine.pl
+++ b/contrib/buildsystems/engine.pl
@@ -135,6 +135,11 @@ sub parseMakeOutput
             }
         } while($ate_next);
 
+        if ($text =~ /^test /) {
+            # options to test (eg -o) may be mistaken for linker options
+            next;
+        }
+
         if($text =~ / -c /) {
             # compilation
             handleCompileLine($text, $line);
@@ -180,9 +185,6 @@ sub parseMakeOutput
 #        } elsif ($text =~ /generate-cmdlist\.sh/) {
 #            # command for generating list of commands
 #
-#        } elsif ($text =~ /^test / && $text =~ /|| rm -f /) {
-#            # commands removing executables, if they exist
-#
 #        } elsif ($text =~ /new locations or Tcl/) {
 #            # command for detecting Tcl/Tk changes
 #
-- 
1.6.6

```

## Sebastian Schuberth, 2010-01-22 23:48

Subject: Re: [PATCH v2 1/4] engine.pl: Fix a recent breakage of the buildsystem generator
Message-ID: <4B5A3958.5060505@gmail.com>
URL: https://gitlist.dev/e/4B5A3958.5060505%40gmail.com
In-Reply-To: <4B5A03EF.1030208@ramsay1.demon.co.uk>

```
On 22.01.2010 21:00, Ramsay Jones wrote:

> Commit ade2ca0c (Do not try to remove directories when removing
> old links, 27-10-2009) added an expression to a 'test' using an
> '-o' or connective. This resulted in the buildsystem generator
> mistaking a conditional 'rm' for a linker command. In order to
> fix the breakage, we filter out all 'test' commands before then
> attempting to identify the commands of interest.
>
> Signed-off-by: Ramsay Jones<ramsay@ramsay1.demon.co.uk>
> ---
>
> Changes from v1, in response to comments from Pete Harlan and
> Sebastian Schuberth:
>
>      - remove second (useless) regex from condition in filter code
>      - remove the commented out version of the original filter code
>      - re-worded commit message to reflect the above changes

Acked-by: Sebastian Schuberth <sschuberth@gmail.com>

Thanks again!

-- 
Sebastian Schuberth

```