automake-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Merging back java-work


From: Stefano Lattarini
Subject: Re: Merging back java-work
Date: Sun, 18 Sep 2011 17:37:25 +0200
User-agent: KMail/1.13.7 (Linux/2.6.30-2-686; KDE/4.6.5; i686; ; )

On Monday 12 September 2011, Stefano Lattarini wrote:
> Hello automakers.
> 
> The java-work branch contains mostly testsuite enahcements, but also a
> couple of bug fixes.  There is no point in merging such bug fixes in
> master only though, since we plan to basically rewrite the whole
> Automake's Java support in the next release, with heavy paradigm and
> API changes (see <http://debbugs.gnu.org/9088>).
> 
> My opinion is that we should cherry-pick the bug-fixing commits from
> java-work and merge them in master (so that they will be in 1.11.2),
>
I obviously meant "merge them in maint" here, sigh :-(

> and then merge java-work in master (mostly for the enhanced testsuite
> coverage, that I hope we could reuse, at least partly, for the new
> `JARS' primary).
> 
> The commits that I plan to cherry-pick for maint are the following two:
>  - d76b503883525aaab6048102cbfe96bbdd9a0a85
>    "java: allow both dist_JAVA and nodist_JAVA in the same Makefile.am"
>  - 01dede4422493957c23b3bc7ecfe596f93a58a54
>    "java: allow both JAVA and nobase_JAVA in the same Makefile.am"
> 
> I will proceed with this plan in a couple of days if there is no
> objection.
> 
> Then we'll have to tackle the more thorny problem of the deprecation
> of the older _JAVA primary ... but that's for when the new interface
> will be ready, so no need to rush.
> 
I've proceeded with this plan.  Merging with master, I've also found a
testuite bug, which I've fixed with the attached patch.

Regards,
  Stefano
From 8d97cb95019b04707fdd60b4d9994d0acc9c6554 Mon Sep 17 00:00:00 2001
Message-Id: <address@hidden>
From: Stefano Lattarini <address@hidden>
Date: Sun, 18 Sep 2011 17:32:50 +0200
Subject: [PATCH] java: fix various blunders in test 'java-mix.test'

* tests/java-mix.test: Add missing call to `set -e'.  Fix inverted
semantics (`.java' files are expected *not* to be distributed by
default, not the other way round).  Fix various typos in the name
of the `.java' files.  Correct other minor blunders.  Improve some
comments.
---
 ChangeLog           |    9 +++++++++
 tests/java-mix.test |   21 +++++++++++----------
 2 files changed, 20 insertions(+), 10 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 1f7a263..468c205 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2011-09-18  Stefano Lattarini  <address@hidden>
+
+       java: fix various blunders in test 'java-mix.test'
+       * tests/java-mix.test: Add missing call to `set -e'.  Fix inverted
+       semantics (`.java' files are expected *not* to be distributed by
+       default, not the other way round).  Fix various typos in the name
+       of the `.java' files.  Correct other minor blunders.  Improve some
+       comments.
+
 2011-04-23  Stefano Lattarini  <address@hidden>
 
        java: allow both JAVA and nobase_JAVA in the same Makefile.am
diff --git a/tests/java-mix.test b/tests/java-mix.test
index 1c8163a..d6a43e7 100755
--- a/tests/java-mix.test
+++ b/tests/java-mix.test
@@ -16,10 +16,12 @@
 
 # Check that the JAVA primary can be used freely in the same Makefile.am
 # with proper combinations of the `dist_', `nodist_' and `nobase_'
-# modifiers .
+# modifiers.  Also check that `.java' files are not ditributed by default.
 
 . ./defs || Exit 1
 
+set -e
+
 cat >> configure.in << 'END'
 AC_OUTPUT
 END
@@ -37,19 +39,18 @@ nobase_java_JAVA = ClassNobase.java
 nobase_dist_java_JAVA = ClassNobaseDist.java
 nobase_nodist_java_JAVA = ClassNobaseNodist.java
 
-Class3.java Class6.java:
+Class.java ClassNodist.java ClassNobase.java ClassNobaseNodist.java:
        @echo '$@ should not be generated!' >&2; exit 1
 END
 
-: > Class.java
 : > ClassDist.java
-: > ClassNobase.java
 : > ClassNobaseDist.java
 
 $ACLOCAL
 $AUTOCONF
-# Automake used to display non-fatal warnings with this test,
-# but those must be seen as a failure by us.
+# Automake used to display non-fatal warnings with this test, but
+# they were unexpected, so we want to consider them as failures in
+# this test.
 $AUTOMAKE 2>stderr || { cat stderr >&2; Exit 1; }
 cat stderr >&2
 test ! -s stderr
@@ -58,11 +59,11 @@ test ! -s stderr
 
 $MAKE distdir
 ls -l $distdir # For debugging.
-test -f $distdir/Class.java
+test ! -f $distdir/Class.java
 test -f $distdir/ClassDist.java
-test -f $distdir/NobaseClass.java
-test -f $distdir/NobaseClassDist.java
+test ! -f $distdir/ClassNobase.java
+test -f $distdir/ClassNobaseDist.java
 test ! -f $distdir/ClassNodist.java
-test ! -f $distdir/NobaseClassNodist.java
+test ! -f $distdir/ClassNobaseNodist.java
 
 :
-- 
1.7.2.3


reply via email to

[Prev in Thread] Current Thread [Next in Thread]