Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(5572)

Issue 10983043: robots.txt belongs in the root dir (issue 3430). (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 10 months ago by Julien Rioux
Modified:
10 years, 9 months ago
Reviewers:
Graham Percival, email, phileholmes, PhilEHolmes
Visibility:
Public.

Description

robots.txt belongs in the root dir (issue 3430). Same goes for favicon.ico, both of them should go to the root directory of the website. See http://code.google.com/p/lilypond/issues/detail?id=3430

Patch Set 1 #

Total comments: 1

Patch Set 2 : Leave tweets alone. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M make/website.make View 1 2 chunks +4 lines, -4 lines 0 comments Download

Messages

Total messages: 10
PhilEHolmes
A small but important comment. https://codereview.appspot.com/10983043/diff/1/make/website.make File make/website.make (right): https://codereview.appspot.com/10983043/diff/1/make/website.make#newcode158 make/website.make:158: $(OUT)/tweets.xml \ I've checked, ...
10 years, 9 months ago (2013-07-08 11:13:00 UTC) #1
Julien Rioux
Leave tweets alone.
10 years, 9 months ago (2013-07-08 12:44:13 UTC) #2
PhilEHolmes
LGTM. FWIW you can check this is working on the live server within less than ...
10 years, 9 months ago (2013-07-08 13:00:25 UTC) #3
Julien Rioux
On 2013/07/08 13:00:25, PhilEHolmes wrote: > LGTM. > > FWIW you can check this is ...
10 years, 9 months ago (2013-07-08 13:19:26 UTC) #4
email_philholmes.net
----- Original Message ----- From: <julien.rioux@gmail.com> To: <phileholmes@googlemail.com> Cc: <reply@codereview-hr.appspotmail.com> Sent: Monday, July 08, 2013 ...
10 years, 9 months ago (2013-07-08 13:38:19 UTC) #5
Graham Percival
I believe that the current version is correct. Phil, see http://lilypond.org/doc/v2.17/Documentation/contributor/uploading-and-security ### make-website.sh (halfway down ...
10 years, 9 months ago (2013-07-11 02:42:22 UTC) #6
email_philholmes.net
----- Original Message ----- From: <graham@percival-music.ca> To: <julien.rioux@gmail.com>; <phileholmes@googlemail.com>; <email@philholmes.net> Cc: <reply@codereview-hr.appspotmail.com> Sent: Thursday, July ...
10 years, 9 months ago (2013-07-11 08:42:07 UTC) #7
Graham Percival
On 2013/07/11 08:42:07, email_philholmes.net wrote: > However, I am currently certain that the current system ...
10 years, 9 months ago (2013-07-11 10:04:50 UTC) #8
email_philholmes.net
----- Original Message ----- From: <graham@percival-music.ca> To: <julien.rioux@gmail.com>; <phileholmes@googlemail.com>; <email@philholmes.net> Cc: <reply@codereview-hr.appspotmail.com> Sent: Thursday, July ...
10 years, 9 months ago (2013-07-11 15:53:00 UTC) #9
Graham Percival
10 years, 9 months ago (2013-07-11 20:36:33 UTC) #10
On Thu, Jul 11, 2013 at 04:52:49PM +0100, Phil Holmes wrote:
> So it looks like all that's necessary is to add
> 
> cp $DEST/website/robots.txt $DEST
> 
> to make-website.sh?  What about favicon.ico?

I guess so, and favico.ico should also be copied.

> BTW - you'll have noticed I've updated the 2 shell scripts that run
> as cron jobs to allow me to check their output.  You want this
> updated in the CG?

Yes please; anything that reduces our "oral culture" is a good
thing.

Cheers,
- Graham
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b