From: Finn Thain <fthain@telegraphics.com.au>
To: "Justin P. Mattock" <justinmattock@gmail.com>
Cc: trivial@kernel.org, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org,
linux-m32r@ml.linux-m32r.org, linux-m68k@lists.linux-m68k.org,
linux-mips@linux-mips.org, linuxppc-dev@lists.ozlabs.org,
linux-laptop@vger.kernel.org,
"Maciej W. Rozycki" <macro@linux-mips.org>,
Randy Dunlap <rdunlap@xenotime.net>
Subject: Re: [PATCH 1/2 v3]Update broken web addresses.
Date: Wed, 22 Sep 2010 13:25:46 +1000 (EST) [thread overview]
Message-ID: <alpine.LNX.2.00.1009221310500.4570@nippy.intranet> (raw)
In-Reply-To: <1285118957-24965-1-git-send-email-justinmattock@gmail.com>
On Tue, 21 Sep 2010, Justin P. Mattock wrote:
> Below is an update from the original, with changes to the broken web
> addresses and removal of archive.org and moved to a seperate patch for
> you guys to decide if you want to use this and/or just leave the old url
> in and leave it at that..
> Please dont apply this to anything just comments and fixes for now,
You can use [RFC] in the Subject line instead of [PATCH] to indicate this.
> diff --git a/arch/m68k/mac/macboing.c b/arch/m68k/mac/macboing.c
> index 8f06408..234d9ee 100644
> --- a/arch/m68k/mac/macboing.c
> +++ b/arch/m68k/mac/macboing.c
> @@ -114,7 +114,8 @@ static void mac_init_asc( void )
> * 16-bit I/O functionality. The PowerBook 500 series computers
> * support 16-bit stereo output, but only mono input."
> *
> - * http://til.info.apple.com/techinfo.nsf/artnum/n16405
> + * Article number 16405:
> + * http://support.apple.com/kb/TA32601
> *
> * --David Kilzer
> */
"TIL article number 16405" would allow people to find the document by
number...
> diff --git a/arch/m68k/q40/README b/arch/m68k/q40/README
> index 6bdbf48..92806c0 100644
> --- a/arch/m68k/q40/README
> +++ b/arch/m68k/q40/README
> @@ -3,7 +3,7 @@ Linux for the Q40
>
> You may try http://www.geocities.com/SiliconValley/Bay/2602/ for
> some up to date information. Booter and other tools will be also
> -available from this place or ftp.uni-erlangen.de/linux/680x0/q40/
> +available from this place or http://www.linux-m68k.org/mail.html
> and mirrors.
>
> Hints to documentation usually refer to the linux source tree in
No. We already discussed this change. Please refer back to my review of
the first version of the patch. You got it right in the second version
(that I also reviewed), but now you've gone back to the first version...
Finn
next prev parent reply other threads:[~2010-09-22 3:25 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-22 1:29 [PATCH 1/2 v3]Update broken web addresses Justin P. Mattock
2010-09-22 1:29 ` [PATCH 2/2 v3]Update broken web addresses archive dot org only Justin P. Mattock
2010-09-22 3:25 ` Finn Thain [this message]
2010-09-22 3:57 ` [PATCH 1/2 v3]Update broken web addresses Justin P. Mattock
2010-09-22 10:58 ` Ralf Baechle
2010-09-22 13:06 ` Jeremy Kerr
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=alpine.LNX.2.00.1009221310500.4570@nippy.intranet \
--to=fthain@telegraphics.com.au \
--cc=justinmattock@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-laptop@vger.kernel.org \
--cc=linux-m32r@ml.linux-m32r.org \
--cc=linux-m68k@lists.linux-m68k.org \
--cc=linux-mips@linux-mips.org \
--cc=linux-omap@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=macro@linux-mips.org \
--cc=rdunlap@xenotime.net \
--cc=trivial@kernel.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 read-only IMAP folder(s) and NNTP newsgroup(s).