From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 30451 invoked by alias); 12 Sep 2013 20:20:13 -0000 Mailing-List: contact kawa-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: kawa-owner@sourceware.org Received: (qmail 30442 invoked by uid 89); 12 Sep 2013 20:20:13 -0000 Received: from mail-wi0-f171.google.com (HELO mail-wi0-f171.google.com) (209.85.212.171) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Thu, 12 Sep 2013 20:20:13 +0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.6 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,KHOP_THREADED,NO_RELAYS autolearn=ham version=3.3.2 X-HELO: mail-wi0-f171.google.com Received: by mail-wi0-f171.google.com with SMTP id hm2so4103342wib.4 for ; Thu, 12 Sep 2013 13:20:09 -0700 (PDT) MIME-Version: 1.0 X-Received: by 10.180.73.40 with SMTP id i8mr23442647wiv.37.1379017209579; Thu, 12 Sep 2013 13:20:09 -0700 (PDT) Received: by 10.216.152.194 with HTTP; Thu, 12 Sep 2013 13:20:09 -0700 (PDT) In-Reply-To: References: Date: Thu, 12 Sep 2013 20:20:00 -0000 Message-ID: Subject: Re: Patch for wrong no declaration seen for command-line-arguments From: Matthieu Vachon To: Charles Turner Cc: "kawa@sourceware.org" Content-Type: text/plain; charset=ISO-8859-1 X-IsSubscribed: yes X-SW-Source: 2013-q3/txt/msg00060.txt.bz2 Hi Charles, Thank you for the procedure. I'm going to check this out later tonight. As for the failed tests, I'm going to take a look. Regards, Matt On Thu, Sep 12, 2013 at 4:09 PM, Charles Turner wrote: > Hi Matt, > > I can't comment on the patch as I haven't been following along. > > The test scripts you've written can be installed by adding the > filenames (in this case unknown1.scm and unknown2.scm) to the > testsuite/Makefile.am file.Specifically, add them to the variable > SCRIPTS_TO_RUN. You then need to regenerate the Makefiles using > autotools. I'm no expert, but aclocal && autoreconf has worked for me > in the past from the top-level kawa directory. If you then do a make > check, your tests will be run. For your patch, I got the following > results: > > # unknown1.scm passes > FAIL unknown2.scm: expected more diagnostics: \Qunknown2.scm:2:16: > warning - no declaration seen for command-line-arguments\E > # unknown2.scm fails > > Which will require further investigation. > > Hope that helps, > Charles. > > > On 12/09/2013, Matthieu Vachon wrote: >> Hi Per, >> >> Here my try at fixing the incorrect warning that occurs when >> generating a main method and using the `command-line-arguments`. It's >> probably not the optimal way to fix the issue but I hope it is good >> enough to be included in the trunk. >> >> I added two test cases but I'm unsure if the syntax I used is right. >> Talking about that, what is the procedure to run the test suite? I >> would like to know so I can test my patches when developing them prior >> to sending them to you. >> >> Regards, >> Matt >>