Re: [www-patch] Change sample rsync command in rsync.html

Re: [www-patch] Change sample rsync command in rsync.html

[Volker Reichelt]

On  3 Jun, Gerald Pfeifer wrote:
> On Tue, 1 Jun 2004, Volker Reichelt wrote:
>> The following patch adds three --exclude options to the sample
>> rsync command in rsync.html to make it work with the latest
>> changes of the repository and to reduce unnecessary load.
>> See http://gcc.gnu.org/ml/gcc/2004-06/msg00043.html
> 
> This looks fine, thanks.
> 
>> The patch also wraps the sample commands in blockquotes and removes
>> the leading shell prompt (to match cvs.html).
> 
> Agreed.  In the mid term, we probably should switch to CSS.
> 
> Gerald

What about Andreas Schwab's suggestion:

> I'd also suggest to remove the --checksum option.  It adds considerable
> load to the process because it forces to read the contents of all files.
> Most of the time the use of timestamps and sizes is good enough.

Does everybody agree with him?

Regards,
Volker

[www-patch] Change sample rsync command in rsync.html

[Volker Reichelt]

The following patch adds three --exclude options to the sample
rsync command in rsync.html to make it work with the latest
changes of the repository and to reduce unnecessary load.
See http://gcc.gnu.org/ml/gcc/2004-06/msg00043.html

The patch also wraps the sample commands in blockquotes and removes
the leading shell prompt (to match cvs.html).

Checked as valid XHTML.

Ok to commit?

Regards,
Volker


Index: rsync.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/rsync.html,v
retrieving revision 1.9
diff -u -p -r1.9 rsync.html
--- rsync.html	14 Jul 2003 19:39:20 -0000	1.9
+++ rsync.html	1 Jun 2004 21:33:32 -0000
@@ -26,10 +26,12 @@ Subsequent synchronizations will be much
 a smart algorithm to only transfer differences over the network.</p>
 
 <p>Here is how you get a copy of the repository:</p>
-<pre>
- % rsync --archive --delete --checksum --compress \
-         --exclude '#cvs.*' rsync://gcc.gnu.org/gcc-cvs gcc-cvs
-</pre>
+<blockquote><pre>
+rsync --archive --delete --checksum --compress \
+      --exclude '#cvs.*' --exclude 'CVSROOT/config' \
+      --exclude 'CVSROOT/history' --exclude 'CVSROOT/updatelog' \
+      rsync://gcc.gnu.org/gcc-cvs gcc-cvs
+</pre></blockquote>
 <p>The same command can be run periodically to synchronize your copy of
 the repository.</p>
 
@@ -41,9 +43,9 @@ including during the initial phase that 
 <a href="http://rsync.samba.org">rsync.samba.org</a>.</p>
 
 <p>To get a list of available rsync targets, run:</p>
-<pre>
- % rsync rsync://gcc.gnu.org/
-</pre>
+<blockquote><pre>
+rsync rsync://gcc.gnu.org/
+</pre></blockquote>
 
 <p>Other interesting data, such as the GCC ftp directories, is also
 available.</p>
===================================================================

Re: [www-patch] Change sample rsync command in rsync.html

[Andreas Schwab]

Volker Reichelt <reichelt@igpm.rwth-aachen.de> writes:

> The following patch adds three --exclude options to the sample
> rsync command in rsync.html to make it work with the latest
> changes of the repository and to reduce unnecessary load.

I'd also suggest to remove the --checksum option.  It adds considerable
load to the process because it forces to read the contents of all files.
Most of the time the use of timestamps and sizes is good enough.

Andreas.

-- 
Andreas Schwab, SuSE Labs, schwab@suse.de
SuSE Linux AG, Maxfeldstraße 5, 90409 Nürnberg, Germany
Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

Re: [www-patch] Change sample rsync command in rsync.html

[Gerald Pfeifer]

On Tue, 1 Jun 2004, Volker Reichelt wrote:
> The following patch adds three --exclude options to the sample
> rsync command in rsync.html to make it work with the latest
> changes of the repository and to reduce unnecessary load.
> See http://gcc.gnu.org/ml/gcc/2004-06/msg00043.html

This looks fine, thanks.

> The patch also wraps the sample commands in blockquotes and removes
> the leading shell prompt (to match cvs.html).

Agreed.  In the mid term, we probably should switch to CSS.

Gerald