Re: Reduces langdefs.py warnings in doc build (issue 6487046)

2012-08-31 Thread John Mandereau
Il giorno ven, 31/08/2012 alle 16.04 +0100, Phil Holmes ha scritto: > I've just got back on this, and confirmed that adding > > $(MAKE) -C $(top-build-dir)/Documentation/po out=www messages > > to the top of the doc-stage-1 recipe gets rid of the warnings, so in fact it > makes the build work co

Re: Reduces langdefs.py warnings in doc build (issue 6487046)

2012-08-31 Thread Phil Holmes
- Original Message - From: To: ; ; Cc: ; Sent: Monday, August 27, 2012 3:37 PM Subject: Re: Reduces langdefs.py warnings in doc build (issue 6487046) I don't see the point of hiding/getting rid of these warnings, when 1) for now these warnings are harmless outside Document

Re: Reduces langdefs.py warnings in doc build (issue 6487046)

2012-08-27 Thread john . mandereau
http://codereview.appspot.com/6487046/diff/1/GNUmakefile.in File GNUmakefile.in (right): http://codereview.appspot.com/6487046/diff/1/GNUmakefile.in#newcode12 GNUmakefile.in:12: input Switching the build order of subdirectories here should not be relevant for solving the issue, if you need it th

Re: Reduces langdefs.py warnings in doc build (issue 6487046)

2012-08-27 Thread John Mandereau
2012/8/27 Phil Holmes : > With my patch, on my machine, I don't get warnings for the regtests, because > they're built after the docs (and therefore after the .mo files are > created). You should not rely on this in general: when documentation-dir variable is empty, and if your patch was applied a

Re: Reduces langdefs.py warnings in doc build (issue 6487046)

2012-08-27 Thread John Mandereau
2012/8/27 Phil Holmes : > I've updated generic-targets as you suggest, to this: > > > doc-stage-1: > make -C $(top-build-dir)/Documentation/po out=www messages > $(MAKE) -C $(depth)/scripts/build out= > $(MAKE) out=www WWW-1 > > It builds the .mo files, as you were intending, but I get this error:

Re: Reduces langdefs.py warnings in doc build (issue 6487046)

2012-08-27 Thread Phil Holmes
- Original Message - From: "John Mandereau" To: "Phil Holmes" Cc: "Lily devel" Sent: Monday, August 27, 2012 4:20 PM Subject: Re: Reduces langdefs.py warnings in doc build (issue 6487046) 2012/8/27 Phil Holmes : - Original Message - From:

Re: Reduces langdefs.py warnings in doc build (issue 6487046)

2012-08-27 Thread Phil Holmes
- Original Message - From: "John Mandereau" To: "Phil Holmes" Cc: "Lily devel" Sent: Monday, August 27, 2012 4:20 PM Subject: Re: Reduces langdefs.py warnings in doc build (issue 6487046) 2012/8/27 Phil Holmes : - Original Message - From:

Re: Reduces langdefs.py warnings in doc build (issue 6487046)

2012-08-27 Thread John Mandereau
2012/8/27 Phil Holmes : > - Original Message - From: > To: ; ; > > Cc: ; > Sent: Monday, August 27, 2012 3:37 PM > Subject: Re: Reduces langdefs.py warnings in doc build (issue 6487046) > > > >> I don't see the point of hiding/getting rid of th

Re: Reduces langdefs.py warnings in doc build (issue 6487046)

2012-08-27 Thread Phil Holmes
- Original Message - From: To: ; ; Cc: ; Sent: Monday, August 27, 2012 3:37 PM Subject: Re: Reduces langdefs.py warnings in doc build (issue 6487046) I don't see the point of hiding/getting rid of these warnings, when 1) for now these warnings are harmless outside Document

Re: Reduces langdefs.py warnings in doc build (issue 6487046)

2012-08-27 Thread john . mandereau
I don't see the point of hiding/getting rid of these warnings, when 1) for now these warnings are harmless outside Documentation/?? 2) if you want to get rid of the cause of the warning it should require not more than adding at the beginning of generic-targets.make:doc-stage-1 make -C $(top-build

Reduces langdefs.py warnings in doc build (issue 6487046)

2012-08-27 Thread PhilEHolmes
Reviewers: Graham Percival, John Mandereau, Reinhold, Message: Please review Description: Reduces langdefs.py warnings in doc build Doesn't quite get rid of them - one to sort. This has been interesting to fix. There were 2 issues - building the regtests before building the docs meant that th