linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Anton Altaparmakov <aia21@cam.ac.uk>
To: tpepper@vato.org
Cc: f5ibh <f5ibh@db0bm.ampr.org>, linux-kernel@vger.kernel.org
Subject: Re: 2.4.9 does not compile [PATCH]
Date: Thu, 16 Aug 2001 23:21:17 +0100	[thread overview]
Message-ID: <5.1.0.14.2.20010816231824.00a94b70@pop.cus.cam.ac.uk> (raw)
In-Reply-To: <20010816144109.A5094@cb.vato.org>
In-Reply-To: <200108162111.XAA07177@db0bm.ampr.org> <200108162111.XAA07177@db0bm.ampr.org>

Wrong fix. Jules Colding already sent correct one (an hour ago or so).

At 22:41 16/08/2001, tpepper@vato.org wrote:
>Confirmed here.  Looks like a pretty obvious goof to me.

Yes. Someone forgot to add kernel.h include after they removed the ntfs 
specific macros... Grr...

>   Does the following fix it for you?
>
>--- linux-2.4.9/fs/ntfs/unistr.c.orig   Thu Aug 16 14:35:03 2001
>+++ linux-2.4.9/fs/ntfs/unistr.c        Thu Aug 16 14:35:15 2001
>@@ -96,7 +96,7 @@
>         __u32 cnt;
>         wchar_t c1, c2;
>
>-       for (cnt = 0; cnt < min(unsigned int, name1_len, name2_len); ++cnt)
>+       for (cnt = 0; cnt < min(name1_len, name2_len); ++cnt)

This won't fix it because the min macro still will not exist.

Anton

>         {
>                 c1 = le16_to_cpu(*name1++);
>                 c2 = le16_to_cpu(*name2++);
>
> > unistr.c: In function `ntfs_collate_names':
> > unistr.c:99: warning: implicit declaration of function `min'
> > unistr.c:99: parse error before `unsigned'
> > unistr.c:99: parse error before `)'
> > unistr.c:97: warning: `c1' might be used uninitialized in this function
> > unistr.c: At top level:
> > unistr.c:118: parse error before `if'
> > unistr.c:123: warning: type defaults to `int' in declaration of `c1'
> > unistr.c:123: `name1' undeclared here (not in a function)
> > unistr.c:123: warning: data definition has no type or storage class
> > unistr.c:124: parse error before `if'
> > make[3]: *** [unistr.o] Erreur 1
> > make[3]: Leaving directory `/usr/src/kernel-sources-2.4.9/fs/ntfs'
> > make[2]: *** [_modsubdir_ntfs] Erreur 2
> > make[2]: Leaving directory `/usr/src/kernel-sources-2.4.9/fs'
> > make[1]: *** [_mod_fs] Erreur 2
> > make[1]: Leaving directory `/usr/src/kernel-sources-2.4.9'
> > make: *** [stamp-build] Erreur 2
>-
>To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>the body of a message to majordomo@vger.kernel.org
>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>Please read the FAQ at  http://www.tux.org/lkml/

-- 
   "Nothing succeeds like success." - Alexandre Dumas
-- 
Anton Altaparmakov <aia21 at cam.ac.uk> (replace at with @)
Linux NTFS Maintainer / WWW: http://linux-ntfs.sf.net/
ICQ: 8561279 / WWW: http://www-stu.christs.cam.ac.uk/~aia21/


  parent reply	other threads:[~2001-08-16 22:21 UTC|newest]

Thread overview: 56+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-08-16 21:11 2.4.9 does not compile f5ibh
2001-08-16 21:41 ` 2.4.9 does not compile [PATCH] tpepper
2001-08-16 22:10 ` 2.4.9 does not compile [better PATCH] tpepper
2001-08-16 22:21 ` Anton Altaparmakov [this message]
2001-08-16 22:28 ` Anton Altaparmakov
2001-08-17  4:23   ` Jeff Chua
2001-08-16 22:31 ` 2.4.9 does not compile [PATCH] David S. Miller
2001-08-16 23:11   ` David S. Miller
2001-08-16 23:13   ` Daniel Phillips
2001-08-16 23:14     ` Alan Cox
2001-08-16 23:40       ` David S. Miller
2001-08-17 21:58         ` David S. Miller
2001-08-16 23:35     ` Anton Altaparmakov
2001-08-17  1:46       ` Jeff Dike
     [not found]         ` <Your message of "Fri, 17 Aug 2001 00:35:02 +0100." <5.1.0.14.2.20010817002825.00b1e4e0@pop.cus.cam.ac.uk>
2001-08-17  1:22           ` Anton Altaparmakov
2001-08-17  1:49             ` Andrew Morton
2001-08-17  2:05               ` Daniel Phillips
2001-08-17  1:53             ` David S. Miller
2001-08-17  3:55             ` Jeff Dike
2001-08-16 23:38   ` David S. Miller
2001-08-17  1:24     ` Roman Zippel
2001-08-17  1:50     ` David S. Miller
2001-08-17  2:29       ` Roman Zippel
2001-08-17  2:38       ` David S. Miller
2001-08-17  2:53         ` Roman Zippel
2001-08-17  2:59         ` David S. Miller
2001-08-17  3:23           ` Roman Zippel
2001-08-17  3:37           ` David S. Miller
2001-08-17  3:59             ` Roman Zippel
2001-08-17  8:35             ` Anton Altaparmakov
2001-08-17  4:29           ` Ben LaHaise
2001-08-17  6:07           ` David S. Miller
2001-08-17  3:08         ` Roman Zippel
2001-08-17  3:13         ` David S. Miller
2001-08-17  3:39           ` Roman Zippel
2001-08-17  3:44           ` David S. Miller
2001-08-17 17:30   ` Rik van Riel
2001-08-16 23:22 ` Anton Altaparmakov
2001-08-17  1:06 ` 2.4.9 does not compile (adaptec new driver) Luigi Genoni
     [not found] <no.id>
2001-08-16 22:41 ` 2.4.9 does not compile [PATCH] Alan Cox
2001-08-16 22:48 ` David S. Miller
2001-08-17 21:12   ` Jes Sorensen
2001-08-16 22:55 ` Alan Cox
2001-08-16 23:02 ` David S. Miller
2001-08-17 21:59   ` David S. Miller
2001-08-16 23:08 ` Alan Cox
2001-08-17  9:11 ` Alan Cox
2001-08-17  9:17 ` Alan Cox
2001-08-17  9:25 ` Alan Cox
2001-08-17 20:57 ` David S. Miller
2001-08-17 21:40 ` Alan Cox
2001-08-17 22:09 ` Alan Cox
2001-08-17 22:11 ` David S. Miller
2001-08-17 23:34   ` Daniel Phillips
2001-08-17 23:38   ` David S. Miller
     [not found] <Your message of "Fri, 17 Aug 2001 00:35:02 +0100."  <5.1.0.14.2.20010817002825.00b1e4e0@pop.cus.cam.ac.uk.suse.lists.linux.kernel>
     [not found] ` <5.1.0.14.2.20010817015007.045689b0@pop.cus.cam.ac.uk.suse.lists.linux.kernel>
     [not found]   ` <3B7C7846.FD9DEE68@zip.com.au.suse.lists.linux.kernel>
     [not found]     ` <20010816.185319.88475216.davem@redhat.com.suse.lists.linux.kernel>
2001-08-17  7:30       ` Andi Kleen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5.1.0.14.2.20010816231824.00a94b70@pop.cus.cam.ac.uk \
    --to=aia21@cam.ac.uk \
    --cc=f5ibh@db0bm.ampr.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tpepper@vato.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).