Been scratching my head on this one.
Another error is in the target sane-gcc-compiler, same kind of thing.
Turns out this one was Solaris only, and the older version of GNU make
seemed to be silent (3.78.1)
but the newer one (3.81) complained about a missing TAB character.
Our RE teams use the older GNU make 3.78.1, I've been trying to
advance (if you want to
call it that) to 3.81, or even 3.82 if possible, but it's not been as
easy as originally planned
due to the issues with Windows, MKS, and escaped quotes in rules. But
3.82 may have a fix
for this.
Could we be using a different GNU make version on some of these Linux
systems?
-kto
On Nov 22, 2010, at 2:33 PM, David Holmes wrote:
Any enlightenment on how this only just started happening? Has the
alsa check been disabled previously?
David
Kelly O'Hair said the following on 11/23/10 08:06:
On Nov 22, 2010, at 1:53 PM, Dr Andrew John Hughes wrote:
On 22:15 Mon 22 Nov , Patrick Reinhart wrote:
Am 22.11.10 22:09, schrieb Dr Andrew John Hughes:
I'm quite puzzled as to how this hasn't been spotted before now,
but I
tried to build jdk7/jdk7 today (b118 from hg), using exactly the
same
script as I usually do, and immediately failed due to a missing
separator in the jdk Sanity.gmk Makefile:
make[1]: Entering directory `/home/andrew/projects/openjdk/
upstream/build/jdk/make'
/home/andrew/projects/openjdk/upstream/build/jdk/make/common/
shared/Sanity.gmk:1392: *** missing separator (did you mean TAB
instead of 8 spaces?). Stop.
make[1]: Leaving directory `/home/andrew/projects/openjdk/
upstream/build/jdk/make'
I've just confirmed it's also broken in the build tree and the
icedtea tree
also fails (which is where I first hit the error).
It seems the sane-alsa-headers target is completely broken.
What's puzzling
is the changes occur in:
http://hg.openjdk.java.net/jdk7/jdk7/jdk/rev/30bf00392b6d
changeset: 914:30bf00392b6d
parent: 796:d8eb2738db6b
user: ohair
date: Sat Jan 31 17:31:21 2009 -0800
summary: 6799141: Build with --hash-style=both so that
binaries can work on SuSE 10
which is nearly two years old.
I managed to make some headway:
* The target uses a mix of tabs and spaces. Replacing with all
tabs gets things further.
* The next issue is fixed by changing $${alsa_version) to $$
{alsa_version}.
It then fails because we have a conditional structured as:
if [ "$(ALSA_CHECK)" != "same" -a "$(ALSA_CHECK)" !=
"newer" ] ; then \
...
fi \
else \
...
fi
There seems to be an if statement missing as, otherwise, having
REQUIRED_ALSA_VERSION
defined always results in an error. I assume there should be a
top level if statement,
similar to the @if [ -f "$(ALSA_VERSION_CHECK)" ]; removed by
this changeset. This would
also explain why the if block is printed when usually such
things are silent.
Anyone care to enlighten us as to the missing if statement? I'd
also love to know how
this has only just started biting me now.
Thanks,
Hi Andrew,
See my statement and correcting diff
http://mail.openjdk.java.net/pipermail/build-dev/2010-November/003578.html
when I tried to get the build running under Fedora 14...
Regards Patrick
Yes, as described in my original e-mail, I've already made those
changes.
I don't believe the final else block is redundant. An if check
seems to be missing, which
is further reinforced if you take a look at other checks in the
file and the CheckVersions macro;
we should be looking for ALSA_CHECK being "missing".
I've posted a webrev here: http://cr.openjdk.java.net/~andrew/build/webrev.07/
which contains corrected indenting, the alsa_version fix and the
additional if test.
Kelly, does this look ok to push? If so, can I have a bug ID for it?
Looks fine. Feel free to push.
7000225: Sanity check on sane-alsa-headers is broken
Thank you. I filed the bug last week I think, just been so busy
lately. :^(
-kto
Thanks,
--
Andrew :)
Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)
Support Free Java!
Contribute to GNU Classpath and the OpenJDK
http://www.gnu.org/software/classpath
http://openjdk.java.net
PGP Key: 94EFD9D8 (http://subkeys.pgp.net)
Fingerprint = F8EF F1EA 401E 2E60 15FA 7927 142C 2591 94EF D9D8
